[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #30544 [Metrics/Library]: Using try-with-resources or close resource
#30544: Using try-with-resources or close resource
-----------------------------+--------------------------------
Reporter: fava | Owner: metrics-team
Type: defect | Status: needs_revision
Priority: Medium | Milestone:
Component: Metrics/Library | Version:
Severity: Major | Resolution:
Keywords: | Actual Points:
Parent ID: | Points:
Reviewer: | Sponsor:
-----------------------------+--------------------------------
Changes (by karsten):
* status: new => needs_revision
Comment:
This took much longer than it should have, sorry for that. Trying to get
faster with future reviews!
Here's what I found:
- There are some minor whitespace issues. Please run `ant checks` before
submitting a patch.
- Those TODO comments look reasonable, but let's maybe try to just fix
them now, each of them in a separate commit. Otherwise we'll just collect
more things to do in the sources.
- Our commit messages typically start with a roughly 50 chars long
summary line, followed by two newlines, followed by text wrapped to 70
chars. Can you try to rephrase your commit messages accordingly?
- Can you rebase your edited commits to master?
- When you post a patch or branch, be sure to change status to
needs_review. To be clear, this is not the main reason for us not
reviewing sooner, but it could be a possible reason for future delays.
Thanks again!
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/30544#comment:5>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs