<div dir="ltr">The Q_DECL_DEPRECATED_CONDITIONAL turned out to be more limited than I was anticipating.  I think using Q_DECL_DEPRECATED_X surrounded by ifdefs would be more explicit and accurate.</div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Wed, Oct 16, 2013 at 8:36 AM, Olivier Goffart <span dir="ltr"><<a href="mailto:olivier@woboq.com" target="_blank">olivier@woboq.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Saturday 12 October 2013 18:33:00 Keith Gardner wrote:<br>
> > > I had an idea about making conditional deprecation warnings for<br>
> > > functions<br>
> > > in Qt based on the destination platform.<br>
> ><br>
> > That might be useful, actually.<br>
> ><br>
> > But not for this case. Trying to call nativeArguments() on a non-Windows<br>
> > platform already signals the problem with a full error. It's impossible to<br>
> > ignore this.<br>
><br>
> Here are my changes:<br>
><br>
> Added Q_DECL_DEPRECATED_X(text)<br>
> <a href="https://codereview.qt-project.org/#change,68028" target="_blank">https://codereview.qt-project.org/#change,68028</a><br>
<br>
</div>Maybe there should also be a QT_DEPRECATED_X to go with QT_DEPRECATED<br>
and we should use QT_DEPRECATED_X ourselfs.<br>
<div class="im"><br>
> Added Q_DECL_DEPRECATED_CONDITIONAL(conditional, text)<br>
> <a href="https://codereview.qt-project.org/#change,68040" target="_blank">https://codereview.qt-project.org/#change,68040</a><br>
<br>
</div>Is there really an advantage of using this, which cannot work with complex<br>
expression  vs.<br>
<br>
#ifdef Q_OS_WIN<br>
Q_DECL_DEPRECATED_X("Not avaialble on windows")<br>
#endif<br>
void myFunction();<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Olivier<br>
<br>
Woboq - Qt services and support - <a href="http://woboq.com" target="_blank">http://woboq.com</a> - <a href="http://code.woboq.org" target="_blank">http://code.woboq.org</a><br>
<br>
<br>
<br>
</font></span></blockquote></div><br></div>