Skip to content

Commit

Permalink
remove cpplint check of unapproved header
Browse files Browse the repository at this point in the history
  • Loading branch information
YinshiSanchez committed Jun 2, 2024
1 parent 18ed4fe commit a6ecc72
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 14 deletions.
26 changes: 13 additions & 13 deletions build_support/cpplint.py
Original file line number Diff line number Diff line change
Expand Up @@ -6125,19 +6125,19 @@ def FlagCxx11Features(filename, clean_lines, linenum, error):
('C++ TR1 headers such as <%s> are unapproved.') % include.group(1))

# Flag unapproved C++11 headers.
if include and include.group(1) in ('cfenv',
'condition_variable',
'fenv.h',
'future',
'mutex',
'thread',
'chrono',
'ratio',
'regex',
'system_error',
):
error(filename, linenum, 'build/c++11', 5,
('<%s> is an unapproved C++11 header.') % include.group(1))
# if include and include.group(1) in ('cfenv',
# 'condition_variable',
# 'fenv.h',
# 'future',
# 'mutex',
# 'thread',
# 'chrono',
# 'ratio',
# 'regex',
# 'system_error',
# ):
# error(filename, linenum, 'build/c++11', 5,
# ('<%s> is an unapproved C++11 header.') % include.group(1))

# The only place where we need to worry about C++11 keywords and library
# features in preprocessor directives is in macro definitions.
Expand Down
1 change: 1 addition & 0 deletions src/include/storage/disk/disk_scheduler.h
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@
#include <thread> // NOLINT
#include <type_traits>
#include <utility>
#include <vector>

#include "common/channel.h"
#include "storage/disk/disk_manager.h"
Expand Down
2 changes: 1 addition & 1 deletion src/storage/disk/disk_manager_memory.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -47,4 +47,4 @@ void DiskManagerMemory::ReadPage(page_id_t page_id, char *page_data) {
memcpy(page_data, memory_ + offset, BUSTUB_PAGE_SIZE);
}

} // namespace bustub
} // namespace bustub

0 comments on commit a6ecc72

Please sign in to comment.