Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Remove DisableSpecificWarnings suppressing C4996 for legacy APIs

Đang mở
  • Có gì không ổn?
#15,535 0 bình luận 0 reaction 1 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 2 ngày

@iamAbhi-916 đang làm issue này rồi.

Từ ngày 12/1/2026.

Đánh giá

Issue này chưa được đánh giá.

Mô tả

enhancement

Ref : https://github.com/microsoft/react-native-windows/pull/15517
We're silencing the C4996 deprecation warnings because the deprecated headers are being pulled in transitively from upstream React Native's New Architecture code (react/runtime/ReactInstance.h → jsireact/JSIExecutor.h → legacy cxxreact headers.

it's upstream React Native that still includes these legacy headers. Until React Native completes their legacy removal, we can't fix these at the source.

RN use of legacy apis in 0.82, verify at ref : https://github.com/facebook/react-native/tree/9731e8ebc5ea87526a91b9903172639e062cd920/packages/react-native/ReactCommon/cxxreact

ReactCommon.vcxproj (Upstream RN cxxreact files)

  • cxxreact/ReactMarker.cpp
  • cxxreact/JSExecutor.cpp
  • cxxreact/Instance.cpp
  • cxxreact/NativeToJsBridge.cpp
  • cxxreact/JSIndexedRAMBundle.cpp
  • cxxreact/MethodCall.cpp
  • cxxreact/RAMBundleRegistry.cpp
  • cxxreact/ModuleRegistry.cpp
  • jsiexecutor/jsireact/JSIExecutor.cpp
  • jsiexecutor/jsireact/JSINativeModules.cpp

##Shared.vcxitems (RNW code with transitive dependencies)

  • Microsoft.ReactNative/ReactCoreInjection.cpp
  • Microsoft.ReactNative/ReactHost/CrashManager.cpp
  • Microsoft.ReactNative/ReactHost/MsoReactContext.cpp
  • Microsoft.ReactNative/ReactHost/ReactInstanceWin.cpp
  • react/renderer/core/EventQueueProcessor.cpp (upstream)
  • react/renderer/uimanager/UIManager.cpp (upstream)
  • react/runtime/ReactInstance.cpp (upstream)

Test Projects

  • Desktop.UnitTests/InstanceMocks.cpp
  • IntegrationTests/TestModule.cpp
  • Desktop.IntegrationTests/Modules/TestDevSettingsModule.cpp
  • Desktop.IntegrationTests/Modules/TestImageLoaderModule.cp
Ngôn ngữ chính
C++
Star
17.4k
Fork
1.2k
Merge trung bình
3 ngày 10 phút
Pull request đã merge (30 ngày)
14

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của microsoft/react-native-windows

Tất cả issue của microsoft/react-native-windows

Issue tương tự

Thêm issue về C++

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.