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: Only consider "near" lines on il2cpp lookup #617

Merged
merged 1 commit into from
Jul 7, 2022
Merged

Conversation

Swatinem
Copy link
Member

@Swatinem Swatinem commented Jul 6, 2022

Previously, it would return the previous mapping on lookup, even if that is hundreds of lines away.

@Swatinem Swatinem requested a review from a team July 6, 2022 16:18
@codecov-commenter
Copy link

Codecov Report

Merging #617 (43012ca) into master (e902660) will increase coverage by 0.27%.
The diff coverage is 97.56%.

❗ Current head 43012ca differs from pull request most recent head d21135c. Consider uploading reports for the commit d21135c to get more accurate results

@@            Coverage Diff             @@
##           master     #617      +/-   ##
==========================================
+ Coverage   68.94%   69.22%   +0.27%     
==========================================
  Files          84       84              
  Lines       16827    16866      +39     
==========================================
+ Hits        11601    11675      +74     
+ Misses       5226     5191      -35     

@Swatinem Swatinem merged commit 7d4e25a into master Jul 7, 2022
@Swatinem Swatinem deleted the fix/il2cpp-near branch July 7, 2022 10:34
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.

None yet

3 participants