Unfork MapBuffer.cpp override added during 0.85 nightly integration
メンテナーはふだん 2 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 52/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 静か
- 技術スタック
- cpp, react-native
- 領域
- build-system
調査の方向性
vnext/ReactCommon/TEMP_UntilReactCommonUpdate/react/renderer/mapbuffer/MapBuffer.cpp から始め、報告された行を issue にリンクされている upstream の MapBuffer.cpp の変更と比較します。Warnings.props が C4018 と C4267 をどのように扱っているかを確認し、その後、該当する Windows ビルドを実行して、両方の警告が解消され、一時的な override が正しく処理されることを確認します。
索引モデルが issue の本文から書いたものです。
説明
Problem Description
During the integration of RN 0.85.0-nightly-20260128-36f07a1b2 (#16245), the upstream MapBuffer.cpp introduced signed/unsigned comparison warnings (C4018 and C4267) that our Warnings.props treats as errors (SDL-mandatory).
- Line 27: header->bufferSize (uint32_t) compared with bytes_.size() (size_t) — C4267
- Line 128: mapBufferLength (int32_t) compared with maxLength (size_t) — C4018
Current Fix:
A local patched copy was created at vnext/ReactCommon/TEMP_UntilReactCommonUpdate/react/renderer/mapbuffer/MapBuffer.cpp with static_cast fixes.
Link to the integration PR where the error originated
https://github.com/microsoft/react-native-windows/pull/16245
Link to react-native commit
https://github.com/facebook/react-native/compare/f15985f4f...36f07a1b2
Forked files
No response
Upstream PR in react-native
No response
Build Configuration
None
- 主要言語
- C++
- スター
- 17.4k
- フォーク
- 1.2k
- 平均マージ
- 2日 20時間
- マージ済み PR（30日）
- 15
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
microsoft/react-native-windows のほかの issue
-
Fabric text is drawn with ClearType onto transparent composition surfaces, fringing thin glyphsオープンNeeds: Triage :mag:
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
microsoft/react-native-windows#16340 ·
メンテナーはふだん 2 日以内に返信
-
bug Needs: Triage :mag:
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
microsoft/react-native-windows#16321 · コメント 1 件 ·
メンテナーはふだん 2 日以内に返信
-
bug Needs: Triage :mag:
難易度 3/5 1〜2日 初心者へのやさしさ 48/100
microsoft/react-native-windows#16479 ·
メンテナーはふだん 2 日以内に返信
-
Needs: Triage :mag:
難易度 5/5 1週間以上 初心者へのやさしさ 30/100
microsoft/react-native-windows#16442 · コメント 2 件 ·
メンテナーはふだん 2 日以内に返信
-
Fabric: activating the window doesn't announce the window or the focused control to a screen readerオープンNeeds: Triage :mag:
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
microsoft/react-native-windows#16435 ·
メンテナーはふだん 2 日以内に返信
microsoft/react-native-windows の issue をすべて見る
似ている issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
メンテナーはふだん 1 日以内に返信
-
c++ good first issue M2: audio plumbing no hardware needed tests
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
GageLawton/noise-cancellation#23 ·
メンテナーはふだん 1 日以内に返信
-
Bug
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
nullmoth/nvidia-macos-driver#126 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
objectionary/eo-graphs#85 ·