RTEMS | psignalclearsignals.c: Eliminate used before initialized warning (!469)

Joel Sherrill (@joel) gitlab at rtems.org
Thu May 1 16:44:17 UTC 2025




Joel Sherrill commented on a discussion on cpukit/posix/src/psignalclearsignals.c: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/469#note_122301

 > +)
 > +{
 > +  bool                 retval;
 > +  Thread_queue_Context  queue_context;
 >  
 >    if ( do_signals_acquire_release ) {
 > +    _Thread_queue_Context_initialize( &queue_context );
 > +    _POSIX_signals_Acquire( &queue_context );
 > +      retval = _POSIX_signals_Clear_signals_helper(
 > +        api,
 > +        signo,
 > +        info,
 > +        is_global,
 > +        check_blocked
 > +      );
 >      _POSIX_signals_Release( &queue_context );

That's unfortunate. But I will remove the indentation. Resolving.

We COULD require {} around the protected code which should make the indentation ok by a tool. But that would not be common and would be a lot of work to even find where we would have to add them. Just a thought.

-- 
View it on GitLab: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/469#note_122301
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/20250501/caf58d4e/attachment-0001.htm>


More information about the bugs mailing list