Joe Perches <[email protected]> wrote: > On Sat, 2012-10-20 at 23:33 +0100, Ken O'Brien wrote: > ... > > - pr_err("binder: %d: binder_alloc_buf failed to " > > + pr_err("binder: %d: binder_alloc_buf failed to " \ > > "map pages in userspace, no vma\n", proc->pid); > ... > Nice try, but the "right" way to do this is to coalesce formats like: > > pr_err("binder: %d: binder_alloc_buf failed to map pages in > userspace, no vma\n", > proc->pid);
Surely the right way is to fix _checkpatch_ to see ["]\n\s*["] and merge the lines before checking them? We shouldn't have to fix up the source in either of these fashions just because checkpatch is broken. David _______________________________________________ devel mailing list [email protected] http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
