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

Groups > linux.kernel > #1547198 > unrolled thread

[PATCHv4 1/8] fixed long description text exceeding 80 characters

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  [PATCHv4 1/8] fixed long description text exceeding 80 characters Scott Matheina <scott@matheina.com> - 2016-12-25 21:10 +0100
    Re: [PATCHv4 1/8] fixed long description text exceeding 80 characters Jonathan Cameron <jic23@kernel.org> - 2016-12-30 19:50 +0100

#1547198 — [PATCHv4 1/8] fixed long description text exceeding 80 characters

The description was split into 2 lines due to the line greatly
exceeding the 80 character soft limit.

Signed-off-by: Scott Matheina <scott@matheina.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@analog.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");
-- 
2.7.4

[toc] | [next] | [standalone]

#1548678

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@matheina.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@analog.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");
> 

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web