Hi Larry,<br><br>Perhaps that a `#warning` along the return NULL would be sufficient ?<br><br>It seems to me that the user should definitely take care not to use (even if the result is "coherent") until it's properly implemented.<br>
<br>Matthieu.<br><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Date: Sun, 13 Feb 2011 07:41:00 -0600<br>
From: Larry Evans <<a href="mailto:cppljevans@suddenlink.net">cppljevans@suddenlink.net</a>><br>
<br>
Here:<br>
<br>
  <a href="http://llvm.org/bugs/show_bug.cgi?id=8992#c7" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=8992#c7</a><br>
<br>
there's a workaround proposed for a bug; however,<br>
I'm worried that this workaround will just<br>
obscure the problem and make it harder for<br>
user's of the library to realize the library<br>
is not really doing the right thing.<br>
<br>
If clang had a:<br>
<br>
  #pragma message "not implmented"<br>
<br>
somewhat like g++'s then that could be used<br>
instead of the proposed workaround.<br>
<br>
There are several other places where:<br>
<br>
  #if __APPLE__<br>
<br>
occurs in locale.cpp and maybe all of them should<br>
adopt this method for flagging inimplemented features<br>
on linux.  For example, in:<br>
<br>
  <a href="http://llvm.org/svn/llvm-project/libcxx/trunk/src/locale.cpp" target="_blank">http://llvm.org/svn/llvm-project/libcxx/trunk/src/locale.cpp</a><br>
<br>
there's:<br>
<br>
> const wchar_t*<br>
> ctype<wchar_t>::do_is(const char_type* low, const char_type* high, mask* vec) const<br>
> {<br>
> #ifdef __APPLE__<br>
>     for (; low != high; ++low, ++vec)<br>
>         *vec = static_cast<mask>(isascii(*low) ? _DefaultRuneLocale.__runetype[*low] : 0);<br>
>     return low;<br>
> #else<br>
>     return NULL;<br>
> #endif<br>
> }<br>
<br>
In this case, is it better to return NULL on linux or issue a<br>
diagnostic with the proposed new clang '#pragma message'?<br>
<br>
-Larry<br>
<br>
<br>
<br>
<br>
------------------------------<br>
<br>
_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@cs.uiuc.edu">cfe-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>
<br>
<br>
End of cfe-dev Digest, Vol 44, Issue 49<br>
***************************************<br>
</blockquote></div><br>