You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jspwiki.apache.org by me...@apache.org on 2012/12/12 19:18:36 UTC
svn commit: r1420901 - in /incubator/jspwiki/branches/JSPWIKI_2_8_BRANCH:
ChangeLog src/com/ecyrd/jspwiki/Release.java
src/com/ecyrd/jspwiki/util/PriorityList.java
Author: metskem
Date: Wed Dec 12 18:18:33 2012
New Revision: 1420901
URL: http://svn.apache.org/viewvc?rev=1420901&view=rev
Log:
Index: src/com/ecyrd/jspwiki/util/PriorityList.java
===================================================================
--- src/com/ecyrd/jspwiki/util/PriorityList.java (revision 1362567)
+++ src/com/ecyrd/jspwiki/util/PriorityList.java (working copy)
@@ -21,7 +21,7 @@
package com.ecyrd.jspwiki.util;
import java.util.AbstractList;
-import java.util.ArrayList;
+import java.util.concurrent.CopyOnWriteArrayList;
/**
* Builds a simple, priority-based List implementation. The list
@@ -32,11 +32,10 @@
* Priority is an integer, and the list is sorted in descending order
* (that is, 100 is before 10 is before 0 is before -40).
*/
-@SuppressWarnings("unchecked")
public class PriorityList
extends AbstractList
{
- private ArrayList<Object> m_elements = new ArrayList<Object>();
+ private CopyOnWriteArrayList<Item> m_elements = new CopyOnWriteArrayList<Item>();
/**
* This is the default priority, which is used if no priority
@@ -57,7 +56,7 @@
for( ; i < m_elements.size(); i++ )
{
- Item item = (Item) m_elements.get(i);
+ Item item = m_elements.get(i);
if( item.m_priority < priority )
{
@@ -93,7 +92,7 @@
*/
public Object get( int index )
{
- return ((Item)m_elements.get( index )).m_object;
+ return m_elements.get( index ).m_object;
}
/**
Index: src/com/ecyrd/jspwiki/Release.java
===================================================================
--- src/com/ecyrd/jspwiki/Release.java (revision 1362567)
+++ src/com/ecyrd/jspwiki/Release.java (working copy)
@@ -77,7 +77,7 @@
* <p>
* If the build identifier is empty, it is not added.
*/
- public static final String BUILD = "13";
+ public static final String BUILD = "14";
/**
* This is the generic version string you should use
Index: ChangeLog
===================================================================
--- ChangeLog (revision 1362567)
+++ ChangeLog (working copy)
@@ -1,3 +1,9 @@
+2012-12-12 Harry Metske <me...@apache.org>
+
+ * 2.8.5-svn-14
+
+ * fixed JSPWIKI-742 NullPointerException in PriorityList (reported by Rakesh K. Cherukuri)
+
2012-04-18 Harry Metske <me...@apache.org>
* 2.8.5-svn-13
Modified:
incubator/jspwiki/branches/JSPWIKI_2_8_BRANCH/ChangeLog
incubator/jspwiki/branches/JSPWIKI_2_8_BRANCH/src/com/ecyrd/jspwiki/Release.java
incubator/jspwiki/branches/JSPWIKI_2_8_BRANCH/src/com/ecyrd/jspwiki/util/PriorityList.java
Modified: incubator/jspwiki/branches/JSPWIKI_2_8_BRANCH/ChangeLog
URL: http://svn.apache.org/viewvc/incubator/jspwiki/branches/JSPWIKI_2_8_BRANCH/ChangeLog?rev=1420901&r1=1420900&r2=1420901&view=diff
==============================================================================
--- incubator/jspwiki/branches/JSPWIKI_2_8_BRANCH/ChangeLog (original)
+++ incubator/jspwiki/branches/JSPWIKI_2_8_BRANCH/ChangeLog Wed Dec 12 18:18:33 2012
@@ -1,3 +1,9 @@
+2012-12-12 Harry Metske <me...@apache.org>
+
+ * 2.8.5-svn-14
+
+ * fixed JSPWIKI-742 NullPointerException in PriorityList (reported by Rakesh K. Cherukuri)
+
2012-04-18 Harry Metske <me...@apache.org>
* 2.8.5-svn-13
Modified: incubator/jspwiki/branches/JSPWIKI_2_8_BRANCH/src/com/ecyrd/jspwiki/Release.java
URL: http://svn.apache.org/viewvc/incubator/jspwiki/branches/JSPWIKI_2_8_BRANCH/src/com/ecyrd/jspwiki/Release.java?rev=1420901&r1=1420900&r2=1420901&view=diff
==============================================================================
--- incubator/jspwiki/branches/JSPWIKI_2_8_BRANCH/src/com/ecyrd/jspwiki/Release.java (original)
+++ incubator/jspwiki/branches/JSPWIKI_2_8_BRANCH/src/com/ecyrd/jspwiki/Release.java Wed Dec 12 18:18:33 2012
@@ -77,7 +77,7 @@ public final class Release
* <p>
* If the build identifier is empty, it is not added.
*/
- public static final String BUILD = "13";
+ public static final String BUILD = "14";
/**
* This is the generic version string you should use
Modified: incubator/jspwiki/branches/JSPWIKI_2_8_BRANCH/src/com/ecyrd/jspwiki/util/PriorityList.java
URL: http://svn.apache.org/viewvc/incubator/jspwiki/branches/JSPWIKI_2_8_BRANCH/src/com/ecyrd/jspwiki/util/PriorityList.java?rev=1420901&r1=1420900&r2=1420901&view=diff
==============================================================================
--- incubator/jspwiki/branches/JSPWIKI_2_8_BRANCH/src/com/ecyrd/jspwiki/util/PriorityList.java (original)
+++ incubator/jspwiki/branches/JSPWIKI_2_8_BRANCH/src/com/ecyrd/jspwiki/util/PriorityList.java Wed Dec 12 18:18:33 2012
@@ -21,7 +21,7 @@
package com.ecyrd.jspwiki.util;
import java.util.AbstractList;
-import java.util.ArrayList;
+import java.util.concurrent.CopyOnWriteArrayList;
/**
* Builds a simple, priority-based List implementation. The list
@@ -32,11 +32,10 @@ import java.util.ArrayList;
* Priority is an integer, and the list is sorted in descending order
* (that is, 100 is before 10 is before 0 is before -40).
*/
-@SuppressWarnings("unchecked")
public class PriorityList
extends AbstractList
{
- private ArrayList<Object> m_elements = new ArrayList<Object>();
+ private CopyOnWriteArrayList<Item> m_elements = new CopyOnWriteArrayList<Item>();
/**
* This is the default priority, which is used if no priority
@@ -57,7 +56,7 @@ public class PriorityList
for( ; i < m_elements.size(); i++ )
{
- Item item = (Item) m_elements.get(i);
+ Item item = m_elements.get(i);
if( item.m_priority < priority )
{
@@ -93,7 +92,7 @@ public class PriorityList
*/
public Object get( int index )
{
- return ((Item)m_elements.get( index )).m_object;
+ return m_elements.get( index ).m_object;
}
/**