On 03/07/2013 03:28 PM, Patrik Flykt wrote:
On Thu, 2013-03-07 at 14:35 +0100, Daniel Wagner wrote:
From: Daniel Wagner <[email protected]>

There are a few useless checks in there. prepare_rule_inclusion()
already does those tests. Also add some comments what is happening
in here.

Are we achieving anything significantly better with this patch?

Yes, I think the code is more readable and we do not call update_hooks() in all cases.

cheers,
daniel
_______________________________________________
connman mailing list
[email protected]
http://lists.connman.net/listinfo/connman

Reply via email to