>>> On 29.05.19 at 06:43, <andrew.coop...@citrix.com> wrote: > On 29/05/2019 05:23, Andrew Cooper wrote: >> Drop introduced trailing whitespace, excessively long lines, mal-indention, >> superfluous use of PRI macros for int-or-smaller types, and incorrect PRI >> macros for gfns and mfns. >> >> Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com> >> --- >> CC: George Dunlap <george.dun...@eu.citrix.com> >> CC: Tamas K Lengyel <ta...@tklengyel.com> >> CC: Jan Beulich <jbeul...@suse.com> >> CC: Wei Liu <w...@xen.org> >> CC: Roger Pau Monné <roger....@citrix.com> > > And can I take this opportunity to remind all committers that `git diff > --check` exists and to please make sure it is used as part of pre-push > checks.
I don't think committers should do any checking. This should be part of the review process (and even better be part of the submitter's actions before sending out a patch). Committing should be a mechanical act. Many of us doing build checks already goes beyond what should really be required. That said I've nevertheless added it to my script, but I'll drop it again if the necessary cleaning up turns out to be too time consuming. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
