[PATCH 18/20] rtems: Remove EVENT_SETS_NONE_PENDING

Joel Sherrill joel at rtems.org
Fri Nov 20 13:30:10 UTC 2020


On Fri, Nov 20, 2020, 5:16 AM Sebastian Huber <
sebastian.huber at embedded-brains.de> wrote:

> This define was only used in one place.
> ---
>  cpukit/include/rtems/rtems/eventimpl.h | 8 +-------
>  1 file changed, 1 insertion(+), 7 deletions(-)
>
> diff --git a/cpukit/include/rtems/rtems/eventimpl.h
> b/cpukit/include/rtems/rtems/eventimpl.h
> index 9ccee0adbd..c65c202de4 100644
> --- a/cpukit/include/rtems/rtems/eventimpl.h
> +++ b/cpukit/include/rtems/rtems/eventimpl.h
> @@ -32,12 +32,6 @@ extern "C" {
>   * @{
>   */
>
> -/**
> - *  The following constant is the value of an event set which
> - *  has no events pending.
> - */
> -#define EVENT_SETS_NONE_PENDING 0
> -
>  rtems_status_code _Event_Seize(
>    rtems_event_set    event_in,
>    rtems_option       option_set,
> @@ -68,7 +62,7 @@ void _Event_Timeout(
>
>  RTEMS_INLINE_ROUTINE void _Event_Initialize( Event_Control *event )
>  {
> -  event->pending_events = EVENT_SETS_NONE_PENDING;
> +  event->pending_events = 0;
>

This loses clarity to me. If you want to ditch the constant, you have to
add a comment.

>  }
>
>  /**
> --
> 2.26.2
>
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/devel/attachments/20201120/739ed8ed/attachment.html>


More information about the devel mailing list