RTEMS | bsps/x86_64: Address -Wsign-compare warnings (!1033)
Chris Johns (@chris)
gitlab at rtems.org
Wed Feb 4 22:23:54 UTC 2026
Merge request https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1033 was reviewed by Chris Johns
--
Chris Johns started a new discussion on bsps/x86_64/amd64/console/eficonsole.c: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1033#note_141297
> if (status == EFI_SUCCESS) {
> - for (int i = 0; i < HandleCount; i++) {
> + for (uint32_t i = 0; i < HandleCount; i++) {
This should use the same type as `HandleCount`. It may equate to `uint32_t` however some reviewing the code like me needs to check or assume this is right.
--
Chris Johns started a new discussion on bsps/x86_64/amd64/console/efigop.c: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1033#note_141298
> if (!EFI_ERROR (status)) {
> - for (int i = 0; i < HandleCount; i++) {
> + for (uint32_t i = 0; i < HandleCount; i++) {
Same issue here.
--
View it on GitLab: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1033
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/20260204/fdad736a/attachment.htm>
More information about the bugs
mailing list