Alessandro Nardin commented on a discussion on cpukit/posix/src/aio_suspend.c: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/275#note_114697

 > +    r_chain = rtems_aio_search_fd( work_req_chain, list[i]->aio_fildes, 0 );
 > +    if ( r_chain != NULL ) {
 > +      //SEARCH IN CHAIN
 > +      fd_chain = &r_chain->perfd;
 > +      node = rtems_chain_first( fd_chain );
 > +      current = (rtems_aio_request *) node;
 > +
 > +      while ( !rtems_chain_is_tail( fd_chain, node ) && current->aiocbp != 
 > list[i] ) {
 > +        node = rtems_chain_next( node );
 > +        current = (rtems_aio_request *) node;
 > +      }
 > +      
 > +      if ( !rtems_chain_is_tail( fd_chain, node ) ){
 > +        if ( current->suspendcbp == NULL ) {
 > +          current->suspendcbp = suspendcbp;
 > +        }

I refactored the entire body of the for loop and extracted the search logic 
into an internal function.

-- 
View it on GitLab: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/275#note_114697
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