On 23/03/2023 3:40 pm, Jan Beulich wrote: > On 23.03.2023 15:30, Andrew Cooper wrote: >> On 22/03/2023 9:35 am, Jan Beulich wrote: >>> + { >>> + if ( mfn_eq(oos_fixup[idx].smfn[i], smfn) >>> + && (oos_fixup[idx].off[i] == off) ) >> Given that you mention style in the commit message, it would be nice to >> move the && onto the previous line. > Sure, done (and there was a 2nd instance).
There are plenty more. I just stopped reporting them. I guess if its "just" whitespace fixes (newlines, rearranging of operands/indents) then that's fine. Everything else can come later, and there is plenty. Subject to the SPDX tweak, Acked-by: Andrew Cooper <andrew.coop...@citrix.com> ~Andrew