<div dir="ltr">Hi,<div><br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div dir="ltr">
<div>About readable of code I just renamed this class to sb instead of buf, strbuf, etc. </div></div></blockquote><div><br></div><div><div><div>I doubt that renaming variables does really improve the code readability. And you changed the indentation (take look at other classes too): </div>
</div><div><br></div><div> 239     public String toString() {</div><div> 240         StringBuilder sb = new StringBuilder();</div><div> 241                 sb.append("init = ").append(init).append('(').append(init >> 10)</div>
<div> 242                                 .append("K) ");</div><div> 243                 sb.append("used = ").append(used).append('(').append(used >> 10)</div><div> 244                                 .append("K) ");</div>
<div> 245                 sb.append("committed = ").append(committed).append('(')</div><div> 246                                 .append(committed >> 10).append("K) ");</div><div> 247                 sb.append("max = ").append(max).append('(').append(max >> 10)</div>
<div> 248                                 .append("K)");</div><div> 249         return sb.toString();</div><div> 250     }</div></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div dir="ltr"><div>Because using StringBuilder beyond be more explicit I can use char instead of a single String.</div></div></blockquote><div><br></div><div>In the most classes I mentioned in my previous mail only the #toString()-methods would be affected by the proposal. And in the most cases, maybe in all cases, the #toString()-methods in this classes exists only to provide nice output. I would not consider them performance-critical. Btw. I see here a nice opportunity to create an RFE for the Javac team. Following code:</div>
<div><br></div><div>Object o1 = ...;</div><div>Object o2 = ...;</div><div>String s = "abc" + o1 + "c" + o2 + "\n";</div><div><br></div><div>should be translated to:</div><div><br></div><div>String s = new StringBuilder().append("abc").append(o1).append('c').append(o2).append('\n').toString();</div>
<div><br></div><div>instead of:</div><div><br></div><div>String s = new StringBuilder().append("abc").append(o1).append("c").append(o2).append("\n").toString();<br></div><div><br></div><div>Best regards,</div>
<div>Andrej Golovnin</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div>
<br></div><div><a href="https://dl.dropboxusercontent.com/u/16109193/open_jdk/string_builder_concat_3.zip" target="_blank">https://dl.dropboxusercontent.com/u/16109193/open_jdk/string_builder_concat_3.zip</a><br>


</div></div><div class="gmail_extra"><div><div class="h5"><br><br><div class="gmail_quote">On Mon, Aug 11, 2014 at 7:56 AM, Pavel Rappo <span dir="ltr"><<a href="mailto:pavel.rappo@oracle.com" target="_blank">pavel.rappo@oracle.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div>> In the class<br>
> src/share/classes/javax/management/openmbean/CompositeType.java you have<br>
> added the<br>
> annotation @SuppressWarnings("StringConcatenationInsideStringBufferAppend")<br>
> instead of fixing the concatenation inside the append method. Why?<br>
<br>
</div>+1 Moreover, I wonder where this value comes from? I've never seen it before. Here are warnings that javac supports:<br>
<br>
all,auxiliaryclass,cast,classfile,deprecation,dep-ann,divzero,empty,fallthrough,finally,options,overloads,overrides,path,processing,rawtypes,serial,static,try,unchecked,varargs<br>
<br>
It doesn't look like one of Eclipse's warnings either.<br>
<div><br>
> And I would like to suggest to drop explicit usage of StringBuilder in some<br>
> methods at all to improve code readability.<br>
<br>
</div>Agree.<br>
<span><font color="#888888"><br>
-Pavel<br>
<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div></div></div>-- <br><div dir="ltr"><div><div style="text-align:center"><div><span style="text-align:start">Otávio Gonçalves de Santana</span><div style="text-align:start">


<div style="text-align:center"><div class=""><br><div style="text-align:left"><font face="Tahoma">blog:     </font><a href="http://otaviosantana.blogspot.com.br/" style="text-align:center" target="_blank">http://otaviosantana.blogspot.com.br/</a></div>


<div style="text-align:left">twitter: <a href="http://twitter.com/otaviojava" target="_blank">http://twitter.com/otaviojava</a></div></div><div style="text-align:left">site:     <span style="text-align:center"><font color="#0000ee"><u><a href="http://about.me/otaviojava" target="_blank">http://about.me/otaviojava</a></u></font></span></div>


<div style="text-align:left">55 (11) 98255-3513</div></div></div></div></div></div></div>
</div>
</blockquote></div><br></div></div></div>