RFR: 8162427: fix indent in CompileTask::print_tty

[Christian Thalinger]

> On Jul 22, 2016, at 12:39 PM, Christian Thalinger <cthalinger at twitter.com> wrote:
> 
> 
>> On Jul 22, 2016, at 9:41 AM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
>> 
>> I would like to avoid going through RFE approval process for this small change with explanation.
>> Please, convert it to bug.
> 
> Done.
> 
>> Fix is good.
> 
> jprt submit… oh, wait! :-)

Here is the changeset:

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 8162427.changeset
Type: application/octet-stream
Size: 949 bytes
Desc: not available
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20160722/1716b6fd/8162427.changeset>
-------------- next part --------------

> 
>> 
>> Thanks,
>> Vladimir
>> 
>> On 7/22/16 11:24 AM, Christian Thalinger wrote:
>>> Most annoying code snippet ever:
>>> 
>>> void CompileTask::print_tty() {
>>> ttyLockerttyl; // keep the following output all in one block
>>> // print compiler name if requested
>>> if (CIPrintCompilerName) tty->print("%s:", CompileBroker::compiler_name(comp_level()));
>>>    print(tty);
>>> }
>>> 
>>> The indent of print "suggests" it is under the if but of course it's not. Use braces!
>>> 
>>> Here is the patch:
>>> 
>>> diff -r 479631362b49 src/share/vm/compiler/compileTask.cpp
>>> --- a/src/share/vm/compiler/compileTask.cppThu Jun 16 20:57:05 2016 +0000
>>> +++ b/src/share/vm/compiler/compileTask.cppFri Jul 22 08:22:35 2016 -1000
>>> @@ -186,8 +186,10 @@
>>> void CompileTask::print_tty() {
>>>   ttyLocker ttyl;  // keep the following output all in one block
>>>   // print compiler name if requested
>>> -  if (CIPrintCompilerName) tty->print("%s:", CompileBroker::compiler_name(comp_level()));
>>> -    print(tty);
>>> +  if (CIPrintCompilerName) {
>>> +    tty->print("%s:", CompileBroker::compiler_name(comp_level()));
>>> +  }
>>> +  print(tty);
>>> }
>>> 
>>> 
>>> // ------------------------------------------------------------------
>>> 
>