[kaffe] CVS kaffe (riccardo): removed an empty method which should not be used anyway

Kaffe CVS cvs-commits at kaffe.org
Mon Apr 24 04:32:59 PDT 2006


PatchSet 7270 
Date: 2006/04/24 11:23:00
Author: riccardo
Branch: HEAD
Tag: (none) 
Log:
removed an empty method which should not be used anyway

Members: 
	ChangeLog:1.4774->1.4775 
	libraries/javalib/awt-implementations/kaffe/java/awt/MouseEvt.java:1.2->1.3 

Index: kaffe/ChangeLog
diff -u kaffe/ChangeLog:1.4774 kaffe/ChangeLog:1.4775
--- kaffe/ChangeLog:1.4774	Sun Apr 23 22:11:01 2006
+++ kaffe/ChangeLog	Mon Apr 24 11:23:00 2006
@@ -1,3 +1,8 @@
+2006-04-24  Riccardo Mottola <riccardo at kaffe.org>
+
+	* libraries/javalib/awt-implementations/kaffe/java/awt/MouseEvt.java:
+	removed an empty method which should not be used anyway
+
 2006-04-24  Dalibor Topic  <robilad at kaffe.org>
 
 	* kaffe/kaffevm/exception.c (dispatchException): Added missing
Index: kaffe/libraries/javalib/awt-implementations/kaffe/java/awt/MouseEvt.java
diff -u kaffe/libraries/javalib/awt-implementations/kaffe/java/awt/MouseEvt.java:1.2 kaffe/libraries/javalib/awt-implementations/kaffe/java/awt/MouseEvt.java:1.3
--- kaffe/libraries/javalib/awt-implementations/kaffe/java/awt/MouseEvt.java:1.2	Wed Apr 19 09:49:29 2006
+++ kaffe/libraries/javalib/awt-implementations/kaffe/java/awt/MouseEvt.java	Mon Apr 24 11:23:06 2006
@@ -47,24 +47,6 @@
 	super( src, evtId, time, modifiers, x, y, clickCount, isPopupTrigger);
 }
 
-static void clickToFocus ( Component newKeyTgt ) {			
-	// The JDK does not automatically set the focus on mouse clicks for lightweights,
-	// (non-lightweights probably will be handled by the native window system), i.e.
-	// if a component explicitly requests the focus in respond to a mouse click, the
-	// focus events will be processed AFTER the mouse event. This is the opposite order
-	// compared to native handling (for toplevels). We try to be compatible with
-	// lightweight behavior
-
-	// note that the JDK allows components which are not isFocusTraversable() to gain
-	// the focus by means of explicit requestFocus() (not very intuitive) OR by means of
-	// mouseclicks (even on components which are not mouse aware, which sounds silly)
-
-/* XXX
-	if ( ((newKeyTgt.flags & Component.IS_NATIVE_LIKE) != 0) && newKeyTgt.isFocusTraversable() )
-		newKeyTgt.requestFocus();	
- */
-}
-
 static Component computeMouseTarget ( Container toplevel, int x, int y ) {
 	Container  cntr;
 	Component  c, tgt;
@@ -185,9 +167,6 @@
 		  break;
 		
 		case MOUSE_PRESSED:
-			if ( (AWTEvent.mouseTgt != AWTEvent.keyTgt) )  // decide upon focus policy
-				clickToFocus( AWTEvent.mouseTgt);
-			
 			buttonPressed = true;
 			modifiers = updateInputModifier( button, true);
 			




More information about the kaffe mailing list