From 9a3c4bc67c6ee06993e295d957681452f829577d Mon Sep 17 00:00:00 2001 From: dan sinclair Date: Fri, 17 Jun 2022 22:58:24 +0000 Subject: [PATCH] Add some c++17 headers to the list of known headers. This CL updates cpplint.py to add some of the c++17 header files to the list of known c++ headers. We're specifically using optional and variant in the Tint project and would like to remove the NOLINT lines necessary to keep the linter happy. Bug: dawn:1379 Change-Id: I41cacd21e0742f992cdd6913b7f4def556b543c5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3709241 Auto-Submit: Dan Sinclair Reviewed-by: Josip Sokcevic Commit-Queue: Josip Sokcevic --- cpplint.py | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/cpplint.py b/cpplint.py index 330b926d0..b8c72dbe7 100755 --- a/cpplint.py +++ b/cpplint.py @@ -329,15 +329,19 @@ _CPP_HEADERS = frozenset([ 'vector.h', # 17.6.1.2 C++ library headers 'algorithm', + 'any', 'array', 'atomic', 'bitset', + 'charconv', 'chrono', 'codecvt', 'complex', 'condition_variable', 'deque', + 'execution', 'exception', + 'filesystem', 'forward_list', 'fstream', 'functional', @@ -354,9 +358,11 @@ _CPP_HEADERS = frozenset([ 'locale', 'map', 'memory', + 'memory_resource', 'mutex', 'new', 'numeric', + 'optional', 'ostream', 'queue', 'random', @@ -381,6 +387,7 @@ _CPP_HEADERS = frozenset([ 'unordered_set', 'utility', 'valarray', + 'variant', 'vector', # 17.6.1.2 C++ headers for C library facilities 'cassert',