Unfork NetworkIOAgent.cpp destructor and enableAgent API changes from 0.85 nightly integration

Offen
#16,263 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
38/100
Issue-Typ
Refactoring
Klarheit
Klar beschrieben
Aktivitätsstatus
Ruhig
Tech-Stack
cpp, react-native
Bereich
tooling

Rechercherichtung

Beginne mit vnext/ReactCommon/TEMP_UntilReactCommonUpdate/jsinspector-modern/NetworkIOAgent.cpp und vergleiche die Datei mit dem im Issue verlinkten React Native-Commit; verwende den Integrations-PR #16245 als Kontext. Als erledigt gilt die Aufgabe, wenn der Override den Destruktor enthält und enableAgent()/disableAgent() konsistent mit der Upstream-API verwendet.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Integration Follow-up Needs: Triage :mag:
Problem Description

During the integration of RN 0.85.0-nightly-20260128-36f07a1b2 (#16245), the upstream NetworkIOAgent API changed from enable()/disable() to enableAgent()/disableAgent(). A destructor was also added to properly clean up the network agent. These changes were applied to the existing override file.

Note: The existing override (tracked by #13587) covers type/namespace fixes. This issue tracks the new destructor and API adaptation.

Current Fix
Updated the existing patched copy at vnext/ReactCommon/TEMP_UntilReactCommonUpdate/jsinspector-modern/NetworkIOAgent.cpp with the destructor and new API calls.

Link to the integration PR where the error originated

#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

Vorherrschende Sprache
C++
Sterne
17.4k
Forks
1.2k
Ø Merge
1 T. 20 Std.
Gemergte PRs (30 T.)
37

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus microsoft/react-native-windows

Alle Issues in microsoft/react-native-windows

Ähnliche Issues

Weitere Issues zu C++

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.