Skip to content

Commit

Permalink
disable C4710
Browse files Browse the repository at this point in the history
  • Loading branch information
flagarde committed Jan 26, 2025
1 parent 21c362a commit c365c36
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 20 deletions.
8 changes: 5 additions & 3 deletions cmake/Warnings.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -64,9 +64,11 @@ set(MSVC_WARNINGS
"/W4"
"/Wall"
"/WL"
"/wd5045"
"/wd4820"
"/wd4514")
"/wd5045" #FIXME
"/wd4820" #FIXME
"/wd4514" #FIXME
"/wd4710" #FIXME
)

target_compile_options(
${PROJECT_NAME}Warnings
Expand Down
8 changes: 0 additions & 8 deletions cpp-terminal/buffer.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -11,15 +11,7 @@

#include <cstddef>

Check warning on line 12 in cpp-terminal/buffer.hpp

View workflow job for this annotation

GitHub Actions / cpp-linter

cpp-terminal/buffer.hpp:12:1 [misc-include-cleaner]

included header cstddef is not used directly
#include <cstdint>

#ifdef _WIN32
#pragma warning( push )
#pragma warning( disable : 4710)
#endif
#include <streambuf>
#ifdef _WIN32
#pragma warning( pop )
#endif

namespace Term
{
Expand Down
8 changes: 0 additions & 8 deletions cpp-terminal/iostream_initializer.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -16,15 +16,7 @@
#include "cpp-terminal/tty.hpp"

#include <cstddef>

#ifdef _WIN32
#pragma warning( push )
#pragma warning( disable : 4710)
#endif
#include <iostream>
#ifdef _WIN32
#pragma warning( pop )
#endif

std::size_t Term::IOStreamInitializer::m_counter{0};

Expand Down
1 change: 0 additions & 1 deletion cpp-terminal/private/terminal_impl.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@
#include <io.h>
#pragma warning(push)
#pragma warning(disable : 4668)
#pragma warning(disable : 4710)
#include <windows.h>
#pragma warning(pop)
#if !defined(ENABLE_VIRTUAL_TERMINAL_PROCESSING)
Expand Down

0 comments on commit c365c36

Please sign in to comment.