Messages in this thread | | | Date | Sat, 3 Aug 2013 18:51:55 +0200 | From | Oleg Nesterov <> | Subject | Re: [PATCH v3 3/3] fs/binfmts: Whitespace fixes with scripts/cleanfile |
| |
On 08/02, Zach Levis wrote: > > Obligatory first-patchset whitespace commit
Looks good, perhaps you send it as 1/3 or even separately, so you won't need to redo it if you need to update 1-2.
> @@ -63,11 +63,11 @@ static int load_em86(struct linux_binprm *bprm) > */ > remove_arg_zero(bprm); > retval = copy_strings_kernel(1, &bprm->filename, bprm); > - if (retval < 0) return retval; > + if (retval < 0) return retval;
And perhaps you can also fix the coding style when your patch touches such a code.
Oleg.
| |