CVSROOT: /sources/classpath
Module name: classpath
Changes by: David Gilbert <trebligd> 06/06/21 21:01:32
Modified files:
. : ChangeLog
javax/swing : DefaultListSelectionModel.java
Log message:
2006-06-21 David Gilbert <[EMAIL PROTECTED]>
* javax/swing/DefaultListSelectionModel.java
(getSelectionMode): Updated API docs,
(setAnchorSelectionIndex): Added ListSelectionEvent generation,
(addSelectionInterval): If mode is SINGLE_SELECTION, just call
setSelectionInterval(),
(setSelectionInterval): Reimplemented SINGLE_SELECTION and
SINGLE_INTERVAL_SELECTION cases.
----------------------------------------------------------------------
CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/classpath/ChangeLog?cvsroot=classpath&r1=1.7905&r2=1.7906
http://cvs.savannah.gnu.org/viewcvs/classpath/javax/swing/DefaultListSelectionModel.java?cvsroot=classpath&r1=1.31&r2=1.32
Patches:
Index: ChangeLog
===================================================================
RCS file: /sources/classpath/classpath/ChangeLog,v
retrieving revision 1.7905
retrieving revision 1.7906
diff -u -b -r1.7905 -r1.7906
--- ChangeLog 21 Jun 2006 17:04:41 -0000 1.7905
+++ ChangeLog 21 Jun 2006 21:01:30 -0000 1.7906
@@ -1,3 +1,13 @@
+2006-06-21 David Gilbert <[EMAIL PROTECTED]>
+
+ * javax/swing/DefaultListSelectionModel.java
+ (getSelectionMode): Updated API docs,
+ (setAnchorSelectionIndex): Added ListSelectionEvent generation,
+ (addSelectionInterval): If mode is SINGLE_SELECTION, just call
+ setSelectionInterval(),
+ (setSelectionInterval): Reimplemented SINGLE_SELECTION and
+ SINGLE_INTERVAL_SELECTION cases.
+
2006-06-21 Roman Kennke <[EMAIL PROTECTED]>
* javax/swing/text/AbstractDocument.java
Index: javax/swing/DefaultListSelectionModel.java
===================================================================
RCS file:
/sources/classpath/classpath/javax/swing/DefaultListSelectionModel.java,v
retrieving revision 1.31
retrieving revision 1.32
diff -u -b -r1.31 -r1.32
--- javax/swing/DefaultListSelectionModel.java 26 Apr 2006 15:29:34 -0000
1.31
+++ javax/swing/DefaultListSelectionModel.java 21 Jun 2006 21:01:32 -0000
1.32
@@ -150,9 +150,14 @@
boolean setLeadCalledFromAdd = false;
/**
- * Gets the value of the [EMAIL PROTECTED] #selectionMode} property.
+ * Returns the selection mode, which is one of [EMAIL PROTECTED]
#SINGLE_SELECTION},
+ * [EMAIL PROTECTED] #SINGLE_INTERVAL_SELECTION} and
+ * [EMAIL PROTECTED] #MULTIPLE_INTERVAL_SELECTION}. The default value is
+ * [EMAIL PROTECTED] #MULTIPLE_INTERVAL_SELECTION}.
*
- * @return The current value of the property
+ * @return The selection mode.
+ *
+ * @see #setSelectionMode(int)
*/
public int getSelectionMode()
{
@@ -187,13 +192,19 @@
/**
* Sets the value of the [EMAIL PROTECTED] #anchorSelectionIndex} property.
*
- * @param anchorIndex The new property value
+ * @param index The new property value
*
* @see #getAnchorSelectionIndex
*/
- public void setAnchorSelectionIndex(int anchorIndex)
+ public void setAnchorSelectionIndex(int index)
+ {
+ if (anchorSelectionIndex != index)
{
- anchorSelectionIndex = anchorIndex;
+ int old = anchorSelectionIndex;
+ anchorSelectionIndex = index;
+ if (leadAnchorNotificationEnabled)
+ fireValueChanged(index, old);
+ }
}
/**
@@ -459,12 +470,14 @@
if (index0 == -1 || index1 == -1)
return;
+ if (selectionMode == SINGLE_SELECTION)
+ setSelectionInterval(index0, index1);
+ else
+ {
int lo = Math.min(index0, index1);
int hi = Math.max(index0, index1);
oldSel = sel.clone();
- if (selectionMode == SINGLE_SELECTION)
- setSelectionInterval(index0, index1);
// COMPAT: Like Sun (but not like IBM), we allow calls to
// addSelectionInterval when selectionMode is
@@ -504,6 +517,7 @@
fireDifference(sel, (BitSet) oldSel);
}
}
+ }
/**
@@ -588,28 +602,83 @@
* the current selection mode is <code>SINGLE_SELECTION</code> only the
* index <code>index2</code> is selected.
*
- * @param index0 The low end of the new selection
- * @param index1 The high end of the new selection
+ * @param anchor the anchor selection index.
+ * @param lead the lead selection index.
*/
- public void setSelectionInterval(int index0, int index1)
+ public void setSelectionInterval(int anchor, int lead)
{
- if (index0 == -1 || index1 == -1)
+ if (anchor == -1 || lead == -1)
return;
+ if (selectionMode == SINGLE_SELECTION)
+ {
+ int lo = lead;
+ int hi = lead;
+ int selected = sel.nextSetBit(0);
+ if (selected == lead)
+ return; // the selection is not changing
+ if (selected >= 0)
+ {
+ lo = Math.min(lo, selected);
+ hi = Math.max(hi, selected);
+ }
+ if (anchorSelectionIndex >= 0)
+ {
+ lo = Math.min(lo, anchorSelectionIndex);
+ hi = Math.max(hi, anchorSelectionIndex);
+ }
+ sel.clear();
+ sel.set(lead);
+ leadSelectionIndex = lead;
+ anchorSelectionIndex = lead;
+ fireValueChanged(lo, hi);
+ }
+ else if (selectionMode == SINGLE_INTERVAL_SELECTION)
+ {
+ // determine the current interval
+ int first = sel.nextSetBit(0);
+ int last = first;
+ if (first >= 0)
+ last += (sel.cardinality() - 1);
+
+ // update the selection
+ int lo = Math.min(anchor, lead);
+ int hi = Math.max(anchor, lead);
+ if (lo == first && hi == last)
+ return; // selected interval is not being changed
+ sel.clear();
+ sel.set(lo, hi + 1);
+ // include the old selection in the event range
+ if (first >= 0)
+ lo = Math.min(lo, first);
+ if (last >= 0)
+ hi = Math.max(hi, last);
+ if (anchorSelectionIndex >= 0)
+ {
+ lo = Math.min(lo, anchorSelectionIndex);
+ hi = Math.max(hi, anchorSelectionIndex);
+ }
+ anchorSelectionIndex = anchor;
+ leadSelectionIndex = lead;
+ fireValueChanged(lo, hi);
+ }
+ else
+ {
BitSet oldSel = (BitSet) sel.clone();
sel.clear();
if (selectionMode == SINGLE_SELECTION)
- index0 = index1;
+ anchor = lead;
- int lo = Math.min(index0, index1);
- int hi = Math.max(index0, index1);
+ int lo = Math.min(anchor, lead);
+ int hi = Math.max(anchor, lead);
sel.set(lo, hi+1);
// update the anchorSelectionIndex and leadSelectionIndex variables
- setAnchorSelectionIndex(index0);
- leadSelectionIndex=index1;
+ setAnchorSelectionIndex(anchor);
+ leadSelectionIndex = lead;
fireDifference(sel, oldSel);
}
+ }
/**
* Inserts a number of indices either before or after a particular
