RTEMS | Reformat cpukit rtems src (!237)

Sebastian Huber (@sebhub) gitlab at rtems.org
Tue Sep 24 00:02:37 UTC 2024




Sebastian Huber commented on a discussion on cpukit/rtems/src/taskconstruct.c: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/237#note_112466

 >  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, I would rather lose a couple of good formatting changes if there are several bad ones. This alignment stuff is a lot more complex in the tool since it has to consider several lines to make decisions. This could lead to a continuous maintenance effort.

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


More information about the bugs mailing list