<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Nov 20, 2020, 5:16 AM Sebastian Huber <<a href="mailto:sebastian.huber@embedded-brains.de">sebastian.huber@embedded-brains.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This define was only used in one place.<br>
---<br>
 cpukit/include/rtems/rtems/eventimpl.h | 8 +-------<br>
 1 file changed, 1 insertion(+), 7 deletions(-)<br>
<br>
diff --git a/cpukit/include/rtems/rtems/eventimpl.h b/cpukit/include/rtems/rtems/eventimpl.h<br>
index 9ccee0adbd..c65c202de4 100644<br>
--- a/cpukit/include/rtems/rtems/eventimpl.h<br>
+++ b/cpukit/include/rtems/rtems/eventimpl.h<br>
@@ -32,12 +32,6 @@ extern "C" {<br>
  * @{<br>
  */<br>
<br>
-/**<br>
- *  The following constant is the value of an event set which<br>
- *  has no events pending.<br>
- */<br>
-#define EVENT_SETS_NONE_PENDING 0<br>
-<br>
 rtems_status_code _Event_Seize(<br>
   rtems_event_set    event_in,<br>
   rtems_option       option_set,<br>
@@ -68,7 +62,7 @@ void _Event_Timeout(<br>
<br>
 RTEMS_INLINE_ROUTINE void _Event_Initialize( Event_Control *event )<br>
 {<br>
-  event->pending_events = EVENT_SETS_NONE_PENDING;<br>
+  event->pending_events = 0;<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">This loses clarity to me. If you want to ditch the constant, you have to add a comment.</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
 }<br>
<br>
 /**<br>
-- <br>
2.26.2<br>
<br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank" rel="noreferrer">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div></div></div>