[riot-notifications] [RIOT-OS/RIOT] dist/tools/vera++: improve support C/C++ support and optimize CI call (#15709)

Martine Lenders notifications at github.com
Wed Jan 6 11:56:19 CET 2021


@miri64 commented on this pull request.



> @@ -38,7 +47,10 @@ foreach f [getSourceFileNames] {
             }
             set state "other"
         } else {
-            if [isKeyword $tokenName] {
+            # Check if it's a C++ file
+            regexp {\.[ch]pp} $f m
+            # C keywords are valid in C++ but not the other way around
+            if {[isCKeyword $tokenName] || ([info exists m] && [isCPPKeyword $tokenName])} {

But I guess this `||` has the same effect?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/15709#pullrequestreview-562567865
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210106/89eb061c/attachment.htm>


More information about the notifications mailing list