<div dir="ltr">+1 for the change. The explicit token is great for readability and will be important as other compilers start warning about these.<div><br></div><div>-1 for forbidding them. A lot of times it just makes the most sense to use a fall through, as long as it's clearly labeled as intentional.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Apr 7, 2015 at 1:49 PM, Sean McBride <span dir="ltr"><<a href="mailto:sean@rogue-research.com" target="_blank">sean@rogue-research.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, 7 Apr 2015 13:42:51 -0400, David Cole said:<br>
<br>
>Is there really no way to refactor and simply eliminate all the<br>
>"intentional" fall throughs? It's a really nasty style to be allowing.<br>
>It should be discouraged in general.<br>
<br>
</span>I did refactor a few, but most I left because I was scared to touch or they were pretty elegant with fall-through.  You can take a look:<br>
 <<a href="https://gitlab.kitware.com/vtk/vtk/merge_requests/59" target="_blank">https://gitlab.kitware.com/vtk/vtk/merge_requests/59</a>><br>
<br>
I'd be in favour of the coding standards saying something like "fall-through is discouraged, but when necessary must be annotated with VTK_FALLTHROUGH".<br>
<div class="HOEnZb"><div class="h5"><br>
Cheers,<br>
<br>
--<br>
____________________________________________________________<br>
Sean McBride, B. Eng                 <a href="mailto:sean@rogue-research.com">sean@rogue-research.com</a><br>
Rogue Research                        <a href="http://www.rogue-research.com" target="_blank">www.rogue-research.com</a><br>
Mac Software Developer              Montréal, Québec, Canada<br>
<br>
<br>
_______________________________________________<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Search the list archives at: <a href="http://markmail.org/search/?q=vtk-developers" target="_blank">http://markmail.org/search/?q=vtk-developers</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/mailman/listinfo/vtk-developers" target="_blank">http://public.kitware.com/mailman/listinfo/vtk-developers</a><br>
<br>
</div></div></blockquote></div><br></div>