« Return to Thread: FilterList + WeakReferenceMatcherEditor --> IllegalState

Re: FilterList + WeakReferenceMatcherEditor --> IllegalState

by James Lemieux :: Rate this Message:

Hey Johannes,

A JUnit testcase was added, and the code fix submitted into HEAD. We'll product a JDK 1.5 snapshot build shortly. If this becomes a problem for a few 1.8 users in the field that cannot (or are unwilling) to move up to the head, we'll package the fix into a point release.

I'll respond on this thread when the snapshot build is updated with this change.

James

On Tue, Apr 7, 2009 at 10:14 AM, Johannes Schneider <mailings@...> wrote:

Do you have any idea when the next bugfix version will be released?

Thanks,

Johannes

On Thu, 2009-04-02 at 10:46 -0700, James Lemieux wrote:
> Johannes,
>
> This is entirely a bug in GL. Your fix is the correct one... we
> should be translating that MatcherEvent before sending it on. I'll
> submit that fix tonight.
>
> James
>
> On Thu, Apr 2, 2009 at 2:24 AM, <mailings@...> wrote:
> If I use a WeakReferenceMatcherEditor as matcher editor for a
> FilterList, I recieve the following exception:
>
> java.lang.IllegalStateException
> at
> ca.odell.glazedlists.FilterList.changeMatcher(FilterList.java:283)
> at
> ca.odell.glazedlists.FilterList.changeMatcherWithLocks(FilterList.java:269)
> at ca.odell.glazedlists.FilterList.access
> $100(FilterList.java:51)
> at ca.odell.glazedlists.FilterList
> $PrivateMatcherEditorListener.changedMatcher(FilterList.java:443)
> at
> ca.odell.glazedlists.impl.matchers.WeakReferenceMatcherEditor
> $WeakMatcherEditorListener.changedMatcher(WeakReferenceMatcherEditor.java:171)
> at
> ca.odell.glazedlists.impl.matchers.WeakReferenceMatcherEditor.changedMatcher(WeakReferenceMatcherEditor.java:112)
> at
> ca.odell.glazedlists.impl.matchers.WeakReferenceMatcherEditor
> $WeakMatcherEditorListener.changedMatcher(WeakReferenceMatcherEditor.java:171)
> at
> ca.odell.glazedlists.matchers.AbstractMatcherEditor.fireChangedMatcher(AbstractMatcherEditor.java:115)
> at
> ca.odell.glazedlists.matchers.AbstractMatcherEditor.fireChanged(AbstractMatcherEditor.java:63)
>
>
> I have patched the WeakReferenceMatcherEditor and changed the
> else path of
> de.avat.jview.diary.model.WeakReferenceMatcherEditor.WeakMatcherEditorListener#changedMatcher:
>
> } else {
> //Convert the event
> Event<E> convertedEvent = new Event<E>( editor,
> matcherEvent.getType(), matcherEvent.getMatcher() );
> matcherEditorListener.changedMatcher( convertedEvent );
> }
>
>
> which seems to solve the problem. Is this a bug in
> GlazedLists? Or (probably) an error in using GlazedLists?
>
>
> Regards,
>
> Johannes
>
>
>
>
>
> ---- Msg sent via @Mail - http://atmail.com/
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail:
> users-unsubscribe@...
> For additional commands, e-mail:
> users-help@...
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@...
For additional commands, e-mail: users-help@...

« Return to Thread: FilterList + WeakReferenceMatcherEditor --> IllegalState