[ros-diffs] 01/01: [EXT2FS][REISERFS] VSSolution: Actually disable C4189 errors (#540)

[Serge Gautherie]

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=c1256dda20bbda822865b…

commit c1256dda20bbda822865b59262587ae44861fa30
Author:     Serge Gautherie <32623169+SergeGautherie(a)users.noreply.github.com>
AuthorDate: Tue Jun 5 20:16:22 2018 +0200
Commit:     Thomas Faber <18138474+ThFabba(a)users.noreply.github.com>
CommitDate: Tue Jun 5 20:16:22 2018 +0200

    [EXT2FS][REISERFS] VSSolution: Actually disable C4189 errors (#540)
    
    Multiple "error C4189: '[...]': local variable is initialized but not
referenced".
    
    Thanks to Yuntian Zhang, who hinted at this in CORE-11836.
    
    CORE-11280
---
 drivers/filesystems/ext2/CMakeLists.txt     | 4 ++--
 drivers/filesystems/reiserfs/CMakeLists.txt | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/filesystems/ext2/CMakeLists.txt
b/drivers/filesystems/ext2/CMakeLists.txt
index 94b077b0c4..864962d9ec 100644
--- a/drivers/filesystems/ext2/CMakeLists.txt
+++ b/drivers/filesystems/ext2/CMakeLists.txt
@@ -94,8 +94,8 @@ endif()
 
 if(MSVC AND (NOT USE_CLANG_CL))
     # Disable warnings: "unreferenced local variable", "initialized, but
not used variable", "benign include"
-    replace_compile_flags("/we\"4189\"" " ")
-    add_target_compile_flags(ext2fs "/wd\"4189\" /wd\"4142\"
/wd\"4101\"")
+    replace_compile_flags("/we4189" " ")
+    add_target_compile_flags(ext2fs "/wd4189 /wd4142 /wd4101")
 else()
     add_target_compile_flags(ext2fs "-Wno-pointer-sign -Wno-unused-function")
     add_target_compile_flags(ext2fs "-Wno-unused-variable
-Wno-missing-braces")
diff --git a/drivers/filesystems/reiserfs/CMakeLists.txt
b/drivers/filesystems/reiserfs/CMakeLists.txt
index 95d70eaf01..9bbfe3775d 100644
--- a/drivers/filesystems/reiserfs/CMakeLists.txt
+++ b/drivers/filesystems/reiserfs/CMakeLists.txt
@@ -86,8 +86,8 @@ if(USE_CLANG_CL OR (NOT MSVC))
     endif()
 else()
     #disable warnings: "unreferenced local variable", "initialized, but
not used variable", "benign include"
-    replace_compile_flags("/we\"4189\"" " ")
-    add_target_compile_flags(reiserfs "/wd\"4189\" /wd\"4142\"
/wd\"4101\"")
+    replace_compile_flags("/we4189" " ")
+    add_target_compile_flags(reiserfs "/wd4189 /wd4142 /wd4101")
 endif()
 
 target_link_libraries(reiserfs memcmp ${PSEH_LIB})