On 11 jun 2014, at 14:57, Alan Bateman <alan.bate...@oracle.com> wrote:

> On 11/06/2014 09:08, Staffan Larsen wrote:
>> 
>> I added:
>> 
>> +     * @throws  IllegalArgumentException
>> +     *          If keys or values in agentProperties are invalid.
>> +     *
> That is probably okay for now, thanks.
> 
> 
>> :
>> 
>> I would prefer to leave them in as they do no harm for non-Eclipse users and 
>> are only present in the tests.
>> 
> I don't use Eclipse but I will guess that it shouldn't complain if these 
> methods are package-private (just an approach that would avoid suppressing 
> non-javac warnings).

Yes, that works. I will push with that change.

Thanks,
/Staffan



> 
> -Alan

Reply via email to