change log for rtems (2011-10-26)

Ralf Corsepius ralf.corsepius at rtems.org
Wed Oct 26 15:05:36 UTC 2011


On 10/26/2011 04:53 PM, Gedare Bloom wrote:
>
>  >
>  > * libmisc/monitor/mon-monitor.c (rtems_monitor_wakeup):
>  > Remove unused var "status" (Avoid warning).
>  >
>  > M
>  > 1.2973
>  > cpukit/ChangeLog
>  > M
>  > 1.44
>  > cpukit/libmisc/monitor/mon-monitor.c
>  >
>  > diff -u rtems/cpukit/ChangeLog:1.2972 rtems/cpukit/ChangeLog:1.2973
>  > --- rtems/cpukit/ChangeLog:1.2972Wed Oct 26 05:32:09 2011
>  > +++ rtems/cpukit/ChangeLogWed Oct 26 09:05:32 2011
>  > @@ -1,3 +1,8 @@
>  > +2011-10-26Ralf Corsépius <ralf.corsepius at rtems.org
> <mailto:ralf.corsepius at rtems.org>>
>  > +
>  > +* libmisc/monitor/mon-monitor.c (rtems_monitor_wakeup):
>  > +Remove unused var "status" (Avoid warning).
>  > +
>  >  2011-10-26Sebastian Huber <sebastian.huber at embedded-brains.de
> <mailto:sebastian.huber at embedded-brains.de>>
>  >
>  >  * ftpd/ftpd.h, ftpd/ftpd.c: Moved buffer and stack size constants into
>  >
>  > diff -u rtems/cpukit/libmisc/monitor/mon-monitor.c:1.43
> rtems/cpukit/libmisc/monitor/mon-monitor.c:1.44
>  > --- rtems/cpukit/libmisc/monitor/mon-monitor.c:1.43Fri Sep  2
> 08:17:35 2011
>  > +++ rtems/cpukit/libmisc/monitor/mon-monitor.cWed Oct 26 09:05:32 2011
>  > @@ -295,9 +295,7 @@
>  >  void
>  >  rtems_monitor_wakeup(void)
>  >  {
>  > -    rtems_status_code status;
>  > -
>  > -    status = rtems_event_send(rtems_monitor_task_id,
> MONITOR_WAKEUP_EVENT);
>  > +    rtems_event_send(rtems_monitor_task_id, MONITOR_WAKEUP_EVENT);
>  >  }
> Should the return value be checked or discarded?

In general, I think the return values should be checked and used 
whenever possible and useful.

In this particular case, I don't know what to do, ...

May-be somebody has a better proposal than simply ignoring the return value.

Ralf




More information about the vc mailing list