[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-openjfx-dev
Subject:    Re: RFR: 8328746: Remove unused imports in demo apps
From:       Karthik P K <kpk () openjdk ! org>
Date:       2024-03-28 9:07:38
Message-ID: 2i6hyUMSMJeg4vvARKhFe0eJalFOv36Kq7axbSUIitY=.8340f08f-6269-4984-84bb-2e810e054f88 () github ! com
[Download RAW message or body]

On Thu, 21 Mar 2024 21:50:37 GMT, Andy Goryachev <angorya@openjdk.org> wrote:

> Using Eclipse IDE to remove unused imports in **demo apps** (3D, Ensemble, etc.) \
> and update the copyright year to 2024. Using wildcard for more than 10 static \
> imports. 
> 
> --
> 
> This is a trivial change (though fairly large), 1 reviewer is probably enough.

Leaving it unspecified would be ok but some kind of soft recommendation could also \
help in my opinion. So there will be a direction for someone who is looking for a \
recommended way to add imports and have consistency class files.

> > static imports (sorted alphabetically)
> > one blank line
> > ordinary imports (sorted alphabetically)
> > No wildcard imports (I favor an exception for static imports).
> 
> +1 for something like this. 

+1 on this and no wildcard imports other than static imports.

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

PR Comment: https://git.openjdk.org/jfx/pull/1420#issuecomment-2024715556


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic