<div dir="ltr">I can&#39;t speak for Qt, but this warning has been helpful for me in the past.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Sep 12, 2013 at 2:07 PM, Ryosuke Niwa <span dir="ltr">&lt;<a href="mailto:rniwa@webkit.org" target="_blank">rniwa@webkit.org</a>&gt;</span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi,<div><br></div><div><a href="http://trac.webkit.org/changeset/155643" target="_blank">http://trac.webkit.org/changeset/155643</a> broke Qt build with an error saying:</div>

<div><br></div><div><div>Source/JavaScriptCore/dfg/DFGGPRInfo.h:169:5: error: control reaches end of non-void function [-Werror=return-type] </div>

<div>cc1plus: all warnings being treated as errors</div></div><div><br></div><div>because of the following code:</div><div><br></div><div><div>GPRReg gpr(WhichValueWord which) const</div><div>{</div><div>    switch (which) {</div>



<div>    case TagWord:</div><div>        return tagGPR();</div><div>    case PayloadWord:</div><div>        return payloadGPR();</div><div>    }</div><div>}</div><div><br></div><div>But the code works just fine as is because WhichValueWord only takes two values (TagWord and PayloadWord) and they&#39;re all handled in the switch statement.</div>



<div><br></div><div>Can we disable this warning so that we don&#39;t have to add a bogus code like the one I had to add in <a href="http://trac.webkit.org/changeset/155649/trunk/Source/JavaScriptCore/dfg/DFGGPRInfo.h" target="_blank">http://trac.webkit.org/changeset/155649/trunk/Source/JavaScriptCore/dfg/DFGGPRInfo.h</a> ?</div>



<div><br></div><div><div>- R. Niwa</div><div><br></div>
</div></div></div>
<br>_______________________________________________<br>
webkit-dev mailing list<br>
<a href="mailto:webkit-dev@lists.webkit.org">webkit-dev@lists.webkit.org</a><br>
<a href="https://lists.webkit.org/mailman/listinfo/webkit-dev" target="_blank">https://lists.webkit.org/mailman/listinfo/webkit-dev</a><br>
<br></blockquote></div><br></div>