Thanks for the review.
Please review the updated webrev :
http://cr.openjdk.java.net/~sundar/8159593/webrev.01/
Thanks
-Sundar
On 6/21/2016 5:33 PM, Jim Laskey (Oracle) wrote:
> 59 public static boolean isPostProcessor(Category category) {
> 60 return category.equals(Category.VERIFIER)
> 61 || category.equals(Category.PROCESSOR)
> 62 || category.equals(Category.PACKAGER);
> 63 }
> 64
> 65 public static boolean isPreProcessor(Category category) {
> 66 return category.equals(Category.COMPRESSOR)
> 67 || category.equals(Category.FILTER)
> 68 || category.equals(Category.MODULEINFO_TRANSFORMER)
> 69 || category.equals(Category.SORTER)
> 70 || category.equals(Category.TRANSFORMER)
> 71 || category.equals(Category.METAINFO_ADDER);
> 72 }
>
> Wondering if we can handle this with a field in the enum.
>
> +1 otherwise.
>
>
>> On Jun 21, 2016, at 7:25 AM, Sundararajan Athijegannathan
>> <[email protected]> wrote:
>>
>> Please review http://cr.openjdk.java.net/~sundar/8159593/webrev.00/ for
>> https://bugs.openjdk.java.net/browse/JDK-8159593
>>
>> Thanks,
>> -Sundar
