Playing a bit around with clang-tidy's new checkers.
Please have a thorough review. I didn't find anything wrong in the patch, but another pair of eyes won't hurt.
dhaumann |
Playing a bit around with clang-tidy's new checkers.
Please have a thorough review. I didn't find anything wrong in the patch, but another pair of eyes won't hurt.
No new test failures in a ctest run. However some vimode related unit tests before & after failed for me:
The following tests FAILED: 63 - vimode_modes (Failed) 67 - vimode_emulatedcommandbar (Failed)
Automatic diff as part of commit; lint not applicable. |
Automatic diff as part of commit; unit tests not applicable. |
The patch looks good. Please commit this weekend, or wait until next Saturday's v5.45 branch.