From 7a7ceb7429136f05be1c2b6cc27be2153607d804 Mon Sep 17 00:00:00 2001
From: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Date: Thu, 10 Jan 2013 08:09:46 +0000
Subject: [PATCH] E_LIST_HANDLER_APPEND now assert()s the event handler

SVN revision: 82521
---
 ChangeLog   | 4 ++++
 NEWS        | 1 +
 src/bin/e.h | 6 ++----
 3 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index fd3d299b5..506789d0b 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2013-01-10 Mike Blumenkrantz
+
+        * E_LIST_HANDLER_APPEND macro now asserts the event handler to make debugging easier
+
 2013-01-10 Deon Thomas
 
         * Fixed bug with desktop config profile where conf module version variable was misnamed
diff --git a/NEWS b/NEWS
index 6fc5ce85a..b49830c6a 100644
--- a/NEWS
+++ b/NEWS
@@ -50,6 +50,7 @@ Improvements:
     * "System Default" language in wizard now displays which language it is and no longer removes language from list
     * Desktop->Shelves menu now shows shelf names
     * geometry_auto_move option now also corrects applications which fail in their attempts to center windows
+    * E_LIST_HANDLER_APPEND macro now asserts the event handler to make debugging easier
 
 Fixes:
     * Force check changed upon confirmation dialog closure for engine settings.
diff --git a/src/bin/e.h b/src/bin/e.h
index de1c7a95b..2c13e6c86 100644
--- a/src/bin/e.h
+++ b/src/bin/e.h
@@ -213,10 +213,8 @@ typedef struct _E_Rect         E_Rect;
     { \
        Ecore_Event_Handler *_eh; \
        _eh = ecore_event_handler_add(type, (Ecore_Event_Handler_Cb)callback, data); \
-       if (_eh) \
-         list = eina_list_append(list, _eh); \
-       else \
-         fprintf(stderr, "E_LIST_HANDLER_APPEND\n"); \
+       assert(_eh); \
+       list = eina_list_append(list, _eh); \
     } \
   while (0)