<br><br><div class="gmail_quote">On Mon, Feb 13, 2012 at 3:30 AM, David Faure <span dir="ltr"><<a href="mailto:faure@kde.org">faure@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">




 <div>
  <div style="font-family:Verdana,Arial,Helvetica,Sans-Serif"><div class="im">
   <table style="border:1px #c9c399 solid" bgcolor="#f9f3c9" cellpadding="8" width="100%">
    <tbody><tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="http://git.reviewboard.kde.org/r/103832/" target="_blank">http://git.reviewboard.kde.org/r/103832/</a>
     </td>
    </tr>
   </tbody></table>
   <br>





 </div><pre style="white-space:pre-wrap;white-space:-moz-pre-wrap;white-space:-pre-wrap;white-space:-o-pre-wrap;word-wrap:break-word">Out of curiosity, if the method returns void anyway, why is this attribute necessary? It's not like the the compiler is going to warn about the lack of a return value...</pre>

 <br></div></div></blockquote><div><br>Noreturn is not a return void.  noreturn tells the compiler that the program will stop right there, killing the app. <br>The <code>noreturn</code> keyword tells the compiler to assume that the function/method cannot return.  It can then optimize without regard to what
would happen if it ever did return.  This makes slightly
better code.  More importantly, it helps avoid spurious warnings of
uninitialized variables. 
<br><br>Tomaz<br><br>- David</div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div style="font-family:Verdana,Arial,Helvetica,Sans-Serif">
<div class="im">


<br>
<p>On January 31st, 2012, 8:58 p.m., Allen Winter wrote:</p>






</div><table style="background-image:url('');background-repeat:repeat-x;border:1px black solid" bgcolor="#fefadf" cellpadding="8" cellspacing="0" width="100%">
 <tbody><tr>
  <td><div class="im">

<div>Review request for kdelibs.</div>
<div>By Allen Winter.</div>


</div><p style="color:grey"><i>Updated Jan. 31, 2012, 8:58 p.m.</i></p><div class="im">






<h1 style="color:#575012;font-size:10pt;margin-top:1.5em">Description </h1>
 <table style="border:1px solid #b8b5a0" bgcolor="#ffffff" cellpadding="10" cellspacing="0" width="100%">
 <tbody><tr>
  <td>
   <pre style="margin:0;padding:0;white-space:pre-wrap;white-space:-moz-pre-wrap;white-space:-pre-wrap;white-space:-o-pre-wrap;word-wrap:break-word">This diff adds a new macro KDE_NO_RETURN that wraps the noreturn attribute which is enabled differently based on the compiler.</pre>

  </td>
 </tr>
</tbody></table>


<h1 style="color:#575012;font-size:10pt;margin-top:1.5em">Testing </h1>
<table style="border:1px solid #b8b5a0" bgcolor="#ffffff" cellpadding="10" cellspacing="0" width="100%">
 <tbody><tr>
  <td>
   <pre style="margin:0;padding:0;white-space:pre-wrap;white-space:-moz-pre-wrap;white-space:-pre-wrap;white-space:-o-pre-wrap;word-wrap:break-word">did a test compile</pre>
  </td>
 </tr>
</tbody></table>




<h1 style="color:#575012;font-size:10pt;margin-top:1.5em">Diffs </h1>
<ul style="margin-left:3em;padding-left:0">

 <li>kdemacros.h.cmake <span style="color:grey">(b93242c)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/103832/diff/" style="margin-left:3em" target="_blank">View Diff</a></p>




  </div></td>
 </tr>
</tbody></table>








  </div>
 </div>


</blockquote></div><br>