Fix event listener unbinding #186, fixes #192

This commit is contained in:
Andrey Novikov 2016-09-28 15:06:27 +03:00 committed by Emux
parent 2b24bf82f9
commit 41c35ea114
2 changed files with 26 additions and 7 deletions

View File

@ -1,3 +1,20 @@
/*
* Copyright 2013 Hannes Janetzek
* Copyright 2016 Andrey Novikov
*
* This file is part of the OpenScienceMap project (http://www.opensciencemap.org).
*
* This program is free software: you can redistribute it and/or modify it under the
* terms of the GNU Lesser General Public License as published by the Free Software
* Foundation, either version 3 of the License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful, but WITHOUT ANY
* WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR A
* PARTICULAR PURPOSE. See the GNU Lesser General Public License for more details.
*
* You should have received a copy of the GNU Lesser General Public License along with
* this program. If not, see <http://www.gnu.org/licenses/>.
*/
package org.oscim.event; package org.oscim.event;
import org.oscim.utils.pool.LList; import org.oscim.utils.pool.LList;
@ -26,7 +43,7 @@ public abstract class EventDispatcher<E extends EventListener, T> {
if (LList.find(mListeners, listener) != null) { if (LList.find(mListeners, listener) != null) {
return; return;
} }
mListeners = LList.push(mListeners, new LList<E>(listener)); mListeners = LList.push(mListeners, listener);
} }
/** /**

View File

@ -1,5 +1,6 @@
/* /*
* Copyright 2013 Hannes Janetzek * Copyright 2013 Hannes Janetzek
* Copyright 2016 Andrey Novikov
* *
* This file is part of the OpenScienceMap project (http://www.opensciencemap.org). * This file is part of the OpenScienceMap project (http://www.opensciencemap.org).
* *
@ -36,18 +37,19 @@ public class LList<T> extends Inlist<LList<T>> {
return list.next; return list.next;
LList<T> prev = list; LList<T> prev = list;
for (LList<T> l = list.next; l != null; l = l.next) for (LList<T> l = list.next; l != null; l = l.next) {
if (l.data == item) { if (l.data == item) {
prev.next = l.next; prev.next = l.next;
break; break;
} }
prev = l;
}
return list; return list;
} }
public static <T extends LList<T>> LList<T> push(LList<T> list, T item) { public static <E extends LList<T>, T> LList<T> push(LList<T> list, T item) {
item.next = list; LList<T> prev = new LList<>(item);
return item; prev.next = list;
return prev;
} }
} }