<div dir="ltr">> <span style="font-size:12.8px">I assume the correct way to fix this is to disable -Wcovered-switch-default while compiling libcxx/src/experimental/</span><span style="font-size:12.8px">filesystem/operations.cpp</span><br style="font-size:12.8px"><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">Agreed. Disabled in r273092.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">Thanks for your patience with this latest change,</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">/Eric</span></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Jun 18, 2016 at 12:54 PM, Adrian Prantl <span dir="ltr"><<a href="mailto:aprantl@apple.com" target="_blank">aprantl@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello Eric,<br>
<br>
this commit causes new warnings on our bots:<br>
<br>
clang/src/projects/libcxx/include/fstream:816:5: warning: default label in switch which covers all enumeration values [-Wcovered-switch-default]<br>
    default:<br>
<br>
The problem is with this defensive default statement in fstream:<br>
<br>
<br>
 template <class _CharT, class _Traits><br>
0792 typename basic_filebuf<_CharT, _Traits>::pos_type<br>
0793 basic_filebuf<_CharT, _Traits>::seekoff(off_type __off, ios_base::seekdir __way,<br>
0794                                         ios_base::openmode)<br>
0795 {<br>
0796 #ifndef _LIBCPP_NO_EXCEPTIONS<br>
0797     if (!__cv_)<br>
0798         throw bad_cast();<br>
0799 #endif<br>
0800     int __width = __cv_->encoding();<br>
0801     if (__file_ == 0 || (__width <= 0 && __off != 0) || sync())<br>
0802         return pos_type(off_type(-1));<br>
0803     // __width > 0 || __off == 0<br>
0804     int __whence;<br>
0805     switch (__way)<br>
0806     {<br>
0807     case ios_base::beg:<br>
0808         __whence = SEEK_SET;<br>
0809         break;<br>
0810     case ios_base::cur:<br>
0811         __whence = SEEK_CUR;<br>
0812         break;<br>
0813     case ios_base::end:<br>
0814         __whence = SEEK_END;<br>
0815         break;<br>
0816     default:<br>
0817         return pos_type(off_type(-1));<br>
0818     }<br>
<br>
<br>
I assume the correct way to fix this is to disable -Wcovered-switch-default while compiling libcxx/src/experimental/filesystem/operations.cpp<br>
<br>
Could you please investigate?<br>
<br>
thanks,<br>
Adrian</blockquote></div><br></div>