[rfc][icedtea-web] PolicyEditor sorted entries list

Jiri Vanek jvanek at redhat.com
Wed Aug 19 12:27:31 UTC 2015


snip
>> Well ths one looks scary :D
>> I would change changelog a bit, as this method do not implement compare.
>> Also  I woudl rename it - like comapreComaprable() ?-)
>
> What do you mean... ? This patch makes it so that PolicyIdentifier implements
> Comparable<PolicyIdentifier> and this 'int compareTo(PolicyIdentifier policyIdentifier)' is there to
> fulfill that contract. The static helper method 'compare(T a, T b)' is just there to simplify
> null-checking and comparison of two Comparables, like codebases and signedBys.
>
> The first part of it is just guaranteeing that ALL_APPLETS_IDENTIFIER is always less than any other
> PolicyIdentifier, other than itself. The following parts just sort the identifiers normally with
> codebase being the primary sort attribute, then signedBy, and finally principals.

Yes I got that.  Please rename the static method. Ok to push then.

Thanx!
   J.



More information about the distro-pkg-dev mailing list