Skip to content

Refactor defaultvaluebehavior to avoid unnecessary allocations#223

Merged
MatthieuDartiailh merged 1 commit intonucleic:mainfrom
frmdstryr:optimize-defaultvaluebehavior
Jan 20, 2025
Merged

Refactor defaultvaluebehavior to avoid unnecessary allocations#223
MatthieuDartiailh merged 1 commit intonucleic:mainfrom
frmdstryr:optimize-defaultvaluebehavior

Conversation

@frmdstryr
Copy link
Contributor

All of the tuple creations can be eliminated by using either vectorcall or the builtin call functions.

@sccolbert
Copy link
Member

Nice!

@MatthieuDartiailh MatthieuDartiailh force-pushed the optimize-defaultvaluebehavior branch from 043a615 to ceaab80 Compare January 20, 2025 09:47
@codecov
Copy link

codecov bot commented Jan 20, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.67%. Comparing base (23aaa52) to head (ceaab80).
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #223   +/-   ##
=======================================
  Coverage   97.67%   97.67%           
=======================================
  Files          24       24           
  Lines        1074     1074           
  Branches      162      162           
=======================================
  Hits         1049     1049           
  Misses         12       12           
  Partials       13       13           

@MatthieuDartiailh MatthieuDartiailh merged commit 1dc853d into nucleic:main Jan 20, 2025
18 checks passed
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