<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Jun 29, 2015, at 11:58 AM, Ed Maste <<a href="mailto:emaste@freebsd.org" class="">emaste@freebsd.org</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="">On 29 June 2015 at 14:47, Enrico Granata <<a href="mailto:egranata@apple.com" class="">egranata@apple.com</a>> wrote:<br class=""><blockquote type="cite" class=""><br class=""><blockquote type="cite" class="">I can see arguments for and against removing the default, but I think<br class="">removing it will cause a new warning to appear if Clang adds other new<br class="">kinds in the future.<br class=""></blockquote><br class="">Yes. And I am slightly more worried about the silent failure when that happens rather than the warning per se - which is why I am hesitant to just say “yes please remove"<br class=""></blockquote><br class="">I mean the warning will prompt us to go and add any new missing enum<br class="">values </div></div></blockquote><div><br class=""></div><div>Until we’re busy and forget to do it and some builds end up with bugs because of silent fall-through at runtime :-)</div><br class=""><blockquote type="cite" class=""><div class=""><div class="">if this happens -- especially if we enable -Werror :)<br class=""></div></div></blockquote><div><br class=""></div><div>Which I suspect we’re not doing anytime soon</div><br class=""><blockquote type="cite" class=""><div class=""><div class=""><br class="">I don't feel too strongly one way or the other, </div></div></blockquote><div><br class=""></div><div>Likewise. I have a slight preference for keeping the default in there, but only so slight</div><br class=""><blockquote type="cite" class=""><div class=""><div class="">but it seems we have<br class="">removed the defaults in all of these cases in the past.</div></div></blockquote></div><div class="">
<div class="" style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><br class="Apple-interchange-newline">Thanks,</div><div class="" style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><i class="">- Enrico</i><br class="">📩 egranata@<font color="#ff2600" class=""></font>.com ☎️ 27683</div>
</div>
<br class=""></body></html>