[fireball] 50419: [WINENT.DRV] - Remove unnecessary events code, it has no use for us.

Author: fireball
Date: Tue Jan 18 12:01:58 2011
New Revision: 50419

URL: http://svn.reactos.org/svn/reactos?rev=50419&view=rev
Log:
[WINENT.DRV]
- Remove unnecessary events code, it has no use for us.

Modified:
    branches/arwinss/reactos/dll/win32/winent.drv/event.c
    branches/arwinss/reactos/dll/win32/winent.drv/window.c

Modified: branches/arwinss/reactos/dll/win32/winent.drv/event.c
URL:
http://svn.reactos.org/svn/reactos/branches/arwinss/reactos/dll/win32/winen…
==============================================================================
--- branches/arwinss/reactos/dll/win32/winent.drv/event.c [iso-8859-1] (original)
+++ branches/arwinss/reactos/dll/win32/winent.drv/event.c [iso-8859-1] Tue Jan 18 12:01:58
2011
@@ -4,7 +4,6 @@
  * FILE:            dll/win32/winent.drv/event.c
  * PURPOSE:         Event handling routines
  * PROGRAMMERS:     Aleksey Bragin (aleksey(a)reactos.org)
- *                  Some parts inspired by Wine's winex11.drv
  */
 
 /* INCLUDES ***************************************************************/
@@ -13,198 +12,8 @@
 #include "wine/debug.h"
 
 WINE_DEFAULT_DEBUG_CHANNEL(event);
-#if 0
+
 /* GLOBALS ****************************************************************/
-
-static void NTDRV_Expose( HWND hwnd, GR_EVENT *event );
-
-struct event_handler
-{
-    GR_EVENT_TYPE       type;    /* event type */
-    ntdrv_event_handler handler; /* corresponding handler function */
-};
-
-#define MAX_EVENT_HANDLERS 64
-
-static struct event_handler handlers[MAX_EVENT_HANDLERS] =
-{
-    /* list must be sorted by event type */
-    //{ KeyPress,         X11DRV_KeyEvent },
-    //{ KeyRelease,       X11DRV_KeyEvent },
-    //{ ButtonPress,      X11DRV_ButtonPress },
-    //{ ButtonRelease,    X11DRV_ButtonRelease },
-    //{ MotionNotify,     X11DRV_MotionNotify },
-    //{ EnterNotify,      X11DRV_EnterNotify },
-    /* LeaveNotify */
-    //{ FocusIn,          X11DRV_FocusIn },
-    //{ FocusOut,         X11DRV_FocusOut },
-    //{ KeymapNotify,     X11DRV_KeymapNotify },
-    { GR_EVENT_TYPE_EXPOSURE, NTDRV_Expose },
-    /* GraphicsExpose */
-    /* NoExpose */
-    /* VisibilityNotify */
-    /* CreateNotify */
-    //{ DestroyNotify,    X11DRV_DestroyNotify },
-    /* UnmapNotify */
-    //{ MapNotify,        X11DRV_MapNotify },
-    /* MapRequest */
-    //{ ReparentNotify,   X11DRV_ReparentNotify },
-    //{ ConfigureNotify,  X11DRV_ConfigureNotify },
-    /* ConfigureRequest */
-    //{ GravityNotify,    X11DRV_GravityNotify },
-    /* ResizeRequest */
-    /* CirculateNotify */
-    /* CirculateRequest */
-    //{ PropertyNotify,   X11DRV_PropertyNotify },
-    //{ SelectionClear,   X11DRV_SelectionClear },
-    //{ SelectionRequest, X11DRV_SelectionRequest },
-    /* SelectionNotify */
-    /* ColormapNotify */
-    //{ ClientMessage,    X11DRV_ClientMessage },
-    //{ MappingNotify,    X11DRV_MappingNotify },
-};
-
-static int nb_event_handlers = 1;  /* change this if you add handlers above */
-
-/* FUNCTIONS **************************************************************/
-
-/***********************************************************************
- *           find_handler
- *
- * Find the handler for a given event type. Caller must hold the x11 lock.
- */
-static inline ntdrv_event_handler find_handler( GR_EVENT_TYPE type )
-{
-    int min = 0, max = nb_event_handlers - 1;
-
-    while (min <= max)
-    {
-        int pos = (min + max) / 2;
-        if (handlers[pos].type == type) return handlers[pos].handler;
-        if (handlers[pos].type > type) max = pos - 1;
-        else min = pos + 1;
-    }
-    return NULL;
-}
-
-enum event_merge_action
-{
-    MERGE_DISCARD,  /* discard the old event */
-    MERGE_HANDLE,   /* handle the old event */
-    MERGE_KEEP      /* keep the old event for future merging */
-};
-
-/***********************************************************************
- *           merge_events
- *
- * Try to merge 2 consecutive events.
- */
-#if 0
-static enum event_merge_action merge_events( GR_EVENT *prev, GR_EVENT *next )
-{
-    switch (prev->type)
-    {
-    case GR_UPDATE_MOVE:
-        switch (next->type)
-        {
-        case GR_UPDATE_MOVE:
-            if (prev->wid == next->wid)
-            {
-                TRACE( "discarding duplicate ConfigureNotify for window %lx\n",
prev->wid );
-                return MERGE_DISCARD;
-            }
-            break;
-        case GR_EVENT_MASK_EXPOSURE:
-        //case PropertyNotify:
-            return MERGE_KEEP;
-        }
-        break;
-    case GR_EVENT_MASK_MOUSE_MOTION:
-        if (prev->wid == next->wid && next->type == MotionNotify)
-        {
-            TRACE( "discarding duplicate MotionNotify for window %lx\n",
prev->wid );
-            return MERGE_DISCARD;
-        }xch
-        break;
-    }
-
-    return MERGE_HANDLE;
-}
-#endif
-
-
-/***********************************************************************
- *           call_event_handler
- */
-static inline void call_event_handler( GR_EVENT *event )
-{
-    HWND hwnd;
-    ntdrv_event_handler handler;
-    //GR_EVENT *prev;
-    //struct ntdrv_thread_data *thread_data;
-
-    if (!(handler = find_handler( event->type )))
-    {
-        TRACE( "%s for win %lx, ignoring\n", "event", -1 );
-        return;  /* no handler, ignore it */
-    }
-
-    hwnd = GrGetHwnd( event->general.wid );
-    //if (!hwnd && event->xany.window == root_window) hwnd =
GetDesktopWindow();
-
-    //TRACE( "%lu %s for hwnd/window %p/%lx\n",
-    //       event->xany.serial, dbgstr_event( event->type ), hwnd,
event->xany.window );
-    //wine_tsx11_unlock();
-    //thread_data = ntdrv_thread_data();
-    //prev = thread_data->current_event;
-    //thread_data->current_event = event;
-    handler( hwnd, event );
-    //thread_data->current_event = prev;
-    //wine_tsx11_lock();
-}
-
-/***********************************************************************
- *           process_events
- */
-static int process_events( int mask )
-{
-    GR_EVENT event;//, prev_event;
-    int count = 0;
-    //enum event_merge_action action = MERGE_DISCARD;
-
-    //prev_event.type = 0;
-    //wine_tsx11_lock();
-    while (GrPeekEvent(&event))
-    {
-        GrGetNextEvent(&event);
-        ERR("Got event type %d\n", event.type);
-        count++;
-#if 0
-        if (prev_event.type) action = merge_events( &prev_event, &event );
-        switch( action )
-        {
-        case MERGE_DISCARD:  /* discard prev, keep new */
-            prev_event = event;
-            break;
-        case MERGE_HANDLE:  /* handle prev, keep new */
-            call_event_handler( display, &prev_event );
-            prev_event = event;
-            break;
-        case MERGE_KEEP:  /* handle new, keep prev for future merging */
-            call_event_handler( display, &event );
-            break;
-        }
-#else
-        call_event_handler( &event );
-#endif
-    }
-    //if (prev_event.type) call_event_handler( &prev_event );
-    //XFlush( gdi_display );
-    //wine_tsx11_unlock();
-    if (count) TRACE( "processed %d events\n", count );
-    return count;
-}
-#endif
 
 /***********************************************************************
  *           MsgWaitForMultipleObjectsEx   (NTDRV.@)
@@ -212,61 +21,9 @@
 DWORD CDECL RosDrv_MsgWaitForMultipleObjectsEx( DWORD count, const HANDLE *handles, DWORD
timeout,
                                                 DWORD mask, DWORD flags )
 {
-    //TRACE("WaitForMultipleObjectsEx(%d %p %d %x %x %x\n", count, handles,
timeout, mask, flags);
+    TRACE("WaitForMultipleObjectsEx(%d %p %d %x %x %x\n", count, handles,
timeout, mask, flags);
 
     if (!count && !timeout) return WAIT_TIMEOUT;
     return WaitForMultipleObjectsEx( count, handles, flags & MWMO_WAITALL,
                                      timeout, flags & MWMO_ALERTABLE );
 }
-
-#if 0
-/***********************************************************************
- *           X11DRV_Expose
- */
-static void NTDRV_Expose( HWND hwnd, GR_EVENT *xev )
-{
-    GR_EVENT_EXPOSURE *event = &xev->exposure;
-    RECT rect;
-    struct ntdrv_win_data *data;
-    int flags = RDW_INVALIDATE | RDW_ERASE;
-
-    ERR( "win %p (%lx) %d,%d %dx%d\n",
-           hwnd, event->wid, event->x, event->y, event->width,
event->height );
-
-    if (!(data = NTDRV_get_win_data( hwnd ))) return;
-
-    rect.left   = event->x;
-    rect.top    = event->y;
-    rect.right  = event->x + event->width;
-    rect.bottom = event->y + event->height;
-    if (event->wid == data->whole_window)
-    {
-        OffsetRect( &rect, data->whole_rect.left - data->client_rect.left,
-                    data->whole_rect.top - data->client_rect.top );
-        flags |= RDW_FRAME;
-    }
-
-    //if (event->wid != root_window)
-    {
-        if (GetWindowLongW( data->hwnd, GWL_EXSTYLE ) & WS_EX_LAYOUTRTL)
-            mirror_rect( &data->client_rect, &rect );
-
-        SERVER_START_REQ( update_window_zorder )
-        {
-            req->window      = wine_server_user_handle( hwnd );
-            req->rect.left   = rect.left;
-            req->rect.top    = rect.top;
-            req->rect.right  = rect.right;
-            req->rect.bottom = rect.bottom;
-            wine_server_call( req );
-        }
-        SERVER_END_REQ;
-
-        flags |= RDW_ALLCHILDREN;
-    }
-    //else OffsetRect( &rect, virtual_screen_rect.left, virtual_screen_rect.top );
-
-    RedrawWindow( hwnd, &rect, 0, flags );
-}
-#endif
-/* EOF */

Modified: branches/arwinss/reactos/dll/win32/winent.drv/window.c
URL:
http://svn.reactos.org/svn/reactos/branches/arwinss/reactos/dll/win32/winen…
==============================================================================
--- branches/arwinss/reactos/dll/win32/winent.drv/window.c [iso-8859-1] (original)
+++ branches/arwinss/reactos/dll/win32/winent.drv/window.c [iso-8859-1] Tue Jan 18
12:01:58 2011
@@ -140,7 +140,6 @@
         //if (win && win != root_window) NTDRV_init_desktop( win, width, height
);
 
         // create a desktop window
-        //GrSelectEvents(GR_ROOT_WINDOW_ID, GR_EVENT_MASK_EXPOSURE);
     }
 
     return TRUE;
@@ -230,6 +229,7 @@
     data->whole_window = data->client_window = root_window;
     SetPropA( data->hwnd, whole_window_prop, (HANDLE)root_window );
     SetPropA( data->hwnd, client_window_prop, (HANDLE)root_window );
+    //SetPropA( data->hwnd, "__wine_x11_managed", (HANDLE)1 );
 
     return data;
 }
@@ -559,8 +559,6 @@
     data->whole_window = SwmNewWindow( root_window,
                                        &data->whole_rect, data->hwnd, ex_style
);
 
-    //GrSelectEvents(data->whole_window, GR_EVENT_MASK_EXPOSURE);
-
     if (!data->whole_window) goto done;
 
     if (!create_client_window( data ))
@@ -574,6 +572,7 @@
     //set_wm_hints( display, data );
 
     SetPropA( data->hwnd, whole_window_prop, (HANDLE)data->whole_window );
+    //SetPropA( data->hwnd, "__wine_x11_managed", (HANDLE)1 );
 
     /* set the window text */
     //if (!InternalGetWindowText( data->hwnd, text, sizeof(text)/sizeof(WCHAR) ))
text[0] = 0;