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

Add the catchall handler to the end of handler list. #729

Merged
merged 1 commit into from Sep 1, 2019

Conversation

amimo
Copy link

@amimo amimo commented Aug 30, 2019

The exception handler list is ordered, if the catchall handler exists, it should be the last one.

The exception handler list is ordered, if the catchall handler exists, it should be the last one.
@codecov-io
Copy link

Codecov Report

Merging #729 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #729   +/-   ##
=======================================
  Coverage   72.72%   72.72%           
=======================================
  Files          51       51           
  Lines       16145    16145           
=======================================
  Hits        11742    11742           
  Misses       4403     4403
Impacted Files Coverage Δ
androguard/core/bytecodes/dvm.py 63.26% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update af32a37...75d8666. Read the comment docs.

@reox reox merged commit acee702 into androguard:master Sep 1, 2019
@reox
Copy link
Member

reox commented Sep 1, 2019

thanks for the fix!

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