[tor-bugs] #30091 [Core Tor/Tor]: Unify parsing code for control.c
    Tor Bug Tracker & Wiki 
    blackhole at torproject.org
       
    Tue Apr 23 23:24:58 UTC 2019
    
    
  
#30091: Unify parsing code for control.c
--------------------------+------------------------------------
 Reporter:  nickm         |          Owner:  nickm
     Type:  enhancement   |         Status:  needs_review
 Priority:  Medium        |      Milestone:  Tor: 0.4.1.x-final
Component:  Core Tor/Tor  |        Version:
 Severity:  Normal        |     Resolution:
 Keywords:                |  Actual Points:  3
Parent ID:  #29210        |         Points:  3
 Reviewer:  catalyst      |        Sponsor:  Sponsor31-can
--------------------------+------------------------------------
Comment (by catalyst):
 Looking at
 {{{
 bb2062dd76 kvline: handle empty alues as well as empty keys
 }}}
 through
 {{{
 5d3dcd6fc9 Update more controller commands, now that we have kvline
 support
 }}}
 Overall this looks good, and deletes a lot of repetitive code.
 There's a typo in the bb2062dd76 summary.
 See technical debt comment in the pull request for
 `control_cmd_parse_args()`, and maybe a few more minor comments.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/30091#comment:7>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
    
    
More information about the tor-bugs
mailing list