Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(extensions): Fix off-by-one in removeInstructions #167

Merged
merged 2 commits into from
May 6, 2023

Conversation

rstular
Copy link
Contributor

@rstular rstular commented May 4, 2023

No description provided.

@oSumAtrIX oSumAtrIX deleted the branch ReVanced:dev May 5, 2023 23:05
@oSumAtrIX oSumAtrIX closed this May 5, 2023
@Sculas
Copy link
Contributor

Sculas commented May 5, 2023

@oSumAtrIX Should be reopened as well.

@oSumAtrIX oSumAtrIX reopened this May 6, 2023
Co-authored-by: oSumAtrIX <johan.melkonyan1@web.de>
@rstular rstular requested a review from oSumAtrIX May 6, 2023 11:23
@oSumAtrIX oSumAtrIX merged commit 6f6434f into ReVanced:dev May 6, 2023
@rstular rstular deleted the 166-fix-removeinstructions branch May 6, 2023 18:13
oSumAtrIX pushed a commit that referenced this pull request May 7, 2023
…nstructions` (#167)

Co-authored-by: oSumAtrIX <johan.melkonyan1@web.de>

BREAKING-CHANGE: This alters the behaviour of the function. Instead of removing `count + 1` instructions, this now removes `count` instructions.
revanced-bot pushed a commit that referenced this pull request May 7, 2023
## [7.1.1-dev.1](v7.1.0...v7.1.1-dev.1) (2023-05-07)

### Bug Fixes

* remove `count` instead of `count + 1` instructions with `removeInstructions` ([#167](#167)) ([98f8eed](98f8eed))
revanced-bot pushed a commit that referenced this pull request May 7, 2023
## [7.1.1](v7.1.0...v7.1.1) (2023-05-07)

### Bug Fixes

* remove `count` instead of `count + 1` instructions with `removeInstructions` ([#167](#167)) ([98f8eed](98f8eed))
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.

3 participants