Gedare Bloom commented on a discussion on cpukit/score/src/corerwlockrelease.c: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/747#note_140266

 >    }
 >  
 >    _Assert(
 > -    the_rwlock->current_state == CORE_RWLOCK_LOCKED_FOR_WRITING
 > -      || ( the_rwlock->current_state == CORE_RWLOCK_LOCKED_FOR_READING
 > -        && the_rwlock->number_of_readers == 0 )
 > +    the_rwlock->current_state == CORE_RWLOCK_LOCKED_FOR_WRITING ||
 > +    ( the_rwlock->current_state == CORE_RWLOCK_LOCKED_FOR_READING &&
 > +      the_rwlock->number_of_readers == 0 )

For the most part, I wrote the coding guidelines over time, based on a 
combination of looking at what we had, and asking Joel. So they're not really 
well defined especially in places where there wasn't much consistency or few 
examples. Older code tended not to have compound expressions, so we didn't see 
many of these cases in the core.

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