[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:55:33 CET 2021


@miri64 commented on this pull request.



> +proc isCPPKeyword {s} {
+    global cpp_keywords
+    return [expr [lsearch $cpp_keywords $s] != -1]
 }

I understood that as much. But C keywords are also C++ keywords, so I would have expected something like

```suggestion
proc isCPPKeyword {s} {
    global cpp_keywords
    return [expr [lsearch $cpp_keywords $s] != -1] || [isCKeyword $s]
}
```

-- 
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#discussion_r552505580
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210106/138265ff/attachment.htm>


More information about the notifications mailing list