[lkml]   [2012]   [Jan]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH 3/5] staging: tidspbridge: Lindent to drv_interface.c
On Mon, Jan 30, 2012 at 11:25:34AM -0600, Ramirez Luna, Omar wrote:
> > +                       pr_info("%s:%d handle(s) still opened\n", __func__,
> > +                               atomic_read(&bridge_cref));
> I remember the rule was to break lines as far to the right as
> possible, no? Chapter 2 CodingStyle, same for the other similar
> changes.

It doesn't mean you have to right justify things, it just means
indented. The original code is fine here and the new code is fine
here. It's up to whoever writes the code to decide.

dan carpenter

[unhandled content-type:application/pgp-signature]
 \ /
  Last update: 2012-01-30 20:31    [W:0.084 / U:4.620 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site