Merge request https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/232 was reviewed by Kinsey Moore
-- Kinsey Moore started a new discussion on cpukit/include/rtems/cpuuse.h: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/232#note_113090 > > + > + Remove these extra lines that were added. There are other extra lines added in this patch that are not necessary. -- Kinsey Moore started a new discussion on cpukit/include/rtems/cpuuse.h: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/232#note_113091 > + * @brief Contains the data for the cpuuse data. > + */ > +typedef struct { Indentation and formatting of the lines within this struct are not consistent. Additionally, the @brief documentation should be more descriptive. -- Kinsey Moore started a new discussion on cpukit/include/rtems/cpuuse.h: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/232#note_113092 > + > + /** > + * @brief name of the thread. Accidental double space, check for more of these. -- Kinsey Moore started a new discussion on cpukit/include/rtems/cpuuse.h: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/232#note_113093 > + > + /** > + * @brief system uptime since last restart . Extra space before the period at the end. This issue exists elsewhere, as well. -- View it on GitLab: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/232 You're receiving this email because of your account on gitlab.rtems.org.
_______________________________________________ bugs mailing list [email protected] http://lists.rtems.org/mailman/listinfo/bugs
