Integrated mouse support

git-svn-id: svn://svn.code.sf.net/p/nuttx/code/trunk@1384 42af7a65-404d-4744-a932-0658087f49c3

3 files changed, 45 insertions, 21 deletions

diff --git a/nuttx/graphics/nxsu/nx_mousein.c b/nuttx/graphics/nxsu/nx_mousein.c
index 31c9483d2..1c5427138 100644
--- a/nuttx/graphics/nxsu/nx_mousein.c
+++ b/nuttx/graphics/nxsu/nx_mousein.c
@@ -43,7 +43,9 @@
 #include <errno.h>
 #include <debug.h>
 
+#include <nuttx/nxglib.h>
 #include <nuttx/nx.h>
+
 #include "nxfe.h"
 
 #ifdef CONFIG_NX_MOUSE
@@ -61,8 +63,8 @@
  ****************************************************************************/
 
 static struct nxgl_point_s g_mpos;
-static struct nxgl_rect_s g_mrange;
-static struct g_mbutton;
+static struct nxgl_point_s g_mrange;
+static ubyte               g_mbutton;
 
 /****************************************************************************
  * Public Data
@@ -99,26 +101,37 @@ void nxsu_mouseinit(int x, int y)
  * Description:
  *   Report mouse position info to the specified window
  *
+ * Input Parameters:
+ *   wnd - The window to receive the mouse report
+ *
+ * Returned Value:
+ *   0: Mouse report sent; >0: Mouse report not sent; <0: An error occurred
+ *
  ****************************************************************************/
 
-void nxsu_mousereport(struct nxbe_window_s *wnd)
+int nxsu_mousereport(struct nxbe_window_s *wnd)
 {
   struct nxgl_point_s relpos;
 
   /* Does this window support mouse callbacks? */
 
-  if (win->cb->mousein)
+  if (wnd->cb->mousein)
     {
-      /* Yes.. Does the mount position lie within the window? */
+      /* Yes.. Is the mouse position visible in this window? */
 
-      if (nxgl_rectinside(wnd->bounds, g_mpos))
+      if (nxbe_visible(wnd, &g_mpos))
         {
           /* Yes... Convert the mouse position to window relative coordinates */
 
-          nxgl_vectsubtract(&relpos, g_mpos, wnd->origin);
-          win->cb->mousein((NXWINDOW)wnd, &relpos, g_mbutton, wnd->arg);
+          nxgl_vectsubtract(&relpos, &g_mpos, &wnd->origin);
+          wnd->cb->mousein((NXWINDOW)wnd, &relpos, g_mbutton, wnd->arg);
+          return OK;
         }
     }
+
+  /* No error occurred, but the mouse report was not sent */
+
+  return 1;
 }
 
 /****************************************************************************
@@ -135,9 +148,7 @@ int nx_mousein(NXHANDLE handle, nxgl_coord_t x, nxgl_coord_t y, ubyte buttons)
 {
   FAR struct nxfe_state_s *fe = (FAR struct nxfe_state_s *)handle;
   struct nxbe_window_s *wnd;
-
-  x_coord_t x = pos->x;
-  x_coord_t y = pos->y;
+  int ret;
 
   /* Clip x and y to within the bounding rectangle */
 
@@ -145,35 +156,42 @@ int nx_mousein(NXHANDLE handle, nxgl_coord_t x, nxgl_coord_t y, ubyte buttons)
     {
       x = 0;
     }
-  else if (x >= g_mbound.x)
+  else if (x >= g_mrange.x)
     {
-      x = g_mbound.x - 1;
+      x = g_mrange.x - 1;
     }
 
   if (y < 0)
     {
       y = 0;
     }
-  else if (y >= g_mbound.y)
+  else if (y >= g_mrange.y)
     {
-      y = g_mbound.y - 1;
+      y = g_mrange.y - 1;
     }
 
  /* Look any change in values */
 
-  if (x != g_mpos.x || y != g_mpos.y || button != g_mbutton)
+  if (x != g_mpos.x || y != g_mpos.y || buttons != g_mbutton)
     {
       /* Update the mouse value */
 
       g_mpos.x  = x;
       g_mpos.y  = y;
-      b_mbutton = button;
+      g_mbutton = buttons;
 
-      /* Pick the window to receive the mouse event */
+      /* Pick the window to receive the mouse event.  Start with
+       * the top window and go down.  Step with the first window
+       * that gets the mouse report
+       */
 
       for (wnd = fe->be.topwnd; wnd; wnd = wnd->below)
         {
-          nxsu_mousereport(wnd);
+          ret = nxsu_mousereport(wnd);
+          if (ret == 0)
+            {
+              break;
+            }
         }
     }
 }
diff --git a/nuttx/graphics/nxsu/nx_requestbkgd.c b/nuttx/graphics/nxsu/nx_requestbkgd.c
index 782679464..32eed1ab1 100644
--- a/nuttx/graphics/nxsu/nx_requestbkgd.c
+++ b/nuttx/graphics/nxsu/nx_requestbkgd.c
@@ -137,7 +137,7 @@ int nx_requestbkgd(NXHANDLE handle, FAR const struct nx_callback_s *cb)
   /* Provide the mouse settings to the client */
 
 #ifdef CONFIG_NX_MOUSE
-  nxsu_mousereport(be->bkgd);
+  nxsu_mousereport(&be->bkgd);
 #endif
 
   /* In this single-user mode, we could return the background window
diff --git a/nuttx/graphics/nxsu/nxfe.h b/nuttx/graphics/nxsu/nxfe.h
index 7fd65f2f3..a95ae9c60 100644
--- a/nuttx/graphics/nxsu/nxfe.h
+++ b/nuttx/graphics/nxsu/nxfe.h
@@ -166,10 +166,16 @@ EXTERN void nxsu_mouseinit(int x, int y);
  * Description:
  *   Report mouse position info to the specified window
  *
+ * Input Parameters:
+ *   wnd - The window to receive the mouse report
+ *
+ * Returned Value:
+ *   0: Mouse report sent; >0: Mouse report not sent; <0: An error occurred
+ *
  ****************************************************************************/
 
 #ifdef CONFIG_NX_MOUSE
-EXTERN void nxsu_mousereport(struct nxbe_window_s *wnd);
+EXTERN int nxsu_mousereport(struct nxbe_window_s *wnd);
 #endif
 
 #undef EXTERN