[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