[rfc][icedtea-web] Temporary Permissions as check boxes

Lukasz Dracz ldracz at redhat.com
Tue Jun 3 13:32:04 UTC 2014



----- Original Message -----
> 
> +                temporaryPermissions.addAll((Collection<? extends
> Permission>) permissions);
> 
> Is the cast here necessary? Please remove it if not. Otherwise, this
> looks good.
> 
> Thanks,
> 
> --
> Andrew A
> 
> 
Hello,

Your right the cast was unnecessary and I have removed it.
The changed patch is attached.

Thank you,
Lukasz Dracz

-------------- next part --------------
A non-text attachment was scrubbed...
Name: CheckBoxMenuItemForTemporaryPermissions-3.patch
Type: text/x-patch
Size: 6866 bytes
Desc: not available
URL: <http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20140603/ced1eb6b/CheckBoxMenuItemForTemporaryPermissions-3.patch>


More information about the distro-pkg-dev mailing list