On 17-01-15 10:01 AM, Jiri Pirko wrote:
Sun, Jan 15, 2017 at 03:01:19PM CET, j...@mojatatu.com wrote:

+cls_set_class(struct tcf_proto *tp, unsigned long *clp,

??

        unsigned long cl)
{
        unsigned long old_cl;
-       
+

??

        tcf_tree_lock(tp);
        old_cl = __cls_set_class(clp, cl);
        tcf_tree_unlock(tp);
-
+

??

        return old_cl;
}

@@ -237,7 +237,7 @@ static inline int tcf_em_early_end(struct tcf_ematch *em, 
int result)

        return 0;
}
-       
+

??


All these things with "??" were annoying whitespace and stylistic
things my fingers couldnt resist. It is legit to include such changes
in a patch when touching the code (as long as not a bug fix).

cheers,
jamal

Reply via email to