>>> On 10.12.18 at 15:00, <andrew.coop...@citrix.com> wrote: > On 07/12/2018 11:18, Jan Beulich wrote: >> While actually benign (operands are either register or memory ones >> anyway), I think it is better to use != instead of == for such checks. >> >> Signed-off-by: Jan Beulich <jbeul...@suse.com> > > I don't see the point of making this change. Code is easier to follow > when there are fewer negations.
Is != a negation? I ask because in general I agree with that position of yours. The idea here is that besides OP_REG and OP_MEM we also have OP_IMM and OP_NONE. By carefully using != and == I'm trying to make sure (and not the least document) that (in the cases here) we don't let things go through in case the value (mistakenly) is one of the "impossible" (in these contexts) two. > But whatever - Acked-by: Andrew Cooper <andrew.coop...@citrix.com> Thanks. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
