<p dir="ltr">Isn&#39;t signed overflow undefined in C and C++? Not sure if this code should explicitly check if it would overflow (version == max int) after increment to be pedantic.</p>
<p dir="ltr">Sent from my phone</p>
<div class="gmail_quote">On Jan 18, 2013 12:49 AM, &quot;<a href="mailto:serguei.spitsyn@oracle.com">serguei.spitsyn@oracle.com</a>&quot; &lt;<a href="mailto:serguei.spitsyn@oracle.com">serguei.spitsyn@oracle.com</a>&gt; wrote:<br type="attribution">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
The fix is good.<br>
<br>
Another way would be to use prefix increment instead of postfix increment:<br>
  _saved._version = (version &gt;=0) ? ++version : version; // keep<br>
<br>
Nice catch, David.<br>
<br>
Thanks,<br>
Serguei<br>
<br>
<br>
On 1/17/13 6:51 PM, Coleen Phillimore wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 1/17/2013 9:15 PM, David Holmes wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
void increment_and_save_version(int version) {<br>
_saved._version = (version &gt;=0) ? version++ : version; // keep<br>
overflowed value<br>
}<br>
</blockquote>
<br>
That saves the pre-incremented value. But otherwise it is a simpler formulation.<br>
<br>
David<br>
<br>
</blockquote>
<br>
I fixed it.  Please review.<br>
<br>
open webrev at <a href="http://cr.openjdk.java.net/~coleenp/8006548/" target="_blank">http://cr.openjdk.java.net/~<u></u>coleenp/8006548/</a><br>
bug link at <a href="http://bugs.sun.com/view_bug.do?bug_id=8006548" target="_blank">http://bugs.sun.com/view_bug.<u></u>do?bug_id=8006548</a><br>
<br>
thanks,<br>
Coleen<br>
</blockquote>
<br>
</blockquote></div>