Re: Solving checkpatch error with lex

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

 



Thanks for your input.

On Thu, Dec 18, 2014, 6:17 AM Greg KH <greg@xxxxxxxxx> wrote:
On Wed, Dec 17, 2014 at 04:24:15PM -0500, Valdis.Kletnieks@xxxxxx wrote:
> On Wed, 17 Dec 2014 19:10:24 +0000, karthik nayak said:
>
> > The only reason I didn't do that is cause the file is too large to
> > manually have a look at, even the diff tends to be very huge.
>
> [/usr/src/linux-next] grep '//' drivers/staging/rtl8192u/r8192U_dm.c | wc -l
> 373
>
> And why use lex when sed is up to the task?
>
>  [/usr/src/linux-next] sed 's?//(.*)$?/* \1 */?' drivers/staging/rtl8192u/r8192U_dm.c >| /tmp/z99
> sed: -e _expression_ #1, char 19: invalid reference \1 on `s' command's RHS
> 1 [/usr/src/linux-next] sed 's?//\(.*\)$?/* \1 */?' drivers/staging/rtl8192u/r8192U_dm.c >| /tmp/z99
> 0 [/usr/src/linux-next] diff -u drivers/staging/rtl8192u/r8192U_dm.c /tmp/z99 | head -20
> --- drivers/staging/rtl8192u/r8192U_dm.c        2014-11-20 16:18:35.941071156 -0500
> +++ /tmp/z99    2014-12-17 16:22:42.831566630 -0500
> @@ -21,9 +21,9 @@
>  #include "r8190_rtl8256.h"
>  #include "r819xU_cmdpkt.h"
>  /*---------------------------Define Local Constant---------------------------*/
> -//
> -// Indicate different AP vendor for IOT issue.
> -//
> +/*  */
> +/*  Indicate different AP vendor for IOT issue. */
> +/*  */
>  static u32 edca_setting_DL[HT_IOT_PEER_MAX] =
>                 { 0x5e4322,     0x5e4322,       0x5e4322,       0x604322,       0xa44f,         0x5ea44f};
>  static u32 edca_setting_UL[HT_IOT_PEER_MAX] =
> @@ -36,11 +36,11 @@
>
>
>  /*------------------------Define global variable-----------------------------*/
> -// Debug variable ?
> 0 [/usr/src/linux-next] diff -u drivers/staging/rtl8192u/r8192U_dm.c /tmp/z99 | wc -l
> 1960

What a mess, I'll gladly reject a patch that does this.

Look at that multi-line crud.

ick ick ick, don't run auto-formatting tools just to fix up checkpatch
issues.  If that was all that is needed, I would have done that a long
time ago, that's not the goal here people...

greg k-h
_______________________________________________
Kernelnewbies mailing list
Kernelnewbies@xxxxxxxxxxxxxxxxx
http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies

[Index of Archives]     [Newbies FAQ]     [Linux Kernel Mentors]     [Linux Kernel Development]     [IETF Annouce]     [Git]     [Networking]     [Security]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux SCSI]     [Linux ACPI]
  Powered by Linux