Skip to content

Commit 205b624

Browse files
author
Hideki Itakura
committed
Merge pull request #1175 from ashihaby/master
Fixed #1174 - Use a synchronizedList for document's change listeners
2 parents 5da9e5b + a40adba commit 205b624

1 file changed

Lines changed: 5 additions & 3 deletions

File tree

src/main/java/com/couchbase/lite/Document.java

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ public class Document {
3333
/**
3434
* Change Listeners
3535
*/
36-
private List<ChangeListener> changeListeners = new ArrayList<ChangeListener>();
36+
private List<ChangeListener> changeListeners = Collections.synchronizedList(new ArrayList<ChangeListener>());
3737

3838
/**
3939
* Constructor
@@ -496,8 +496,10 @@ else if (rev.isDeleted())
496496
}
497497

498498
if (notify) {
499-
for (ChangeListener listener : changeListeners) {
500-
listener.changed(new ChangeEvent(this, change));
499+
synchronized (changeListeners) {
500+
for (ChangeListener listener : changeListeners) {
501+
listener.changed(new ChangeEvent(this, change));
502+
}
501503
}
502504
}
503505
}

0 commit comments

Comments
 (0)