Enlightenment CVS committal

Author  : pfritz
Project : e17
Module  : libs/ewl

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


Modified Files:
        ewl_engine_x11.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/x11/ewl_engine_x11.c,v
retrieving revision 1.54
retrieving revision 1.55
diff -u -3 -r1.54 -r1.55
--- ewl_engine_x11.c    2 Oct 2007 08:16:16 -0000       1.54
+++ ewl_engine_x11.c    12 Nov 2007 22:42:21 -0000      1.55
@@ -173,7 +173,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)
@@ -304,7 +304,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);
@@ -391,8 +391,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,
@@ -418,8 +418,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));
@@ -435,8 +435,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,
@@ -451,8 +451,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));
@@ -470,8 +470,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_set((Ecore_X_Window)win->window,
                                        0, ECORE_X_GRAVITY_NW,
@@ -490,8 +490,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);
@@ -503,8 +503,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);
@@ -516,8 +516,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);
@@ -529,8 +529,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));
@@ -542,8 +542,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)));
@@ -555,8 +555,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)) ?
@@ -572,8 +572,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);
@@ -596,8 +596,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)
        {
@@ -613,8 +613,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);
@@ -639,8 +639,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 */
@@ -672,8 +672,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;
@@ -701,8 +701,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,
@@ -722,8 +722,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;
@@ -744,8 +744,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);
 
@@ -756,8 +756,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);
 
@@ -770,8 +770,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)))
        {
@@ -791,8 +791,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;
@@ -806,8 +806,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)))
        {
@@ -827,8 +827,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;
@@ -840,9 +840,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);
 
        if (txt)
                ecore_x_selection_primary_set(
@@ -869,8 +869,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);
 
@@ -881,8 +881,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_window_size_get(0, w, h);
 
@@ -893,8 +893,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_aware_set((Ecore_X_Window)embed->canvas_window,
                              (num > 0 ? 1 : 0));
@@ -907,8 +907,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((Ecore_X_Window)embed->canvas_window, NULL, 0);
 
@@ -919,8 +919,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();
 
@@ -933,9 +933,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);
@@ -949,8 +949,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);
@@ -960,8 +960,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);
 
@@ -976,8 +976,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);
 }
@@ -988,8 +988,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;
@@ -1353,7 +1353,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;
 
@@ -1407,7 +1407,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;
 
@@ -1448,7 +1448,7 @@
        int will_accept = 0;
 
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR_RET("e", e, FALSE);
+       DCHECK_PARAM_PTR_RET(e, FALSE);
 
        ev = e;
 
@@ -1499,7 +1499,7 @@
        int i = 0;
 
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR_RET("e", e, FALSE);
+       DCHECK_PARAM_PTR_RET(e, FALSE);
 
        ev = e;
 
@@ -1522,7 +1522,7 @@
        int i;
 
        DENTER_FUNCTION(DLEVEL_STABLE);
-       DCHECK_PARAM_PTR_RET("e", e, FALSE);
+       DCHECK_PARAM_PTR_RET(e, FALSE);
 
        ev = e;
 
@@ -1549,7 +1549,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