Enlightenment CVS committal

Author  : pfritz
Project : e17
Module  : libs/ewl

Dir     : e17/libs/ewl/src/engines/xcb


Modified Files:
        ewl_engine_xcb.c 


Log Message:
- remove the first argument of the parameter and type check macros, althoug
  they had a different intention they were only used as the parameter name
- don't fail on type checks if the pointer is NULL

===================================================================
RCS file: /cvs/e/e17/libs/ewl/src/engines/xcb/ewl_engine_xcb.c,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -3 -r1.1 -r1.2
--- ewl_engine_xcb.c    7 Nov 2007 18:01:30 -0000       1.1
+++ ewl_engine_xcb.c    12 Nov 2007 22:42:21 -0000      1.2
@@ -172,7 +172,7 @@
        int i;
 
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR_RET("engine", engine, FALSE);
+       DCHECK_PARAM_PTR_RET(engine, FALSE);
 
        /* If the event handlers are already setup don't do it again */
        if (ee_expose_handler)
@@ -303,7 +303,7 @@
 ee_shutdown(Ewl_Engine *engine)
 {
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("engine", engine);
+       DCHECK_PARAM_PTR(engine);
 
        if (ee_expose_handler)
                ecore_event_handler_del(ee_expose_handler);
@@ -390,8 +390,8 @@
        Ecore_X_Window xwin;
 
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("win", win);
-       DCHECK_TYPE("win", win, EWL_WINDOW_TYPE);
+       DCHECK_PARAM_PTR(win);
+       DCHECK_TYPE(win, EWL_WINDOW_TYPE);
 
        if (win->flags & EWL_WINDOW_OVERRIDE)
                xwin = ecore_x_window_override_new(0,
@@ -419,8 +419,8 @@
 ee_window_destroy(Ewl_Window *win)
 {
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("win", win);
-       DCHECK_TYPE("win", win, EWL_WINDOW_TYPE);
+       DCHECK_PARAM_PTR(win);
+       DCHECK_TYPE(win, EWL_WINDOW_TYPE);
 
        ee_window_hide(win);
        ecore_x_window_del((Ecore_X_Window)(EWL_EMBED(win)->canvas_window));
@@ -436,8 +436,8 @@
 ee_window_move(Ewl_Window *win)
 {
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("win", win);
-       DCHECK_TYPE("win", win, EWL_WINDOW_TYPE);
+       DCHECK_PARAM_PTR(win);
+       DCHECK_TYPE(win, EWL_WINDOW_TYPE);
 
        ecore_x_window_move((Ecore_X_Window)win->window,
                                        EWL_EMBED(win)->x,
@@ -452,8 +452,8 @@
        int width, height;
 
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("win", win);
-       DCHECK_TYPE("win", win, EWL_WINDOW_TYPE);
+       DCHECK_PARAM_PTR(win);
+       DCHECK_TYPE(win, EWL_WINDOW_TYPE);
 
        width = ewl_object_current_w_get(EWL_OBJECT(win));
        height = ewl_object_current_h_get(EWL_OBJECT(win));
@@ -471,8 +471,8 @@
 ee_window_min_max_size_set(Ewl_Window *win)
 {
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("win", win);
-       DCHECK_TYPE("win", win, EWL_WINDOW_TYPE);
+       DCHECK_PARAM_PTR(win);
+       DCHECK_TYPE(win, EWL_WINDOW_TYPE);
 
        ecore_x_icccm_size_pos_hints_get_prefetch((Ecore_X_Window)win->window);
        ecore_x_icccm_size_pos_hints_get_fetch();
@@ -493,8 +493,8 @@
 ee_window_show(Ewl_Window *win)
 {
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("win", win);
-       DCHECK_TYPE("win", win, EWL_WINDOW_TYPE);
+       DCHECK_PARAM_PTR(win);
+       DCHECK_TYPE(win, EWL_WINDOW_TYPE);
 
        ecore_x_window_show((Ecore_X_Window)win->window);
        ecore_x_window_show((Ecore_X_Window)EWL_EMBED(win)->canvas_window);
@@ -506,8 +506,8 @@
 ee_window_hide(Ewl_Window *win)
 {
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("win", win);
-       DCHECK_TYPE("win", win, EWL_WINDOW_TYPE);
+       DCHECK_PARAM_PTR(win);
+       DCHECK_TYPE(win, EWL_WINDOW_TYPE);
 
        ecore_x_window_hide((Ecore_X_Window)EWL_EMBED(win)->canvas_window);
        ecore_x_window_hide((Ecore_X_Window)EWL_WINDOW(win)->window);
@@ -519,8 +519,8 @@
 ee_window_title_set(Ewl_Window *win)
 {
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("win", win);
-       DCHECK_TYPE("win", win, EWL_WINDOW_TYPE);
+       DCHECK_PARAM_PTR(win);
+       DCHECK_TYPE(win, EWL_WINDOW_TYPE);
 
        ecore_x_icccm_title_set((Ecore_X_Window)win->window, win->title);
        ecore_x_netwm_name_set((Ecore_X_Window)win->window, win->title);
@@ -532,8 +532,8 @@
 ee_window_name_class_set(Ewl_Window *win)
 {
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("win", win);
-       DCHECK_TYPE("win", win, EWL_WINDOW_TYPE);
+       DCHECK_PARAM_PTR(win);
+       DCHECK_TYPE(win, EWL_WINDOW_TYPE);
 
        ecore_x_icccm_name_class_set((Ecore_X_Window)win->window, win->name,
                                (win->classname ? win->classname : win->name));
@@ -545,8 +545,8 @@
 ee_window_borderless_set(Ewl_Window *win)
 {
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("win", win);
-       DCHECK_TYPE("win", win, EWL_WINDOW_TYPE);
+       DCHECK_PARAM_PTR(win);
+       DCHECK_TYPE(win, EWL_WINDOW_TYPE);
 
        ecore_x_mwm_borderless_set((Ecore_X_Window)win->window,
                                (!!(win->flags & EWL_WINDOW_BORDERLESS)));
@@ -558,8 +558,8 @@
 ee_window_dialog_set(Ewl_Window *win)
 {
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("win", win);
-       DCHECK_TYPE("win", win, EWL_WINDOW_TYPE);
+       DCHECK_PARAM_PTR(win);
+       DCHECK_TYPE(win, EWL_WINDOW_TYPE);
 
        ecore_x_netwm_window_type_set((Ecore_X_Window)win->window,
                        ((!!(win->flags & EWL_WINDOW_DIALOG)) ?
@@ -575,8 +575,8 @@
        int diff;
 
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("win", win);
-       DCHECK_TYPE("win", win, EWL_WINDOW_TYPE);
+       DCHECK_PARAM_PTR(win);
+       DCHECK_TYPE(win, EWL_WINDOW_TYPE);
 
        diff = win->flags ^ win->old_flags;
        if (!diff) DRETURN(DLEVEL_STABLE);
@@ -599,8 +599,8 @@
        int state = 0;
 
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("win", win);
-       DCHECK_TYPE("win", win, EWL_WINDOW_TYPE);
+       DCHECK_PARAM_PTR(win);
+       DCHECK_TYPE(win, EWL_WINDOW_TYPE);
 
        if (states & ewl_flag)
        {
@@ -616,8 +616,8 @@
 ee_window_state_request(Ewl_Window *win, int states)
 {
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("win", win);
-       DCHECK_TYPE("win", win, EWL_WINDOW_TYPE);
+       DCHECK_PARAM_PTR(win);
+       DCHECK_TYPE(win, EWL_WINDOW_TYPE);
 
        ee_window_state_handle(win, states, EWL_WINDOW_FULLSCREEN,
                                        ECORE_X_WINDOW_STATE_FULLSCREEN);
@@ -642,8 +642,8 @@
        int count = 0;
 
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("win", win);
-       DCHECK_TYPE("win", win, EWL_WINDOW_TYPE);
+       DCHECK_PARAM_PTR(win);
+       DCHECK_TYPE(win, EWL_WINDOW_TYPE);
 
        /* Note: We're ignoring the attention demand state in here as it is
         * only relevant if the window has been realized */
@@ -675,8 +675,8 @@
        int urgent = FALSE;
 
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("win", win);
-       DCHECK_TYPE("win", win, EWL_WINDOW_TYPE);
+       DCHECK_PARAM_PTR(win);
+       DCHECK_TYPE(win, EWL_WINDOW_TYPE);
 
        if (win->flags & EWL_WINDOW_LEADER)
                win_group = win->leader.ewl->window;
@@ -704,8 +704,8 @@
 ee_window_transient_for(Ewl_Window *win)
 {
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("win", win);
-       DCHECK_TYPE("win", win, EWL_WINDOW_TYPE);
+       DCHECK_PARAM_PTR(win);
+       DCHECK_TYPE(win, EWL_WINDOW_TYPE);
 
        if (win->flags & EWL_WINDOW_TRANSIENT)
                ecore_x_icccm_transient_for_set((Ecore_X_Window)win->window,
@@ -725,8 +725,8 @@
        Ewl_Embed_Window *leader;
 
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("win", win);
-       DCHECK_TYPE("win", win, EWL_WINDOW_TYPE);
+       DCHECK_PARAM_PTR(win);
+       DCHECK_TYPE(win, EWL_WINDOW_TYPE);
 
        if (win->flags & EWL_WINDOW_LEADER)
                leader = win->leader.ewl->window;
@@ -747,8 +747,8 @@
 ee_window_raise(Ewl_Window *win)
 {
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("win", win);
-       DCHECK_TYPE("win", win, EWL_WINDOW_TYPE);
+       DCHECK_PARAM_PTR(win);
+       DCHECK_TYPE(win, EWL_WINDOW_TYPE);
 
        ecore_x_window_raise((Ecore_X_Window)win->window);
 
@@ -759,8 +759,8 @@
 ee_window_lower(Ewl_Window *win)
 {
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("win", win);
-       DCHECK_TYPE("win", win, EWL_WINDOW_TYPE);
+       DCHECK_PARAM_PTR(win);
+       DCHECK_TYPE(win, EWL_WINDOW_TYPE);
 
        ecore_x_window_lower((Ecore_X_Window)win->window);
 
@@ -773,8 +773,8 @@
        int ret = FALSE;
 
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR_RET("win", win, FALSE);
-       DCHECK_TYPE_RET("win", win, EWL_WINDOW_TYPE, FALSE);
+       DCHECK_PARAM_PTR_RET(win, FALSE);
+       DCHECK_TYPE_RET(win, EWL_WINDOW_TYPE, FALSE);
 
        if ((!!(win->flags & EWL_WINDOW_GRAB_KEYBOARD)))
        {
@@ -794,8 +794,8 @@
 ee_keyboard_ungrab(Ewl_Window *win)
 {
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("win", win);
-       DCHECK_TYPE("win", win, EWL_WINDOW_TYPE);
+       DCHECK_PARAM_PTR(win);
+       DCHECK_TYPE(win, EWL_WINDOW_TYPE);
 
        ecore_x_keyboard_ungrab();
        ee_current_key_grab_window = NULL;
@@ -809,8 +809,8 @@
        int ret = FALSE;
 
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR_RET("win", win, FALSE);
-       DCHECK_TYPE_RET("win", win, EWL_WINDOW_TYPE, FALSE);
+       DCHECK_PARAM_PTR_RET(win, FALSE);
+       DCHECK_TYPE_RET(win, EWL_WINDOW_TYPE, FALSE);
 
        if ((!!(win->flags & EWL_WINDOW_GRAB_POINTER)))
        {
@@ -830,8 +830,8 @@
 ee_pointer_ungrab(Ewl_Window *win)
 {
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("win", win);
-       DCHECK_TYPE("win", win, EWL_WINDOW_TYPE);
+       DCHECK_PARAM_PTR(win);
+       DCHECK_TYPE(win, EWL_WINDOW_TYPE);
 
        ecore_x_pointer_ungrab();
        ee_current_pointer_grab_window = NULL;
@@ -843,9 +843,9 @@
 ee_window_selection_text_set(Ewl_Window *win, const char *txt)
 {
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("win", win);
-       DCHECK_PARAM_PTR("txt", txt);
-       DCHECK_TYPE("win", win, EWL_WINDOW_TYPE);
+       DCHECK_PARAM_PTR(win);
+       DCHECK_PARAM_PTR(txt);
+       DCHECK_TYPE(win, EWL_WINDOW_TYPE);
 
        ecore_x_selection_primary_prefetch();
        ecore_x_selection_primary_fetch();
@@ -876,8 +876,8 @@
 ee_dnd_aware_set(Ewl_Embed *embed)
 {
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("embed", embed);
-       DCHECK_TYPE("embed", embed, EWL_EMBED_TYPE);
+       DCHECK_PARAM_PTR(embed);
+       DCHECK_TYPE(embed, EWL_EMBED_TYPE);
 
        ecore_x_dnd_aware_set((Ecore_X_Window)embed->canvas_window, TRUE);
 
@@ -888,8 +888,8 @@
 ee_desktop_size_get(Ewl_Embed *embed, int *w, int *h)
 {
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("embed", embed);
-       DCHECK_TYPE("embed", embed, EWL_EMBED_TYPE);
+       DCHECK_PARAM_PTR(embed);
+       DCHECK_TYPE(embed, EWL_EMBED_TYPE);
 
        ecore_x_drawable_geometry_get_prefetch((Ecore_X_Window)0);
        ecore_x_drawable_geometry_get_fetch();
@@ -902,8 +902,8 @@
 ee_dnd_drag_types_set(Ewl_Embed *embed, const char **types, unsigned int num)
 {
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("embed", embed);
-       DCHECK_TYPE("embed", embed, EWL_EMBED_TYPE);
+       DCHECK_PARAM_PTR(embed);
+       DCHECK_TYPE(embed, EWL_EMBED_TYPE);
 
        ecore_x_dnd_type_get_prefetch((Ecore_X_Window)embed->canvas_window);
        ecore_x_dnd_aware_set((Ecore_X_Window)embed->canvas_window,
@@ -918,8 +918,8 @@
 ee_dnd_drag_begin(Ewl_Embed *embed)
 {
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("embed", embed);
-       DCHECK_TYPE("embed", embed, EWL_EMBED_TYPE);
+       DCHECK_PARAM_PTR(embed);
+       DCHECK_TYPE(embed, EWL_EMBED_TYPE);
 
        ecore_x_dnd_begin_prefetch((Ecore_X_Window)embed->canvas_window);
        ecore_x_dnd_begin_fetch();
@@ -932,8 +932,8 @@
 ee_dnd_drag_drop(Ewl_Embed *embed)
 {
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("embed", embed);
-       DCHECK_TYPE("embed", embed, EWL_EMBED_TYPE);
+       DCHECK_PARAM_PTR(embed);
+       DCHECK_TYPE(embed, EWL_EMBED_TYPE);
 
        ecore_x_dnd_drop();
 
@@ -946,9 +946,9 @@
        Ecore_X_Event_Selection_Request *request = handle;
 
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR_RET("embed", embed, FALSE);
-       DCHECK_PARAM_PTR_RET("handle", handle, FALSE);
-       DCHECK_TYPE_RET("embed", embed, EWL_EMBED_TYPE, FALSE);
+       DCHECK_PARAM_PTR_RET(embed, FALSE);
+       DCHECK_PARAM_PTR_RET(handle, FALSE);
+       DCHECK_TYPE_RET(embed, EWL_EMBED_TYPE, FALSE);
 
        ecore_x_window_prop_property_set(request->requestor, request->property,
                                         request->target, 8, data, len);
@@ -962,8 +962,8 @@
 ee_pointer_data_new(Ewl_Embed *embed, int *data, int w, int h)
 {
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR_RET("embed", embed, 0);
-       DCHECK_TYPE_RET("embed", embed, EWL_EMBED_TYPE, 0);
+       DCHECK_PARAM_PTR_RET(embed, 0);
+       DCHECK_TYPE_RET(embed, EWL_EMBED_TYPE, 0);
 
        DRETURN_INT(ecore_x_cursor_new((Ecore_X_Window)embed->canvas_window,
                                data, w, h, 0, 0), DLEVEL_STABLE);
@@ -973,8 +973,8 @@
 ee_pointer_free(Ewl_Embed *embed, int pointer)
 {
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("embed", embed);
-       DCHECK_TYPE("embed", embed, EWL_EMBED_TYPE);
+       DCHECK_PARAM_PTR(embed);
+       DCHECK_TYPE(embed, EWL_EMBED_TYPE);
 
        ecore_x_cursor_free(pointer);
 
@@ -989,8 +989,8 @@
 ee_pointer_get(Ewl_Embed *embed)
 {
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR_RET("embed", embed, 0);
-       DCHECK_TYPE_RET("embed", embed, EWL_EMBED_TYPE, 0);
+       DCHECK_PARAM_PTR_RET(embed, 0);
+       DCHECK_TYPE_RET(embed, EWL_EMBED_TYPE, 0);
 
        DRETURN_INT(ecore_x_cursor_size_get(), DLEVEL_STABLE);
 }
@@ -1001,8 +1001,8 @@
        Ecore_X_Cursor cur;
 
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR("embed", embed);
-       DCHECK_TYPE("embed", embed, EWL_EMBED_TYPE);
+       DCHECK_PARAM_PTR(embed);
+       DCHECK_TYPE(embed, EWL_EMBED_TYPE);
 
        if (pointer >= EWL_MOUSE_CURSOR_MAX)
                cur = pointer;
@@ -1366,7 +1366,7 @@
        Ecore_X_Event_Selection_Notify *ev;
 
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR_RET("e", e, FALSE);
+       DCHECK_PARAM_PTR_RET(e, FALSE);
 
        ev = e;
 
@@ -1420,7 +1420,7 @@
        Ecore_X_Event_Selection_Request *ev;
 
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR_RET("e", e, FALSE);
+       DCHECK_PARAM_PTR_RET(e, FALSE);
 
        ev = e;
 
@@ -1467,7 +1467,7 @@
        int will_accept = 0;
 
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR_RET("e", e, FALSE);
+       DCHECK_PARAM_PTR_RET(e, FALSE);
 
        ev = e;
 
@@ -1518,7 +1518,7 @@
        int i = 0;
 
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR_RET("e", e, FALSE);
+       DCHECK_PARAM_PTR_RET(e, FALSE);
 
        ev = e;
 
@@ -1541,7 +1541,7 @@
        int i;
 
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR_RET("e", e, FALSE);
+       DCHECK_PARAM_PTR_RET(e, FALSE);
 
        ev = e;
 
@@ -1568,7 +1568,7 @@
        int internal = 0;
 
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR_RET("e", e, FALSE);
+       DCHECK_PARAM_PTR_RET(e, FALSE);
 
        ev = e;
 



-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
enlightenment-cvs mailing list
enlightenment-cvs@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-cvs

Reply via email to