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.

Sign up for GitHub

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

Jump to bottom

Recursive inliner adjustments/fixes + comments #2

Merged
merged 6 commits into from
Aug 8, 2023
Merged

Recursive inliner adjustments/fixes + comments #2

merged 6 commits into from
Aug 8, 2023

Commits on Aug 7, 2023

  1. Add more comments to the RecursiveInliner

    @MaartenS11
    MaartenS11 committed Aug 7, 2023
    Configuration menu
    Copy the full SHA
    33aa83d View commit details
    Browse the repository at this point in the history

  2. Only stop inlining if we actually have the current lambda in one of t… 

    …he arguments
    @MaartenS11
    MaartenS11 committed Aug 7, 2023
    Configuration menu
    Copy the full SHA
    3d3bf8f View commit details
    Browse the repository at this point in the history

Commits on Aug 8, 2023

  1. Use argument index from LambdaUsageFinder instead of searching in the… 

    … descriptor

This is better because it fixes the case where a lambda is attempted to be inlined into a method that takes object as argument which is then later casted back into a lambda.
    @MaartenS11
    MaartenS11 committed Aug 8, 2023
    Configuration menu
    Copy the full SHA
    813be34 View commit details
    Browse the repository at this point in the history

  2. RecursiveInliner should also use the argument index from the lambda u… 

    …sage finder
    @MaartenS11
    MaartenS11 committed Aug 8, 2023
    Configuration menu
    Copy the full SHA
    f155107 View commit details
    Browse the repository at this point in the history

  3. Remove findFirstLambdaParameter method since it is no longer needed

    @MaartenS11
    MaartenS11 committed Aug 8, 2023
    Configuration menu
    Copy the full SHA
    7d82d2d View commit details
    Browse the repository at this point in the history

  4. Merge branch 'lambda-inlining' of github.com:MaartenS11/proguard into… 

    … recursive-inliner-adjustments
    @MaartenS11
    MaartenS11 committed Aug 8, 2023
    Configuration menu
    Copy the full SHA
    5cc6748 View commit details
    Browse the repository at this point in the history