Skip to content
Compliance Checks

drivers: can: shell: use shell_fprintf_normal instead of shell_fprintf #148406

Sign in to view logs

drivers: can: shell: use shell_fprintf_normal instead of shell_fprintf

drivers: can: shell: use shell_fprintf_normal instead of shell_fprintf #148406

Triggered via pull request September 10, 2024 02:07
@ndrs-pstndrs-pst
edited #77165
DDC-NDRS:pr_drivers_can_shell_use_shell_print_impl
Status Success
Total duration 2m 28s
Artifacts 1

compliance.yml

on: pull_request
Run compliance checks on patch series (PR)
2m 19s
Run compliance checks on patch series (PR)
Fit to window
Zoom out
Zoom in

Annotations

1 error, 1 warning, and 1 notice
Run compliance checks on patch series (PR)
Process completed with exit code 1.
Run compliance checks on patch series (PR): ClangFormat.txt#L1
See https://docs.zephyrproject.org/latest/contribute/guidelines.html#clang-format for more details. You may want to run clang-format on this change: - shell_fprintf_normal(sh, "%c%c ", - (frame->flags & CAN_FRAME_BRS) == 0 ? '-' : 'B', + shell_fprintf_normal(sh, "%c%c ", (frame->flags & CAN_FRAME_BRS) == 0 ? '-' : 'B', (frame->flags & CAN_FRAME_ESI) == 0 ? '-' : 'P'); #endif /* CONFIG_CAN_FD_MODE */ /* CAN ID */ - shell_fprintf_normal(sh, "%*s%0*x ", - (frame->flags & CAN_FRAME_IDE) != 0 ? 0 : 5, "", + shell_fprintf_normal(sh, "%*s%0*x ", (frame->flags & CAN_FRAME_IDE) != 0 ? 0 : 5, "", (frame->flags & CAN_FRAME_IDE) != 0 ? 8 : 3, - (frame->flags & CAN_FRAME_IDE) != 0 ? - frame->id & CAN_EXT_ID_MASK : frame->id & CAN_STD_ID_MASK); + (frame->flags & CAN_FRAME_IDE) != 0 ? frame->id & CAN_EXT_ID_MASK + : frame->id & CAN_STD_ID_MASK); /* DLC as number of bytes */ - shell_fprintf_normal(sh, "%s[%0*d] ", - (frame->flags & CAN_FRAME_FDF) != 0 ? "" : " ", - (frame->flags & CAN_FRAME_FDF) != 0 ? 2 : 1, - nbytes); + shell_fprintf_normal(sh, "%s[%0*d] ", (frame->flags & CAN_FRAME_FDF) != 0 ? "" : " ", + (frame->flags & CAN_FRAME_FDF) != 0 ? 2 : 1, nbytes); File:drivers/can/can_shell.c Line:117
You may want to run clang-format on this change: drivers/can/can_shell.c#L117
drivers/can/can_shell.c:117 - shell_fprintf_normal(sh, "%c%c ", - (frame->flags & CAN_FRAME_BRS) == 0 ? '-' : 'B', + shell_fprintf_normal(sh, "%c%c ", (frame->flags & CAN_FRAME_BRS) == 0 ? '-' : 'B', (frame->flags & CAN_FRAME_ESI) == 0 ? '-' : 'P'); #endif /* CONFIG_CAN_FD_MODE */ /* CAN ID */ - shell_fprintf_normal(sh, "%*s%0*x ", - (frame->flags & CAN_FRAME_IDE) != 0 ? 0 : 5, "", + shell_fprintf_normal(sh, "%*s%0*x ", (frame->flags & CAN_FRAME_IDE) != 0 ? 0 : 5, "", (frame->flags & CAN_FRAME_IDE) != 0 ? 8 : 3, - (frame->flags & CAN_FRAME_IDE) != 0 ? - frame->id & CAN_EXT_ID_MASK : frame->id & CAN_STD_ID_MASK); + (frame->flags & CAN_FRAME_IDE) != 0 ? frame->id & CAN_EXT_ID_MASK + : frame->id & CAN_STD_ID_MASK); /* DLC as number of bytes */ - shell_fprintf_normal(sh, "%s[%0*d] ", - (frame->flags & CAN_FRAME_FDF) != 0 ? "" : " ", - (frame->flags & CAN_FRAME_FDF) != 0 ? 2 : 1, - nbytes); + shell_fprintf_normal(sh, "%s[%0*d] ", (frame->flags & CAN_FRAME_FDF) != 0 ? "" : " ", + (frame->flags & CAN_FRAME_FDF) != 0 ? 2 : 1, nbytes);

Artifacts

Produced during runtime
Name Size
compliance.xml
879 Bytes