I have not looked at entire file but since this function was modified, I made 
that suggestion. I am not that particular if you wish to ignore that.

Regards
Prasanta

Get Outlook for Android<https://aka.ms/AAb9ysg>
________________________________
From: 2d-dev <2d-dev-r...@openjdk.java.net> on behalf of Alexey Ivanov 
<aiva...@openjdk.java.net>
Sent: Wednesday, April 7, 2021 4:22:29 PM
To: 2d-dev@openjdk.java.net <2d-dev@openjdk.java.net>
Subject: Re: [OpenJDK 2D-Dev] RFR: 8263984: Invalidate printServices when there 
are no printers [v3]

On Wed, 7 Apr 2021 04:12:55 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org> 
wrote:

>> [Code Conventions for 
>> Java](https://www.oracle.com/java/technologies/javase/codeconventions-indentation.html#248)
>>  say, “Line wrapping for `if` statements should generally use the 8-space 
>> rule, since conventional (4 space) indentation makes seeing the body 
>> difficult.” (It's the second to last block on the page.)
>
> If we are adding a new line, then I think we should need to add at l129, l138 
> otherwise it will look odd doing it at one place only.

I haven't touched that code at all.
Not that odd because it's isolated to the new function now.

What is your suggestion? Refactor all if statements in these two functions? 
Submit a separate bug for refactor all if statements the entire file? Revert 
back to no new line, leaving this particular if untouched as well?

-------------

PR: https://git.openjdk.java.net/jdk/pull/3151

Reply via email to