On 05/05/2012 11:22, sebb AT ASF wrote:
> On 4 May 2012 19:05, sebb AT ASF <s...@apache.org> wrote:
>> The following change to the GKOP ctor seems to fix the problem:
>>
>>
>> if (factory == null) {
>> this.close(); // <============= close the parent
>> throw new IllegalArgumentException("factory may not be null");
>> }
>>
>> Not 100% sure if that is the correct fix - perhaps tearDown() needs to
>> be adjusted?
>
> I've fixed GOP and GKOP to call jmxUnregister() instead.
> Hope that's OK; it fixes the remaining test errors.
Looks good to me.
Mark
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org
