Following up on things that -Xlint told me, som...

Following up on things that -Xlint told me, sometimes because Mr. Linty
was being sensible and other times just to shut him up so we can hear
better in the future.

- made xml.Equality public because it occurs in public method signatures
- made some actor classes with inaccessible-unoverridable methods final
- eliminated a bunch of "dead code follows" warnings by deleting the
dead code which really did follow - improved the reliability of warnings
about inaccessible types

For the changes in actors, review by phaller.

1 files changed, 2 insertions, 2 deletions

diff --git a/src/swing/scala/swing/event/ListEvent.scala b/src/swing/scala/swing/event/ListEvent.scala
index 9f8fcf7db8..b29e5936fe 100644
--- a/src/swing/scala/swing/event/ListEvent.scala
+++ b/src/swing/scala/swing/event/ListEvent.scala
@@ -28,7 +28,7 @@ object ListChanged {
 class ListChanged[A](override val source: ListView[A]) extends ListChange(source)
 
 object ListElementsAdded {
-  def unapply[A](e: ListElementsAdded[A]) = Some(e.source, e.range)
+  def unapply[A](e: ListElementsAdded[A]) = Some((e.source, e.range))
   def apply[A](source: ListView[A], range: Range) = new ListElementsAdded(source, range)
 }
 
@@ -36,7 +36,7 @@ class ListElementsAdded[A](override val source: ListView[A], val range: Range)
            extends ListChange(source)
 
 object ListElementsRemoved {
-  def unapply[A](e: ListElementsRemoved[A]) = Some(e.source, e.range)
+  def unapply[A](e: ListElementsRemoved[A]) = Some((e.source, e.range))
   def apply[A](source: ListView[A], range: Range) = new ListElementsRemoved(source, range)
 }
 class ListElementsRemoved[A](override val source: ListView[A], val range: Range)