Skip to content

Fix incompatible JSONTokenerWrapper for Android#942

Merged
jotapeg merged 3 commits into
masterfrom
fix/android-json-tokener
Feb 19, 2025
Merged

Fix incompatible JSONTokenerWrapper for Android#942
jotapeg merged 3 commits into
masterfrom
fix/android-json-tokener

Conversation

@jotapeg

@jotapeg jotapeg commented Feb 19, 2025

Copy link
Copy Markdown
Contributor

Issue #203424.
Broken #938.

@jotapeg jotapeg requested review from fpanizza and iroqueta February 19, 2025 01:59
@genexusbot

genexusbot commented Feb 19, 2025

Copy link
Copy Markdown
Collaborator
Cherry pick to beta success

fpanizza
fpanizza approved these changes Feb 19, 2025
View reviewed changes

@fpanizza fpanizza left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jotapeg jotapeg merged commit 1f810bb into master Feb 19, 2025
@jotapeg jotapeg deleted the fix/android-json-tokener branch February 19, 2025 14:24
jotapeg added a commit that referenced this pull request Feb 19, 2025
@jotapeg
Fix incompatible JSONTokenerWrapper for Android (#942)
c99ea79 
* Revert "Fix StackOverflowException when instantiating JSONObject on Android (#938)"

This reverts commit c0f92a2.

* Add specific JSONTokenerWrapper for Android

---------

Co-authored-by: Juan Pablo González <juan.gonzalezabella@globant.com>
(cherry picked from commit 1f810bb)
@fpanizza fpanizza mentioned this pull request Mar 28, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fpanizza fpanizza fpanizza approved these changes

@iroqueta iroqueta iroqueta approved these changes

Assignees

No one assigned

Labels

bot closed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@jotapeg @genexusbot @fpanizza @iroqueta