On Thu, 27 Apr 2023 10:44:27 GMT, Andrey Turbanov <[email protected]> wrote:
>> Damon Nguyen has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Remove extra folders
>
> test/jdk/java/awt/dnd/SkipDropCompleteTest/SkipDropCompleteTest.java line 56:
>
>> 54: public void init() {
>> 55: sourceFrame = new SourceFrame();
>> 56: targetFrame = new TargetFrame();
>
> Suggestion:
>
> targetFrame = new TargetFrame();
Fixed
> test/jdk/java/awt/dnd/SkipDropCompleteTest/SourceFrame.java line 82:
>
>> 80: public void dragDropEnd(DragSourceDropEvent dsde) { }
>> 81:
>> 82: public void dragGestureRecognized(DragGestureEvent dge) {
>
> Suggestion:
>
> public void dragGestureRecognized(DragGestureEvent dge) {
Fixed
> test/jdk/java/awt/dnd/SkipDropCompleteTest/TransferableObject.java line 52:
>
>> 50: super();
>> 51: Vector v = new Vector();
>> 52: if(data instanceof String) {
>
> Strange indentation
> Suggestion:
>
> if (data instanceof String) {
Fixed
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13653#discussion_r1180495371
PR Review Comment: https://git.openjdk.org/jdk/pull/13653#discussion_r1180495443
PR Review Comment: https://git.openjdk.org/jdk/pull/13653#discussion_r1180495554
