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

[tor-bugs] #6400 [Tor Client]: Eliminate long SMARTLIST_FOREACH statements

#6400: Eliminate long SMARTLIST_FOREACH statements
-------------------------+--------------------------------------------------
 Reporter:  nickm        |          Owner:                    
     Type:  enhancement  |         Status:  new               
 Priority:  normal       |      Milestone:  Tor: 0.2.4.x-final
Component:  Tor Client   |        Version:                    
 Keywords:               |         Parent:                    
   Points:               |   Actualpoints:                    
-------------------------+--------------------------------------------------
 I introduced SMARTLIST_FOREACH_{BEGIN,END} a while back so that we
 wouldn't have the attendant problems from having large SMARTLIST_FOREACH()
 blocks.  But there are still some of those lingering in the codebase.

 I propose a 10-line maximum for a SMARTLIST_FOREACH statement, and that we
 prohibit them from nesting.

 I was going to do this on 0.2.3, but I realize that if somebody else
 proposed doing this on 0.2.3 I would say "no", so this is 0.2.4 material.

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