Documentation | eng: remove links to devel.rtems.org (!146)

Chris Johns (@chris) gitlab at rtems.org
Fri Feb 7 00:57:59 UTC 2025



Merge request https://gitlab.rtems.org/rtems/docs/rtems-docs/-/merge_requests/146 was reviewed by Chris Johns

--
  
Chris Johns started a new discussion on eng/coding-formatting.rst: https://gitlab.rtems.org/rtems/docs/rtems-docs/-/merge_requests/146#note_119297

 >  
 > +* Minimize reformatting existing code in RTEMS unless the file undergoes
 > +  substantial non-style changes.

And if the file is one of a number in the same area of code. There is no point changing a file in a set unless the whole set of files is changed.

--
  
Chris Johns started a new discussion on eng/coding-conventions.rst: https://gitlab.rtems.org/rtems/docs/rtems-docs/-/merge_requests/146#note_119298

 > +    #if defined(__rtems__)
 > +      /* changes made */
 > +    #endif

Should this be:

```
#endif /* __rtems__ */
```

--
  
Chris Johns started a new discussion on eng/coding-conventions.rst: https://gitlab.rtems.org/rtems/docs/rtems-docs/-/merge_requests/146#note_119299

 > +        #ifdef FEATURE_X
 > +          /* Do something */
 > +        #else

Comments to help isolate which define the `#else` and `#endif` is linked to the define?


-- 
View it on GitLab: https://gitlab.rtems.org/rtems/docs/rtems-docs/-/merge_requests/146
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/20250207/ee6542fa/attachment.htm>


More information about the bugs mailing list