Skip to content

Reduce name compression overhead and complexity#978

Merged
bdraco merged 9 commits into
python-zeroconf:masterfrom
bdraco:name_compression
Sep 16, 2021
Merged

Reduce name compression overhead and complexity#978
bdraco merged 9 commits into
python-zeroconf:masterfrom
bdraco:name_compression

Conversation

@bdraco
Copy link
Copy Markdown
Member

@bdraco bdraco commented Sep 16, 2021

Reduces the algorithmic complexity of write_name

Before

Screen Shot 2021-09-15 at 5 03 26 PM

After

Screen Shot 2021-09-15 at 5 03 31 PM

@bdraco bdraco changed the title Reduce name compression overhead Reduce name compression overhead and complexity Sep 16, 2021
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Sep 16, 2021

Codecov Report

Merging #978 (0b6928a) into master (769b397) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #978   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           21        21           
  Lines         2489      2488    -1     
  Branches       411       408    -3     
=========================================
- Hits          2489      2488    -1     
Impacted Files Coverage Δ
zeroconf/_protocol.py 100.00% <100.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 769b397...0b6928a. Read the comment docs.

@bdraco bdraco marked this pull request as ready for review September 16, 2021 20:25
@bdraco bdraco merged commit f1d6fc3 into python-zeroconf:master Sep 16, 2021
@bdraco bdraco deleted the name_compression branch September 16, 2021 20:25
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.

2 participants