Skip to content

Commit

Permalink
Fix C5271 warning in IJW tests (#106153)
Browse files Browse the repository at this point in the history
  • Loading branch information
jkoritzinsky committed Aug 9, 2024
1 parent 9f44bcf commit aa3825e
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 6 deletions.
8 changes: 7 additions & 1 deletion eng/native/ijw/IJW.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ if (CLR_CMAKE_HOST_WIN32)

# 4365 - signed/unsigned mismatch
# 4679 - Could not import member. This is an issue with IJW and static abstract methods in interfaces.
add_compile_options(/wd4365 /wd4679 /wd5271)
add_compile_options(/wd4365 /wd4679)

# IJW
add_compile_options(/clr:netcore)
Expand Down Expand Up @@ -93,6 +93,12 @@ if (CLR_CMAKE_HOST_WIN32)

add_compile_options(/AI${CLR_SDK_REF_PACK})

file(GLOB CLR_SDK_REF_PACK_LIBS "${CLR_SDK_REF_PACK}/*.dll")

foreach(lib ${CLR_SDK_REF_PACK_LIBS})
add_compile_options(/FU${lib})
endforeach()

list(APPEND LINK_LIBRARIES_ADDITIONAL ijwhost)

endif()
3 changes: 0 additions & 3 deletions src/native/corehost/test/ijw/ijw.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,6 @@

#include <iostream>

#using <System.Console.dll>
#using <System.Runtime.Loader.dll>

using namespace System;
using namespace System::Reflection;
using namespace System::Runtime::Loader;
Expand Down
2 changes: 0 additions & 2 deletions src/tests/Interop/IJW/NativeVarargs/IjwNativeVarargs.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,6 @@
#include <array>
#include <functional>
#include <iostream>
#using <System.Runtime.dll>
#using <System.Collections.dll>
using namespace System::Collections::Generic;

public enum class TestCases
Expand Down

0 comments on commit aa3825e

Please sign in to comment.