Comment 21 for bug 1086999

Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

> > * reset()/setTarget() ..
> For me it was not the question of a simple patch but of a simple code.
> For me it is cleaner to have one calculating function with all relevant
> data as parameter instead of dealing with member variables.

Well, you just moved the member variable from one class to another :). I think of a 'set point' as being a property of a PID controller which is why I put it in there.