Skip to content

fix(Strava): Fix Overwrite media upload parameters patch (alt)#6492

Draft
xehpuk wants to merge 1 commit intoReVanced:devfrom
xehpuk:fix/strava/overwrite-media-upload-parameters-alt
Draft

fix(Strava): Fix Overwrite media upload parameters patch (alt)#6492
xehpuk wants to merge 1 commit intoReVanced:devfrom
xehpuk:fix/strava/overwrite-media-upload-parameters-alt

Conversation

@xehpuk
Copy link
Contributor

@xehpuk xehpuk commented Jan 16, 2026

A "less invasive" alternative to #6491. Adds a helper method inside the patch.

@xehpuk xehpuk changed the title fix(Strava): Fix Overwrite media upload parameters patch fix(Strava): Fix Overwrite media upload parameters patch (alt) Jan 16, 2026
@oSumAtrIX
Copy link
Member

Whats meant with "less invasive"?

@xehpuk
Copy link
Contributor Author

xehpuk commented Jan 19, 2026

It's just a minimal change to BytecodeUtils:

  • Fixes the calls to Double.valueOf(double) and Long.valueOf(long).
  • Throws if the method has less than 2 registers.

While the other PR needs at least Kotlin v2.2 for the context parameters.

@xehpuk
Copy link
Contributor Author

xehpuk commented Jan 19, 2026

I guess the helper method is unnecessary. Could recreate the original method instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants