Opened 4 months ago

Closed 4 months ago

#30806 closed enhancement (fixed)

Make a subsystem for evloop

Reported by: nickm Owned by: nickm
Priority: Medium Milestone: Tor: 0.4.2.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: asn-merge
Cc: Actual Points: .1
Parent ID: Points:
Reviewer: dgoulet Sponsor: Sponsor31-can

Description

The evloop module should use the subsystem mechanism.

Child Tickets

Change History (7)

comment:1 Changed 4 months ago by nickm

Actual Points: .1
Milestone: Tor: 0.4.2.x-final
Status: assignedneeds_review

See branch libevent_subsystem with PR at https://github.com/torproject/tor/pull/1089

comment:2 Changed 4 months ago by gaba

wow! I saw the .1 and was quite suspicious but it was really .1 estimation :)

comment:3 Changed 4 months ago by dgoulet

Reviewer: dgoulet

comment:4 Changed 4 months ago by dgoulet

Status: needs_reviewneeds_information

One small comment. If that was on purpose, just go to merge_ready!

comment:5 Changed 4 months ago by nickm

Status: needs_informationneeds_review

Added a fixup commit. Better?

comment:6 Changed 4 months ago by dgoulet

Keywords: asn-merge added
Status: needs_reviewmerge_ready

comment:7 Changed 4 months ago by asn

Resolution: fixed
Status: merge_readyclosed

Merged!

Note: See TracTickets for help on using tickets.