On Wed, 4 Dec 2024 13:33:56 GMT, Abhishek Kumar <[email protected]> wrote:

>> Prasanta Sadhukhan has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   review
>
> test/jdk/javax/swing/JToolBar/4529206/bug4529206.java line 1:
> 
>> 1: /*
> 
> Since you are modifying this test, couple of suggestions:
> 
> 1. Please limit L67 to 80 cols.
> 2. I guess L91 and L92 can be merged, it's slightly over 80 cols but should 
> be ok. If not then may be split like
>     throw new RuntimeException("setFloating" +
>                             "does not work correctly");

ok

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22536#discussion_r1869553180

Reply via email to