Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1264932

Re: [PATCH] Staging: iio: Fix some character line limit coding style issues

Show all headers | View raw

On Sat, Nov 07, 2015 at 10:02:04AM -0800, Joe Perches wrote:
> On Sat, 2015-11-07 at 19:30 +0300, Dan Carpenter wrote:
> > The original is easier to read, sorry.  Checkpatch is a tool not the
> > king of the world to obeyed at all times.
> 
> Completely agree.
> 
> Perhaps there could be some comment indicators added
> to tell checkpatch to ignore certain warnings like
> 80 columns.

Applying or rejecting this patch was an easy call so it doesn't cause
decision fatigue or drain my emotions.  I feel like these patches are
instructional so people learn not to trust tools.  Checkpatch.pl is fine
as-is.

regards,
dan carpenter
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH] Staging: iio: Fix some character line limit coding style  issues Evan Hosseini <hosse005@umn.edu> - 2015-11-07 16:20 +0100
  Re: [PATCH] Staging: iio: Fix some character line limit coding style  issues Dan Carpenter <dan.carpenter@oracle.com> - 2015-11-07 17:40 +0100
    Re: [PATCH] Staging: iio: Fix some character line limit coding  style issues Joe Perches <joe@perches.com> - 2015-11-07 19:10 +0100
      Re: [PATCH] Staging: iio: Fix some character line limit coding style  issues Dan Carpenter <dan.carpenter@oracle.com> - 2015-11-07 21:50 +0100

csiph-web