Re: [PATCH] staging: speakup: rename enum constant CT_Off

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

 



Arnabjyoti Kalita, le mer. 25 juil. 2018 15:39:38 -0400, a ecrit:
> checkpatch.pl generates the following warning.
> 
> CHECK: Avoid CamelCase: <CT_Off>
> 
> CT_Off is written in CamelCase notation which is not allowed as 
> per the preferred coding style in the Linux kernel and hence 
> appears as a warning by checkpatch. Rename CT_Off to CT_OFF so 
> that it is no longer in CamelCase notation.
> 
> Signed-off-by: Arnabjyoti Kalita <arnabjyotikalita35@xxxxxxxxx>
> ---
>  drivers/staging/speakup/main.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/staging/speakup/main.c b/drivers/staging/speakup/main.c
> index 869f40e..7824aa3 100644
> --- a/drivers/staging/speakup/main.c
> +++ b/drivers/staging/speakup/main.c
> @@ -95,7 +95,7 @@ static int cursor_track = 1, prev_cursor_track = 1;
>  
>  /* cursor track modes, must be ordered same as cursor_msgs */
>  enum {
> -	CT_Off = 0,
> +	CT_OFF = 0,
>  	CT_On,

Err, please do make all others all upper case too.

Samuel
_______________________________________________
Speakup mailing list
Speakup@xxxxxxxxxxxxxxxxx
http://linux-speakup.org/cgi-bin/mailman/listinfo/speakup



[Index of Archives]     [Linux for the Blind]     [Fedora Discussioin]     [Linux Kernel]     [Yosemite News]     [Big List of Linux Books]

  Powered by Linux