Skip to content

fix: ensure IPv6 scoped address construction uses the string cache#1336

Merged
bdraco merged 1 commit into
masterfrom
fix_scoped_addreses_not_using_string_cache
Dec 14, 2023
Merged

fix: ensure IPv6 scoped address construction uses the string cache#1336
bdraco merged 1 commit into
masterfrom
fix_scoped_addreses_not_using_string_cache

Conversation

@bdraco

@bdraco bdraco commented Dec 14, 2023

Copy link
Copy Markdown
Member

Instead of using the __str__ cache on the object it was calling __format__ which isn't optimized

Screenshot 2023-12-14 at 11 04 06 AM

@codecov

codecov Bot commented Dec 14, 2023

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (9b595a1) 99.77% compared to head (6e92caa) 99.77%.
Report is 3 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1336   +/-   ##
=======================================
  Coverage   99.77%   99.77%           
=======================================
  Files          30       30           
  Lines        3164     3164           
  Branches      528      528           
=======================================
  Hits         3157     3157           
  Misses          5        5           
  Partials        2        2           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@bdraco bdraco marked this pull request as ready for review December 14, 2023 21:06
@bdraco bdraco merged commit f78a196 into master Dec 14, 2023
@bdraco bdraco deleted the fix_scoped_addreses_not_using_string_cache branch December 14, 2023 21:06
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.

1 participant