RTEMS | Draft: cpukit/posix/aio*: Create notification at request completion (!118)

Fri Jul 19 18:17:16 UTC 2024

Alessandro Nardin commented on a discussion on cpukit/posix/src/aio_misc.c: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/118#note_109627

 > +{
 > +
 > +#ifdef RTEMS_POSIX_API
 > +
 > +  int result;
 > +
 > +  _Assert( sigp != NULL );
 > +
 > +  switch ( sigp->sigev_notify ) {
 > +    case SIGEV_SIGNAL:
 > +      result = sigqueue(
 > +        getpid(),
 > +        sigp->sigev_signo,
 > +        sigp->sigev_value
 > +      );
 > +      _Assert( result == 0);

Shouldn't I add a space before the ")" instead? It would be more coherent with how I formatted the other function calls.

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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/bugs/attachments/20240719/96f783ee/attachment-0001.htm>