[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

Re: [tor-bugs] #13827 [Core Tor/Tor]: Cell handling code duplication in channel.c

#13827: Cell handling code duplication in channel.c
-----------------------------------------------+---------------------------
 Reporter:  rl1987                             |          Owner:  pingl
     Type:  defect                             |         Status:
                                               |  needs_review
 Priority:  Medium                             |      Milestone:  Tor:
                                               |  0.2.9.x-final
Component:  Core Tor/Tor                       |        Version:
 Severity:  Normal                             |     Resolution:
 Keywords:  refactoring, easy, review-group-9  |  Actual Points:
Parent ID:                                     |         Points:
                                               |  small/medium
 Reviewer:                                     |        Sponsor:
-----------------------------------------------+---------------------------

Comment (by asn):

 Hey pingl, you might consider uploading your patches on a git repo
 somewhere, and then post a link to the branch here. As an example, here is
 another ticket where this is done:
 https://trac.torproject.org/projects/tor/ticket/19487#comment:3

 This will make review slightly easier and it's a better way to contribute
 code to Tor.

 Thanks :)

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/13827#comment:10>
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