Thread overview
[Issue 11759] New: Poor error message trying to use lowercase L in literal suffix.
Dec 17, 2013
Peter Alexander
Dec 18, 2013
Kenji Hara
Dec 18, 2013
Kenji Hara
Dec 18, 2013
Kenji Hara
December 17, 2013
https://d.puremagic.com/issues/show_bug.cgi?id=11759

           Summary: Poor error message trying to use lowercase L in
                    literal suffix.
           Product: D
           Version: D2
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: DMD
        AssignedTo: nobody@puremagic.com
        ReportedBy: peter.alexander.au@gmail.com


--- Comment #0 from Peter Alexander <peter.alexander.au@gmail.com> 2013-12-17 13:23:12 PST ---
ulong x = 123ul;

ul.d(1): Error: semicolon expected, not 'l'
ul.d(1): Error: no identifier for declarator l


D, unlike C/C++, requires uppercase L for the suffix.

I think this will be a common enough mistake, and easy/unambiguous to detect, so I believe it deserves a better error message. Something along the lines of "Error: uppercase L required for long integer suffix".

-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
December 18, 2013
https://d.puremagic.com/issues/show_bug.cgi?id=11759


Kenji Hara <k.hara.pg@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |diagnostic, pull


--- Comment #1 from Kenji Hara <k.hara.pg@gmail.com> 2013-12-17 17:13:34 PST ---
https://github.com/D-Programming-Language/dmd/pull/2979 https://github.com/D-Programming-Language/dlang.org/pull/448

"D language spec" should not allow 'l' suffix, but compiler implementation can parse it for the particular use case.

-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
December 18, 2013
https://d.puremagic.com/issues/show_bug.cgi?id=11759


Kenji Hara <k.hara.pg@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |diagnostic, pull


--- Comment #1 from Kenji Hara <k.hara.pg@gmail.com> 2013-12-17 17:13:34 PST ---
https://github.com/D-Programming-Language/dmd/pull/2979 https://github.com/D-Programming-Language/dlang.org/pull/448

"D language spec" should not allow 'l' suffix, but compiler implementation can parse it for the particular use case.

--- Comment #2 from github-bugzilla@puremagic.com 2013-12-17 17:13:38 PST ---
Commits pushed to master at https://github.com/D-Programming-Language/dmd

https://github.com/D-Programming-Language/dmd/commit/e1d179032b08e313e0fdba3709cf66a0e8f9d99f
fix Issue 11759 - Poor error message trying to use lowercase L in literal
suffix

https://github.com/D-Programming-Language/dmd/commit/ccea9b03310ad89aa2d38ec3d90f24d91ecae526 Merge pull request #2979 from 9rnsr/fix11759

Issue 11759 - Poor error message trying to use lowercase L in literal suffix

-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
December 18, 2013
https://d.puremagic.com/issues/show_bug.cgi?id=11759



--- Comment #3 from github-bugzilla@puremagic.com 2013-12-17 17:15:33 PST ---
Commit pushed to master at https://github.com/D-Programming-Language/dlang.org

https://github.com/D-Programming-Language/dlang.org/commit/c4a66b074d44bbaa249eaac4990234a982bdd896 Add note for lower case suffix 'l' and allow it in lexing level.

Related: Issue 11759 - Poor error message trying to use lowercase L in literal suffix.

-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
December 18, 2013
https://d.puremagic.com/issues/show_bug.cgi?id=11759


Kenji Hara <k.hara.pg@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED


-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
December 19, 2013
https://d.puremagic.com/issues/show_bug.cgi?id=11759



--- Comment #4 from github-bugzilla@puremagic.com 2013-12-19 05:47:41 PST ---
Commit pushed to 2.065 at https://github.com/D-Programming-Language/dlang.org

https://github.com/D-Programming-Language/dlang.org/commit/c4a66b074d44bbaa249eaac4990234a982bdd896 Add note for lower case suffix 'l' and allow it in lexing level.

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