[icedtea-web] RFC: use Arrays.asList instead of custom implementation

Omair Majid omajid at redhat.com
Mon Sep 16 08:39:38 PDT 2013


Hi Andrew,

On 09/16/2013 09:27 AM, Andrew Azores wrote:
> On 09/13/2013 05:19 PM, Omair Majid wrote:
>> There's a couple of places in icedtea-web where we are using a custom
>> method to do what's already being done by Arrays.asList. Lets remove
>> this 'duplicate' code.
> 
> Looks good to me. Not sure how I feel about the "import static" for the
> UnsignedAppletTrustConfirmationTest, but no big deal.

Do you see a (potential) issue? Or do you (only) have a stylistic
concern? There was already a static import, so I added another.

Fixed in updated patch.

Thanks,
Omair

-- 
PGP Key: 66484681 (http://pgp.mit.edu/)
Fingerprint = F072 555B 0A17 3957 4E95  0056 F286 F14F 6648 4681
-------------- next part --------------
A non-text attachment was scrubbed...
Name: arrays-as-list-02.patch
Type: text/x-patch
Size: 4425 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20130916/2399b47e/arrays-as-list-02.patch 


More information about the distro-pkg-dev mailing list