Opened 3 years ago

Closed 3 years ago

#24030 closed defect (implemented)

Wrap types in

Reported by: nickm Owned by:
Priority: Medium Milestone: Tor: 0.3.3.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: rust
Cc: chelseakomlo, coreyf+tor@… Actual Points:
Parent ID: Points:
Reviewer: Sponsor:


Our rust protover implementation throws around HashSet and HashMap with wild abandon. We should probably wrap those types in struct declarations, to make the intent more clear.

Child Tickets

Change History (9)

comment:1 Changed 3 years ago by frewsxcv

I haven't contributed to Tor before, but this looks like a ticket I could tackle! I'll take a stab a it…

comment:2 Changed 3 years ago by chelseakomlo

Hi, thanks for taking this on! This will be helpful. Let us know what questions you have or how we can help.

comment:3 Changed 3 years ago by frewsxcv

Cc: coreyf+tor@… added

comment:4 Changed 3 years ago by frewsxcv

Thanks for the offer for help! I'll have a branch for this ready soon.

I found some places where heap allocations happen unnecessarily and opened a separate ticket addressing it. Should I assign this ticket to someone?

Also, I'm not very familiar with the Trac workflow, but if it's too time consuming to submit/review/merge tickets with these small changesets, let me know if it's preferable to bundle them together with other commits.

comment:5 Changed 3 years ago by nickm

frewsxcv, any progress here?

comment:6 Changed 3 years ago by frewsxcv

Yep! I'll try to open a new issue w/ patch for this in the next day or two. Thanks for your patience! πŸ™

comment:7 Changed 3 years ago by gk

#25067 it is.

comment:8 Changed 3 years ago by teor

Status: new β†’ needs_review

comment:9 Changed 3 years ago by nickm

Resolution: β†’ implemented
Status: needs_review β†’ closed

Merged that; thanks, frewsxcv!

Note: See TracTickets for help on using tickets.