Opened 2 months ago

Last modified 3 weeks ago

#28821 new defect

Introduce timer_is_scheduled() method and replace padding_scheduled_at_us

Reported by: asn Owned by:
Priority: Low Milestone: Tor: unspecified
Component: Core Tor/Tor Version:
Severity: Normal Keywords: wtf-pad, tor-relay, tor-cell, padding, refactoring, 041-proposed
Cc: nickm Actual Points:
Parent ID: #28631 Points: 1
Reviewer: Sponsor: Sponsor2

Description

This came up during #28142 review: https://github.com/torproject/tor/pull/461#discussion_r231889687

When #28142 gets merged, we can do a small refactoring by removing padding_scheduled_at_us and replacing it with a new timer method that returns true if the timer is scheduled, and false if it's disabled.

Child Tickets

Change History (5)

comment:1 Changed 5 weeks ago by mikeperry

Keywords: 041-proposed added

comment:2 Changed 5 weeks ago by mikeperry

Points: 1

comment:3 Changed 5 weeks ago by mikeperry

Priority: MediumLow

comment:4 Changed 4 weeks ago by nickm

Sponsor: Sponsor2

comment:5 Changed 3 weeks ago by nickm

Parent ID: #28637#28631

Reparent children of #28637 into #28631

Note: See TracTickets for help on using tickets.