On Sun, Feb 7, 2016 at 11:50 PM, Tony Mechelynck <[email protected]> wrote: > On Sun, Feb 7, 2016 at 3:57 PM, Bram Moolenaar <[email protected]> wrote: >> >> Dominique Pellé wrote: >> >>> Attached patch fixes the following compilation >>> warnings in vim-7.4.1273 with gcc-.5.2.1 on >>> Ubuntu MATE 15.10 on raspberry PI2: > [...] > > I had the same with gcc (SUSE Linux) 4.8.5 on openSUSE 42.1; and sorry > I sent an email about finding it (in Huge but not in Tiny) before I > saw this message. > >> >> I intentionally removed the "default" from the switch, so that it's >> easier to find places that need to change when a new variable type is >> added. I used that when adding VAR_JOB. >> >> I don't quite see how the "may be used uninitialized" warning can >> happen, since all possible enum values are covered. Does this compiler >> not recognize that? > > I suppose it's too much hassle to report it to the appropriate bug tracker. >> >> Perhaps it's better to assign a default value, even though it's dead >> code.
P.S. Maybe put the "least likely" case at the bottom, add /* */ around "case <value>:", and add a "default:" line just below that? > > Best regards, > Tony. -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are replying to. For more information, visit http://www.vim.org/maillist.php --- You received this message because you are subscribed to the Google Groups "vim_dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/d/optout.
