RTEMS | cpukit/score: reformat with clang-format (!747)

Joel Sherrill (@joel) gitlab at rtems.org
Wed Jan 28 21:02:34 UTC 2026




Joel Sherrill commented on a discussion on cpukit/score/src/corerwlockrelease.c: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/747#note_140264

 >    }
 >  
 >    _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 )

This change is right per what OAR style does. && and || go at end of a line.

I thought that spaces after an opening ( and before the closing ) were not in OAR style. 

We may have evolved away from that over the years. But the code had to stay under 80 columns and those spaces would have been making lines longer.

-- 
View it on GitLab: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/747#note_140264
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/20260128/6a128573/attachment.htm>


More information about the bugs mailing list