www.digitalmars.com         C & C++   DMDScript  

digitalmars.D.bugs - [Issue 10167] New: Wrong Document Comment on std.format.d(181)

reply d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=10167

           Summary: Wrong Document Comment on std.format.d(181)
           Product: D
           Version: D2
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Phobos
        AssignedTo: nobody puremagic.com
        ReportedBy: r.97all gmail.com


--- Comment #0 from Ryuichi OHORI <r.97all gmail.com> 2013-05-25 04:13:38 PDT
---
https://github.com/D-Programming-Language/phobos/blob/96c51fae9b12b0c09df7e28acdd0b3a400f35323/std/format.d

line 181-182:
$(TR $(TD $(B ' ')) $(TD numeric)) $(TD Prefix positive
    numbers in a signed conversion with a space.))

should be:
$(TR $(TD $(B ' ')) $(TD numeric) $(TD Prefix positive
    numbers in a signed conversion with a space.))

with an extra ')' removed.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
May 25 2013
next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=10167



--- Comment #1 from github-bugzilla puremagic.com 2013-05-25 14:55:23 PDT ---
Commits pushed to master at https://github.com/D-Programming-Language/phobos

https://github.com/D-Programming-Language/phobos/commit/d47fdc4d32c8a981476215be4648c71b8302c818
fix issue 10167

https://github.com/D-Programming-Language/phobos/commit/0440437bd5737cea78f52d387394b0415b5bf7e2
Merge pull request #1304 from majiang/patch-2

fix issue 10167

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
May 25 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=10167


Andrej Mitrovic <andrej.mitrovich gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |andrej.mitrovich gmail.com
         Resolution|                            |FIXED


-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
May 25 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=10167



--- Comment #2 from github-bugzilla puremagic.com 2013-05-27 02:34:19 PDT ---
Commit pushed to 2.063 at https://github.com/D-Programming-Language/phobos

https://github.com/D-Programming-Language/phobos/commit/d6488d76a8dfcb2ef6d2cd094cfaeb13f4df9e69
Merge pull request #1304 from majiang/patch-2

fix issue 10167

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
May 27 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=10167



--- Comment #3 from Ryuichi OHORI <r.97all gmail.com> 2013-05-28 01:16:28 PDT
---
I found that line 176 of std.format.d is wrong:
$(TR $(TD $(B '#')) $(TD numeric ($(B '0'))) $(TD Use leading
and thought it should be:
$(TR $(TD $(B '0')) $(TD numeric ($(B '0'))) $(TD Use leading
but Mr. Kenji Hara pointed that actually I could remove unnecessary text and
made it:
$(TR $(TD $(B '0')) $(TD numeric) $(TD Use leading

Sorry for making incomplete pull request first:
https://github.com/D-Programming-Language/phobos/pull/1307

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
May 28 2013
prev sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=10167



--- Comment #4 from github-bugzilla puremagic.com 2013-05-28 17:53:02 PDT ---
Commit pushed to https://github.com/D-Programming-Language/phobos

https://github.com/D-Programming-Language/phobos/commit/d6488d76a8dfcb2ef6d2cd094cfaeb13f4df9e69
Merge pull request #1304 from majiang/patch-2

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
May 28 2013