Kinsey Moore started a new discussion on cpukit/posix/src/aio_misc.c: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/118#note_109824

 >    return 0;
 >  }
 >  
 > -static void *rtems_aio_handle( void *arg )
 > +int rtems_aio_check_sigevent( struct sigevent *sigp )
 > +{
 > +  _Assert( sigp != NULL );
 > +
 > +  switch (sigp->sigev_notify) {
 > +    case SIGEV_NONE:
 > +      break;
 > +
 > +    case SIGEV_SIGNAL:
 > +      if (sigp->sigev_signo < 1 && sigp->sigev_signo > 32 ) {

Was this meant to be a || instead of a &&? This was flagged by a coverity scan 
as dead code.

-- 
View it on GitLab: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/118#note_109824
You're receiving this email because of your account on gitlab.rtems.org.


_______________________________________________
bugs mailing list
[email protected]
http://lists.rtems.org/mailman/listinfo/bugs

Reply via email to