AfflictionNetworks/samskivert
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Only sort our array if it has more than one element; fixed a bug where

Browse Source 
_size was not being decremented after removing an item.


git-svn-id: https://samskivert.googlecode.com/svn/trunk@586 6335cc39-0255-0410-8fd6-9bcaacd3b74c
This commit is contained in:
mdb
2002-02-19 03:53:37 +00:00
parent 8969525a40
commit a947e12e27
1 changed files with 9 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
projects/samskivert/src/java/com/samskivert/util/SortableArrayList.java
+9 -3
View File
@@ -1,5 +1,5 @@
// //
// $Id: SortableArrayList.java,v 1.1 2002/02/19 03:38:58 mdb Exp $ // $Id: SortableArrayList.java,v 1.2 2002/02/19 03:53:37 mdb Exp $
// //
// samskivert library - useful routines for java programs // samskivert library - useful routines for java programs
// Copyright (C) 2001 Michael Bayne // Copyright (C) 2001 Michael Bayne
@@ -62,7 +62,9 @@ public class SortableArrayList extends AbstractList
*/ */
public void sort (Comparator comp) public void sort (Comparator comp)
{ {
QuickSort.csort(_elements, 0, _size, comp); if (_size > 1) {
QuickSort.csort(_elements, 0, _size-1, comp);
}
} }
// documentation inherited from interface // documentation inherited from interface
@@ -156,7 +158,11 @@ public class SortableArrayList extends AbstractList
// documentation inherited from interface // documentation inherited from interface
public Object remove (int index) public Object remove (int index)
{ {
return ListUtil.remove(_elements, index); Object removed = ListUtil.remove(_elements, index);
if (removed != null) {
_size--;
}
return removed;
} }
// documentation inherited from interface // documentation inherited from interface