Opened 2 weeks ago

Last modified 3 days ago

#30406 needs_review defect

Refactor bandwidth file header constants

Reported by: juga Owned by: juga
Priority: Medium Milestone: sbws: 1.2.x-final
Component: Core Tor/sbws Version:
Severity: Normal Keywords:
Cc: juga Actual Points:
Parent ID: #30255 Points: 1
Reviewer: asn Sponsor:

Description

As commented in https://trac.torproject.org/projects/tor/ticket/30196#comment:9.
It would need only 2 changes, but since we have #29057, refactor to start replacing as much as possible sbws code with stem's one so that we don't need to maintain 2 versions.

Child Tickets

Change History (5)

comment:1 Changed 2 weeks ago by juga

Status: assignedneeds_review

comment:2 Changed 2 weeks ago by asn

Reviewer: asn

comment:3 Changed 12 days ago by asn

Status: needs_reviewneeds_revision

Did a review. Might need revisions or just clarifications. Putting it in needs_revision anyway.

comment:4 Changed 11 days ago by juga

I replied to the questions. Not changing to needs_review because not doing any commit yet until knowing your opinion on my answers.

comment:5 Changed 3 days ago by juga

Status: needs_revisionneeds_review

Ok, i made the changes i commented in the PR.

Note: See TracTickets for help on using tickets.