<div class="gmail_quote">On Thu, Sep 13, 2012 at 10:51 PM, Damian Johnson <span dir="ltr"><<a href="mailto:atagar@torproject.org" target="_blank">atagar@torproject.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

* Split low and high level controller methods<br>
<br>
The Controller class is growing to be pretty bulky, and its methods<br>
can generally be split into two categories...<br>
<br>
- Low level methods that simply mirror what the control-spec provides.<br>
- Higher level methods (like the get_relay() and get_all_relays()<br>
above) which are more user friendly and build on what the control spec<br>
provides.<br>
<br>
Ravi: Do you have any thoughts on this? Do you think this would be a<br>
good way to break up the controller, or would this just make it more<br>
confusing? Alternatively we could wait until we try using stem in arm<br>
to see if this makes sense or not...<br></blockquote></div><div><br></div>Break up - add another class between BaseController and Controller?<div>I would prefer leaving it as it as it is for now.<br clear="all"><div><br>

</div>-- <br>neena<br>
</div>