From f6b61b57159b252c4fba166f9e0ae771bde75cda Mon Sep 17 00:00:00 2001 From: Matt Tucker Date: Fri, 17 Jan 2003 17:50:56 +0000 Subject: [PATCH] Small fixes. git-svn-id: http://svn.igniterealtime.org/svn/repos/smack/trunk@1803 b35dd754-fafc-0310-a699-88a17e54d16e --- source/org/jivesoftware/smack/PacketCollector.java | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/source/org/jivesoftware/smack/PacketCollector.java b/source/org/jivesoftware/smack/PacketCollector.java index 4aae9d9c2..326adedc1 100644 --- a/source/org/jivesoftware/smack/PacketCollector.java +++ b/source/org/jivesoftware/smack/PacketCollector.java @@ -64,7 +64,7 @@ import java.util.LinkedList; * use than a {@link PacketListener} when you need to wait for a specific * result. * - * @see PacketReader#createPacketCollector(PacketFilter) + * @see XMPPConnection#createPacketCollector(PacketFilter) * @author Matt Tucker */ public class PacketCollector { @@ -85,8 +85,7 @@ public class PacketCollector { this.packetReader = packetReader; this.packetFilter = packetFilter; this.resultQueue = new LinkedList(); - - // Add the collector to the packet reader's list of active collector. + // Add the collector to the packet reader's list of active collector. synchronized (packetReader.collectors) { packetReader.collectors.add(this); } @@ -107,8 +106,10 @@ public class PacketCollector { // Remove object from collectors list by setting the value in the // list at the correct index to null. The collector thread will // automatically remove the actual list entry when it can. - int index = packetReader.collectors.indexOf(this); - packetReader.collectors.set(index, null); + synchronized (packetReader.collectors) { + int index = packetReader.collectors.indexOf(this); + packetReader.collectors.set(index, null); + } } }