[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #21662 [Core Tor/Tor]: prop278: Add support for LZMA2 and/or Zstandard
#21662: prop278: Add support for LZMA2 and/or Zstandard
----------------------------------------+----------------------------------
Reporter: ahf | Owner: ahf
Type: task | Status: needs_review
Priority: Medium | Milestone: Tor:
| 0.3.1.x-final
Component: Core Tor/Tor | Version:
Severity: Normal | Resolution:
Keywords: TorCoreTeam201703, prop278 | Actual Points:
Parent ID: | Points: 2
Reviewer: | Sponsor: Sponsor4
----------------------------------------+----------------------------------
Changes (by ahf):
* status: needs_revision => needs_review
Comment:
I believe I've been over everything you have commented on in the reviews
now. We've postponed merging the test functions into a simplified function
that tests all backends and we have postponed doing coverage "hints" in
the code.
I've added a clean-up patch of some of the Zstandard code with regards to
flushing the internal buffer in a more eager manner.
Let me know if this looks acceptable for merge and I'll happily do the
squashing as well (whatever you find easiest!)
Note that this is build on the comments already made in both
https://gitlab.com/ahf/tor/merge_requests/1 and
https://gitlab.com/ahf/tor/merge_requests/2 - Gitlab seems to hide that a
bit.
Once this code lands we can close both #21662, #21663 and #21664.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/21662#comment:7>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs