digitalmars.D.bugs - [Issue 4899] New: Ddoc's warnings about stray parens should include file and line numbers
- d-bugmail puremagic.com (25/25) Sep 19 2010 http://d.puremagic.com/issues/show_bug.cgi?id=4899
- d-bugmail puremagic.com (16/16) Sep 20 2010 http://d.puremagic.com/issues/show_bug.cgi?id=4899
- d-bugmail puremagic.com (13/13) Sep 20 2010 http://d.puremagic.com/issues/show_bug.cgi?id=4899
- d-bugmail puremagic.com (28/28) Sep 21 2010 http://d.puremagic.com/issues/show_bug.cgi?id=4899
- d-bugmail puremagic.com (10/10) Sep 21 2010 http://d.puremagic.com/issues/show_bug.cgi?id=4899
- d-bugmail puremagic.com (12/12) Jul 01 2013 http://d.puremagic.com/issues/show_bug.cgi?id=4899
- d-bugmail puremagic.com (14/14) Jul 02 2013 http://d.puremagic.com/issues/show_bug.cgi?id=4899
- d-bugmail puremagic.com (10/10) Jul 02 2013 http://d.puremagic.com/issues/show_bug.cgi?id=4899
http://d.puremagic.com/issues/show_bug.cgi?id=4899 Summary: Ddoc's warnings about stray parens should include file and line numbers Product: D Version: unspecified Platform: Other OS/Version: Linux Status: NEW Severity: normal Priority: P2 Component: DMD AssignedTo: nobody puremagic.com ReportedBy: jmdavisProg gmx.com PDT --- Ddoc can produce these two warnings: Warning: Ddoc: Stray '('. This may cause incorrect Ddoc output. Use $(LPAREN) instead for unpaired left parentheses. Warning: Ddoc: Stray ')'. This may cause incorrect Ddoc output. Use $(RPAREN) instead for unpaired right parentheses. However, if you'll notice, they don't specify which file or line number the warning is for. So, the warning borders on useless. It really should specify the file and line number where the problem is. -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email ------- You are receiving this mail because: -------
Sep 19 2010
http://d.puremagic.com/issues/show_bug.cgi?id=4899 Johannes Pfau <johannespfau gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |johannespfau gmail.com PDT --- Printing line numbers is implemented and should work. It works with the test case in 3554. test.d(6): Warning: Ddoc: Stray '('. This may cause incorrect Ddoc output. Use $(LPAREN) instead for unpaired left parentheses. test.d(16): Warning: Ddoc: Stray ')'. This may cause incorrect Ddoc output. Use $(RPAREN) instead for unpaired right parentheses. Could you provide a test case where the line numbers aren't printed? -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email ------- You are receiving this mail because: -------
Sep 20 2010
http://d.puremagic.com/issues/show_bug.cgi?id=4899 PDT --- /++ (See accompanying file LICENSE_1_0.txt or copy at http://www.boost.org/LICENSE_1_0.txt) +/ module d; void main() { } -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email ------- You are receiving this mail because: -------
Sep 20 2010
http://d.puremagic.com/issues/show_bug.cgi?id=4899 PDT --- OK, I can confirm that. So the line information is missing if the comment is a module comment. The dmd main function calls Module::gendocfile() which calls DocComment::writeSections() directly. In this case the Module's loc member is never set and therefore the line information is missing, I don't know how to fix this though. In case you're interested why the parenthesis error occurs: The second line in the comment, "http://www.boost.org/LICENSE_1_0.txt)" is the problem. Because of the colon dmd thinks you define a ddoc section named "http" with content "//www.boost.org/LICENSE_1_0.txt)". Because parenthesis must be closed in the section they were opened in the stray parenthesis error occurs. Possible workarounds are: 1. Join those two lines into one 2. Section names cannot contain spaces, so move the 'at' to the second line (See accompanying file LICENSE_1_0.txt or copy at http://www.boost.org/LICENSE_1_0.txt) 3. Use a macro. _Every_ macro will work (See accompanying file LICENSE_1_0.txt or copy at $(LINK http://www.boost.org/LICENSE_1_0.txt)) (See accompanying file LICENSE_1_0.txt or copy at $(TEXT http://www.boost.org/LICENSE_1_0.txt)) Macros: TEXT=$0 -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email ------- You are receiving this mail because: -------
Sep 21 2010
http://d.puremagic.com/issues/show_bug.cgi?id=4899 PDT --- that's from the license declaration used at the top of pretty much every module in Phobos, so unless they were changed recently, they're likely all generating this error if compiled with -w (I'm not sure whether Phobos is normally compiled with -w or not though). -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email ------- You are receiving this mail because: -------
Sep 21 2010
http://d.puremagic.com/issues/show_bug.cgi?id=4899 Kenji Hara <k.hara.pg gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |ddoc, pull Platform|Other |All OS/Version|Linux |All https://github.com/D-Programming-Language/dmd/pull/2291 -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email ------- You are receiving this mail because: -------
Jul 01 2013
http://d.puremagic.com/issues/show_bug.cgi?id=4899 Commits pushed to master at https://github.com/D-Programming-Language/dmd https://github.com/D-Programming-Language/dmd/commit/328969f671c9fcdc2fb7b9ba7ae65e483074961d fix Issue 4899 - Ddoc: Warnings about stray parens do not include file and line numbers for module comments Add ModuleDeclaration::loc and use it for "Stray paren" ddoc warning. https://github.com/D-Programming-Language/dmd/commit/23c3c6add8162693f85b3b41c9bf6550a71a57d3 Issue 4899 - Ddoc: Warnings about stray parens do not include file and line numbers for module comments -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email ------- You are receiving this mail because: -------
Jul 02 2013
http://d.puremagic.com/issues/show_bug.cgi?id=4899 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: -------
Jul 02 2013