Re: [PATCH 0/9] staging: unisys: checkpatch strict cleanup in include

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 2014-09-25 at 16:30 +0300, Dan Carpenter wrote:
> Probably that's fine, but normally we suggest:
> 
> patch 1: clean up long lines
> patch 2: delete extra blank lines
> patch 3: comments
> 
> or whatever...
> 

OK, thanks. I suppose if I'm not sure, I'll make individual patches and
I can just squash them together if someone complains that they're too
small.

> What really helps me is if you list the exact variables that were
> changed in the changelog like this:
> 
> Cleanup CamelCase names:
> FooBar => foo_bar
> OneTwo => one_two
> 
> I use a script to sed these renames out so I can just cut and paste the
> renames and verify that nothing else changed.

I'll do that! Thanks for the help! :)

-- Ben
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux