> > * 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.
> > * 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.