RTEMS | Reformat cpukit rtems src (!237)

Amar Takhar (@amar) gitlab at rtems.org
Tue Sep 24 00:17:32 UTC 2024




Amar Takhar commented on a discussion on cpukit/rtems/src/taskconstruct.c: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/237#note_112470

 >  static User_extensions_Control _RTEMS_tasks_User_extensions = {
 >    .Callouts = {
 >  #if defined( RTEMS_MULTIPROCESSING )
 > -    .thread_terminate = _RTEMS_tasks_Terminate_extension,
 > +               .thread_terminate = _RTEMS_tasks_Terminate_extension,
 >  #endif
 > -    .thread_start   = _RTEMS_tasks_Start_extension,
 > -    .thread_restart = _RTEMS_tasks_Start_extension
 > +               .thread_start   = _RTEMS_tasks_Start_extension,
 > +               .thread_restart = _RTEMS_tasks_Start_extension

Yes, that was my understanding I'm talking about this specific option that I would like to see it enabled and getting the indent wrong in one place is not enough to lose the other formatting it has done.  We'll see what the others think your one point is for sure valid about it being a moving target but again if that's a problem we can disable it then.

-- 
View it on GitLab: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/237#note_112470
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/20240924/e461a356/attachment.htm>


More information about the bugs mailing list