Unfork MapBuffer.cpp override added during 0.85 nightly integration
Los mantenedores suelen responder en 2 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 52/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- cpp, react-native
- Área
- build-system
Línea de trabajo
Comienza con vnext/ReactCommon/TEMP_UntilReactCommonUpdate/react/renderer/mapbuffer/MapBuffer.cpp y compara las líneas indicadas con los cambios de upstream en MapBuffer.cpp enlazados en el issue. Comprueba cómo Warnings.props trata C4018 y C4267 y, después, ejecuta la compilación relevante de Windows para verificar que ambas advertencias se han resuelto y que el override temporal se gestiona correctamente.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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
- Lenguaje dominante
- C++
- Estrellas
- 17.4k
- Forks
- 1.2k
- Merge medio
- 2 d 17 h
- PR fusionados (30 d)
- 13
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de microsoft/react-native-windows
-
Fabric text is drawn with ClearType onto transparent composition surfaces, fringing thin glyphsAbiertoNeeds: Triage :mag:
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
microsoft/react-native-windows#16340 ·
Los mantenedores suelen responder en 2 días
-
bug Needs: Triage :mag:
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
microsoft/react-native-windows#16321 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
Needs: Triage :mag:
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
microsoft/react-native-windows#16442 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
Fabric: activating the window doesn't announce the window or the focused control to a screen readerAbiertoNeeds: Triage :mag:
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
microsoft/react-native-windows#16435 ·
Los mantenedores suelen responder en 2 días
-
bug Needs: Triage :mag:
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
microsoft/react-native-windows#16410 ·
Los mantenedores suelen responder en 2 días
Todos los issues de microsoft/react-native-windows
Issues similares
-
libasr headers?Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
Los mantenedores suelen responder en 1 día
-
bug iOS 🍎 ui/ux
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
MerginMaps/mobile#4744 ·
Los mantenedores suelen responder en 1 día
-
Component: Ruby Type: bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
Los mantenedores suelen responder en 3 días
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
kokkos/kokkos-kernels#3328 ·
Los mantenedores suelen responder en 1 día