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

Use ASM9 Opcodes, add missing non-void overrides #67

Merged
merged 1 commit into from
May 13, 2021

Conversation

jpenilla
Copy link
Contributor

This fixes support for Records; would previously throw java.lang.UnsupportedOperationException: Records requires ASM8

This fixes support for Records; would previously throw `java.lang.UnsupportedOperationException: Records requires ASM8`
@codecov-commenter
Copy link

Codecov Report

Merging #67 (1671623) into master (d5ef1e6) will decrease coverage by 1.54%.
The diff coverage is 60.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #67      +/-   ##
============================================
- Coverage     85.85%   84.31%   -1.55%     
  Complexity       63       63              
============================================
  Files             6        6              
  Lines           198      204       +6     
  Branches         22       22              
============================================
+ Hits            170      172       +2     
- Misses           21       25       +4     
  Partials          7        7              
Impacted Files Coverage Δ Complexity Δ
...afer/jdependency/asm/DependenciesClassAdapter.java 67.64% <60.00%> (-7.36%) 3.00 <0.00> (ø)

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 d5ef1e6...1671623. Read the comment docs.

@tcurdt tcurdt merged commit be6ec6d into tcurdt:master May 13, 2021
@tcurdt
Copy link
Owner

tcurdt commented May 13, 2021

Thanks for the PR!

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