linux-kernel - Re: [PATCHv4 1/8] fixed long description text exceeding 80 characters

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <ad36749f-c14b-55cf-cbf6-d40a4f8b572c@kernel.org>
Date:   Fri, 30 Dec 2016 18:40:56 +0000
From:   Jonathan Cameron <jic23@...nel.org>
To:     Scott Matheina <scott@...heina.com>, linux-kernel@...r.kernel.org
Cc:     Hartmut Knaack <knaack.h@....de>,
        Peter Meerwald-Stadler <pmeerw@...erw.net>,
        linux-iio@...r.kernel.org, devel@...verdev.osuosl.org
Subject: Re: [PATCHv4 1/8] fixed long description text exceeding 80 characters

On 25/12/16 19:56, Scott Matheina wrote:
> The description was split into 2 lines due to the line greatly
> exceeding the 80 character soft limit.

This is often deliberately done with strings.  It makes them
much easier to grep for in the source.  Adding the artificial
newline may destroy attempts by userspace tools to do their own
formatting of this string.

So best to leave this one alone.

Jonathan
> 
> Signed-off-by: Scott Matheina <scott@...heina.com>
> ---
>  drivers/staging/iio/addac/adt7316.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/staging/iio/addac/adt7316.c b/drivers/staging/iio/addac/adt7316.c
> index a7d90c8..08413a8 100644
> --- a/drivers/staging/iio/addac/adt7316.c
> +++ b/drivers/staging/iio/addac/adt7316.c
> @@ -2181,5 +2181,6 @@ int adt7316_probe(struct device *dev, struct adt7316_bus *bus,
>  EXPORT_SYMBOL(adt7316_probe);
>  
>  MODULE_AUTHOR("Sonic Zhang <sonic.zhang@...log.com>");
> -MODULE_DESCRIPTION("Analog Devices ADT7316/7/8 and ADT7516/7/9 digital temperature sensor, ADC and DAC driver");
> +MODULE_DESCRIPTION("Analog Devices ADT7316/7/8 and ADT7516/7/9 digital\n"
> +		   "temperature sensor, ADC and DAC driver");
>  MODULE_LICENSE("GPL v2");
>