clang can format the macros just fine as they are. I added clang for checking the style of c++ code (mainly vapi code). C files are still checked using indent, which doesn't understand the macros ... Also I wasn't able to come up with a clang-format which matches the current indent style 1:1 so I don't see a switch to clang-format coming any time soon...
Thanks, Klement Quoting Stephen Hemminger (2018-11-16 02:02:55) > I just discovered the clang-format file and noticed that Linux kernel used > the ForEachMacros option to format for() wrappers. > > VPP seems to be riddled with INDENT-OFF when it does vector loops. (2123 > times!) > Has anyone looked into replacing these with ForEachMacros? > > > -=-=-=-=-=-=-=-=-=-=-=- > Links: You receive all messages sent to this group. > > View/Reply Online (#11274): https://lists.fd.io/g/vpp-dev/message/11274 > Mute This Topic: https://lists.fd.io/mt/28156993/675704 > Group Owner: [email protected] > Unsubscribe: https://lists.fd.io/g/vpp-dev/unsub [[email protected]] > -=-=-=-=-=-=-=-=-=-=-=-
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#11280): https://lists.fd.io/g/vpp-dev/message/11280 Mute This Topic: https://lists.fd.io/mt/28156993/21656 Group Owner: [email protected] Unsubscribe: https://lists.fd.io/g/vpp-dev/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
