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.
_______________________________________________ bugs mailing list [email protected] http://lists.rtems.org/mailman/listinfo/bugs
