Skip to content

Migrate YogaNodeJNIBase.java to Kotlin (#1939)#1939

Closed
cortinico wants to merge 1 commit into
facebook:mainfrom
cortinico:export-D104666335
Closed

Migrate YogaNodeJNIBase.java to Kotlin (#1939)#1939
cortinico wants to merge 1 commit into
facebook:mainfrom
cortinico:export-D104666335

Conversation

@cortinico
Copy link
Copy Markdown
Contributor

@cortinico cortinico commented May 12, 2026

Summary:

X-link: facebook/react-native#56780

Migrate YogaNodeJNIBase.java to YogaNodeJNIBase.kt, the last hand-written Java source file in the Yoga core library. This required converting all function overrides to Kotlin property overrides where the abstract YogaNode class declares them as val/var properties. Also changed nativePointer visibility in YogaConfigJNIBase from protected to internal to maintain same-module access that Java protected provided via package access.

Changelog:
[Internal] -

Differential Revision: D104666335

@vercel
Copy link
Copy Markdown

vercel Bot commented May 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
yoga-website Ready Ready Preview, Comment May 13, 2026 1:58pm

Request Review

@meta-codesync
Copy link
Copy Markdown

meta-codesync Bot commented May 12, 2026

@cortinico has exported this pull request. If you are a Meta employee, you can view the originating Diff in D104666335.

@cortinico cortinico force-pushed the export-D104666335 branch from 9b61740 to 91e3b84 Compare May 12, 2026 12:55
cortinico added a commit to cortinico/react-native that referenced this pull request May 12, 2026
Summary:
X-link: facebook/yoga#1939

Pull Request resolved: facebook#56780

Migrate YogaNodeJNIBase.java to YogaNodeJNIBase.kt, the last hand-written Java source file in the Yoga core library. This required converting all function overrides to Kotlin property overrides where the abstract YogaNode class declares them as val/var properties. Also changed nativePointer visibility in YogaConfigJNIBase from protected to internal to maintain same-module access that Java protected provided via package access.

Changelog:
[Internal] -

Differential Revision: D104666335
@meta-codesync meta-codesync Bot changed the title Migrate YogaNodeJNIBase.java to Kotlin Migrate YogaNodeJNIBase.java to Kotlin (#1939) May 13, 2026
@cortinico cortinico force-pushed the export-D104666335 branch from 91e3b84 to 4342881 Compare May 13, 2026 09:50
cortinico added a commit to cortinico/yoga that referenced this pull request May 13, 2026
Summary:
Pull Request resolved: facebook#1939

X-link: facebook/react-native#56780

Migrate YogaNodeJNIBase.java to YogaNodeJNIBase.kt, the last hand-written Java source file in the Yoga core library. This required converting all function overrides to Kotlin property overrides where the abstract YogaNode class declares them as val/var properties. Also changed nativePointer visibility in YogaConfigJNIBase from protected to internal to maintain same-module access that Java protected provided via package access.

Changelog:
[Internal] -

Differential Revision: D104666335
cortinico added a commit to cortinico/react-native that referenced this pull request May 13, 2026
Summary:
X-link: facebook/yoga#1939

Pull Request resolved: facebook#56780

Migrate YogaNodeJNIBase.java to YogaNodeJNIBase.kt, the last hand-written Java source file in the Yoga core library. This required converting all function overrides to Kotlin property overrides where the abstract YogaNode class declares them as val/var properties. Also changed nativePointer visibility in YogaConfigJNIBase from protected to internal to maintain same-module access that Java protected provided via package access.

Changelog:
[Internal] -

Differential Revision: D104666335
cortinico added a commit to cortinico/react-native that referenced this pull request May 13, 2026
Summary:
X-link: facebook/yoga#1939

Pull Request resolved: facebook#56780

Migrate YogaNodeJNIBase.java to YogaNodeJNIBase.kt, the last hand-written Java source file in the Yoga core library. This required converting all function overrides to Kotlin property overrides where the abstract YogaNode class declares them as val/var properties. Also changed nativePointer visibility in YogaConfigJNIBase from protected to internal to maintain same-module access that Java protected provided via package access.

Changelog:
[Internal] -

Differential Revision: D104666335
cortinico added a commit to cortinico/yoga that referenced this pull request May 13, 2026
Summary:
Pull Request resolved: facebook#1939

X-link: facebook/react-native#56780

Migrate YogaNodeJNIBase.java to YogaNodeJNIBase.kt, the last hand-written Java source file in the Yoga core library. This required converting all function overrides to Kotlin property overrides where the abstract YogaNode class declares them as val/var properties. Also changed nativePointer visibility in YogaConfigJNIBase from protected to internal to maintain same-module access that Java protected provided via package access.

Changelog:
[Internal] -

Differential Revision: D104666335
@cortinico cortinico force-pushed the export-D104666335 branch from 4342881 to 3907ff5 Compare May 13, 2026 12:31
cortinico added a commit to cortinico/react-native that referenced this pull request May 13, 2026
Summary:
X-link: facebook/yoga#1939

Pull Request resolved: facebook#56780

Migrate YogaNodeJNIBase.java to YogaNodeJNIBase.kt, the last hand-written Java source file in the Yoga core library. This required converting all function overrides to Kotlin property overrides where the abstract YogaNode class declares them as val/var properties. Also changed nativePointer visibility in YogaConfigJNIBase from protected to internal to maintain same-module access that Java protected provided via package access.

Changelog:
[Internal] -

Differential Revision: D104666335
cortinico added a commit to cortinico/yoga that referenced this pull request May 13, 2026
Summary:
Pull Request resolved: facebook#1939

X-link: facebook/react-native#56780

Migrate YogaNodeJNIBase.java to YogaNodeJNIBase.kt, the last hand-written Java source file in the Yoga core library. This required converting all function overrides to Kotlin property overrides where the abstract YogaNode class declares them as val/var properties. Also changed nativePointer visibility in YogaConfigJNIBase from protected to internal to maintain same-module access that Java protected provided via package access.

Changelog:
[Internal] -

Differential Revision: D104666335
@cortinico cortinico force-pushed the export-D104666335 branch from 3907ff5 to 559dab5 Compare May 13, 2026 13:51
Summary:
Pull Request resolved: facebook#1939

X-link: facebook/react-native#56780

Migrate YogaNodeJNIBase.java to YogaNodeJNIBase.kt, the last hand-written Java source file in the Yoga core library. This required converting all function overrides to Kotlin property overrides where the abstract YogaNode class declares them as val/var properties. Also changed nativePointer visibility in YogaConfigJNIBase from protected to internal to maintain same-module access that Java protected provided via package access.

Changelog:
[Internal] -

Differential Revision: D104666335
@cortinico cortinico force-pushed the export-D104666335 branch from 559dab5 to bac628f Compare May 13, 2026 13:56
cortinico added a commit to cortinico/react-native that referenced this pull request May 13, 2026
Summary:
X-link: facebook/yoga#1939

Pull Request resolved: facebook#56780

Migrate YogaNodeJNIBase.java to YogaNodeJNIBase.kt, the last hand-written Java source file in the Yoga core library. This required converting all function overrides to Kotlin property overrides where the abstract YogaNode class declares them as val/var properties. Also changed nativePointer visibility in YogaConfigJNIBase from protected to internal to maintain same-module access that Java protected provided via package access.

Changelog:
[Internal] -

Differential Revision: D104666335
@meta-codesync meta-codesync Bot closed this in e114242 May 15, 2026
@meta-codesync
Copy link
Copy Markdown

meta-codesync Bot commented May 15, 2026

This pull request has been merged in e114242.

meta-codesync Bot pushed a commit to facebook/react-native that referenced this pull request May 15, 2026
Summary:
X-link: facebook/yoga#1939

Pull Request resolved: #56780

Migrate YogaNodeJNIBase.java to YogaNodeJNIBase.kt, the last hand-written Java source file in the Yoga core library. This required converting all function overrides to Kotlin property overrides where the abstract YogaNode class declares them as val/var properties. Also changed nativePointer visibility in YogaConfigJNIBase from protected to internal to maintain same-module access that Java protected provided via package access.

Changelog:
[Internal] -

Reviewed By: alanleedev

Differential Revision: D104666335

fbshipit-source-id: c60285453e8a6ec834924be461da3c669d9a7be4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant