Andre Poenitz wrote:
Patch attached. Includes various cleanup, too. Will commit.

I see you didn't wait for an answer to your previous question (my answer particularly)... While I am happy that you work on this, I think you could have committed your changes one class at a time, couldn't you? I also think that you could have ditch th QController thing altogether using the method I outlined in my answer to your question.

By the way, what's the need to post such a big patch a minute before committing?

Anyway, it is done now so need to make a fuss of it.

Abdel.

Reply via email to