www.digitalmars.com         C & C++   DMDScript  

digitalmars.D.bugs - [Issue 11749] New: switch case fallthrough error is enabled with -w, but cannot be made informational warning

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

           Summary: switch case fallthrough error is enabled with -w, but
                    cannot be made informational warning
           Product: D
           Version: D2
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P2
         Component: DMD
        AssignedTo: nobody puremagic.com
        ReportedBy: r.sagitario gmx.de



PST ---
This code from issue 9642 (slightly modified to also trigger the message with
dmd 2.064):

void main() {
    int x;
    switch ('x') {
        case 'a':
            x++;
        case 'b':
            x++;
            break;
        default:
    }
}

compiles without additional options, but reports error (not warning) with -w
and -wi.


dmd fail9642.d && echo Success


Success


dmd -w fail9642.d && echo Success


fail9642.d(6): Error: switch case fallthrough - use 'goto case;' if intended


dmd -wi fail9642.d && echo Success


fail9642.d(6): Error: switch case fallthrough - use 'goto case;' if intended

Even with -wi compilation fails.

-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Dec 15 2013
next sibling parent d-bugmail puremagic.com writes:
https://d.puremagic.com/issues/show_bug.cgi?id=11749


Rainer Schuetze <r.sagitario gmx.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |pull



PST ---
https://github.com/D-Programming-Language/dmd/pull/2965

-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Dec 15 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
https://d.puremagic.com/issues/show_bug.cgi?id=11749


bearophile_hugs eml.cc changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bearophile_hugs eml.cc



I don't understand this issue. Isn't switch case fallthrough is always an error
in D?

-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Dec 15 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
https://d.puremagic.com/issues/show_bug.cgi?id=11749




Commit pushed to master at https://github.com/D-Programming-Language/dmd

https://github.com/D-Programming-Language/dmd/commit/f9111190ae1b38938a7b02676cc84a7b6e4ea1d2


fix  Issue 11749 - switch case fallthrough error is enabled with -w, but cannot
be made informational warning

-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Dec 15 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
https://d.puremagic.com/issues/show_bug.cgi?id=11749


yebblies <yebblies gmail.com> changed:

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


-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Dec 15 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
https://d.puremagic.com/issues/show_bug.cgi?id=11749




This should be an error in every case, I think.

-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Dec 15 2013
prev sibling parent d-bugmail puremagic.com writes:
https://d.puremagic.com/issues/show_bug.cgi?id=11749




PST ---

 This should be an error in every case, I think.


I guess you are right. But that would probably mean that it needs to go through
deprecation.
The patch only tries to remove "D with informational warnings" from the set of
languages with different semantics due to warnings.

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