RFR: 8162427: fix indent in CompileTask::print_tty

[Vladimir Kozlov]

I would like to avoid going through RFE approval process for this small change with explanation.
Please, convert it to bug.
Fix is good.

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);
>   }
>
>
>   // ------------------------------------------------------------------
>