[tor-bugs] #27629 [Core Tor/Tor]: add len argument to consensus parsing functions
Tor Bug Tracker & Wiki
blackhole at torproject.org
Tue Sep 11 03:08:13 UTC 2018
#27629: add len argument to consensus parsing functions
--------------------------+------------------------------------
Reporter: cyberpunks | Owner: (none)
Type: defect | Status: needs_review
Priority: Medium | Milestone: Tor: 0.3.5.x-final
Component: Core Tor/Tor | Version:
Severity: Normal | Resolution:
Keywords: | Actual Points:
Parent ID: #27244 | Points:
Reviewer: | Sponsor: Sponsor8
--------------------------+------------------------------------
Changes (by teor):
* status: new => needs_review
* sponsor: => Sponsor8
* milestone: Tor: 0.3.6.x-final => Tor: 0.3.5.x-final
Comment:
Thanks for this branch.
It looks good to me, but I'd like to make sure that CI passes, and that
someone who is more familiar with this code also does a review.
Please don't assign tickets to releases, we'll do that based on our
roadmap for the release.
(This task is on our roadmap as sponsored work, so we will prioritise
reviewing it before the feature freeze on Friday.)
Can you explain the fix in 5d27c458 in the commit message and changes
file?
Each fix and feature also gets its own ticket number.
We usually open a child ticket for each unrelated fix in a branch.
Or is this change just refactoring?
For refactoring, the changes file should have the heading:
Code simplification and refactoring
https://github.com/torproject/tor/blob/master/scripts/maint/lintChanges.py#L22
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/27629#comment:2>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
More information about the tor-bugs
mailing list