On 19.08.2019 03:25, Chao Gao wrote: > +static enum microcode_match_result compare_patch( > + const struct microcode_patch *new, const struct microcode_patch *old) > +{ > + return (new->mc_intel->hdr.rev > old->mc_intel->hdr.rev) ? NEW_UCODE : > + OLD_UCODE;
There's one too many blank after the ? here. Also we commonly align the : under the ? in cases like this one. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel