Hi Tom,
Thomas Fitzsimmons wrote:
@@ -185,8 +185,10 @@
{
if (listener == null)
return;
-
- progressListeners.add(listener);
+ if (progressListeners != null)
+ {
+ progressListeners.add(listener);
+ }
If progressListeners is null here it should be initialized to a new
ArrayList, otherwise no listener could ever be added. Otherwise this
patch looks good.
Ok, but this would apply to all addIIO*Listeners, right? In ImageReader
and ImageWriter. (progressListeners, updateListeners & warningListeners)
Would this be ok? Or should the ArrayList be initialized with the
default 10 elements? new ArrayList()?
public void addIIOReadProgressListener(IIOReadProgressListener listener)
{
if (listener == null)
return;
if (progressListeners != null)
{
progressListeners.add(listener);
}
else
{
progressListeners = new ArrayList(0);
progressListeners.add(listener);
}
}
Thanks for review!
Andreas
_______________________________________________
Classpath-patches mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/classpath-patches
