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

surface (fix): Reduce generated code size for MethodSurface in Scala 3 #3146

Merged
merged 6 commits into from
Aug 20, 2023

Conversation

xerial
Copy link
Member

@xerial xerial commented Aug 20, 2023

  • surface (fix): Do not use internal cache to save the byte code size
  • Cache Surface only for LazySurface

This PR addresses a huge byte code issue, failed with: [error] Generated bytecode for method (extending RxRouterProvider) is too large. Size: 159519 bytes. Limit is 64KB message

@github-actions github-actions bot added the bug label Aug 20, 2023
@xerial xerial force-pushed the reduce-method-surface-bytecode branch from 197742c to f6dd617 Compare August 20, 2023 00:48
@xerial xerial changed the title surface (fix): Reduce generated code for MethodSurface in Scala 3 surface (fix): Reduce generated code size for MethodSurface in Scala 3 Aug 20, 2023
@codecov
Copy link

codecov bot commented Aug 20, 2023

Codecov Report

Merging #3146 (84ee17b) into main (85035e0) will increase coverage by 0.00%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #3146   +/-   ##
=======================================
  Coverage   82.89%   82.90%           
=======================================
  Files         350      350           
  Lines       14740    14740           
  Branches     2443     2443           
=======================================
+ Hits        12219    12220    +1     
+ Misses       2521     2520    -1     

see 3 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

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

@xerial xerial force-pushed the reduce-method-surface-bytecode branch from f6dd617 to 84ee17b Compare August 20, 2023 00:55
@xerial xerial merged commit f721dcb into main Aug 20, 2023
17 checks passed
@xerial xerial deleted the reduce-method-surface-bytecode branch August 20, 2023 01:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant