Weasis icon indicating copy to clipboard operation
Weasis copied to clipboard

whats the reason for this code

Open paytam opened this issue 3 years ago • 3 comments

in class org.weasis.dicom.explorer.LocalImport method initialize protected void initialize(boolean afirst) { if (afirst) {} } if has no body, so you can remove the whole method. method usages are 2 points 1)constructor of class (initialize(true)) 2)in method resetSettingsToDefault() in that class, too(initialize(false))

paytam avatar Jun 29 '21 11:06 paytam

Yes for this particular page, there is no implementation for that method.

nroduit avatar Jun 30 '21 06:06 nroduit

So is it safe to remove the method? or do you have some usages in future development?

On Wed, Jun 30, 2021 at 10:51 AM Nicolas Roduit @.***> wrote:

Yes for this particular page, there is no implementation for that method.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/nroduit/Weasis/issues/198#issuecomment-871129377, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABLW6RYJBN47THSYH7MDCWLTVKZX7ANCNFSM47P4URPA .

-- Best regards, Mohammad.S.Paytam

paytam avatar Jul 01 '21 06:07 paytam

I don't think so. You can make a pull request to clean this class.

nroduit avatar Jul 02 '21 04:07 nroduit