Fix looks good.

+1 on test renaming.

-Sundar


On 5/12/2016 12:02 AM, Mandy Chung wrote:
>> On May 11, 2016, at 11:00 AM, Jim Laskey (Oracle) <james.las...@oracle.com> 
>> wrote:
>>
>> http://cr.openjdk.java.net/~jlaskey/8156781/webrev/index.html
>> https://bugs.openjdk.java.net/browse/JDK-8156781
> Looks okay.  Since the plugin is renamed to OrderResourcesPlugin, maybe good 
> to rename it to OrderResourcesPluginTest.
>
> Mandy

Reply via email to