]> git.proxmox.com Git - mirror_ubuntu-jammy-kernel.git/commit
staging: lustre: Convert "return seq_printf(...)" uses
authorJoe Perches <joe@perches.com>
Sun, 22 Feb 2015 02:53:28 +0000 (18:53 -0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 2 Mar 2015 01:21:21 +0000 (17:21 -0800)
commit8faeebdf1535e61baab53daeca12f2e8a0071db7
treee87811d36625bc69b6c62f2b5856665dcd139177
parent4643b70a08442d2aa494eabd6b024a7102aeaa40
staging: lustre: Convert "return seq_printf(...)" uses

The seq_printf return value, because it's frequently misused,
will eventually be converted to void.

See: commit 1f33c41c03da ("seq_file: Rename seq_overflow() to
     seq_has_overflowed() and make public")

Convert these "return seq_printf(...)" uses to:

seq_printf(seq, ...);
return 0;

Done via cocci script:

@@
struct seq_file *seq;
@@

- return
\(seq_printf\|seq_puts\|seq_putc\)(seq,
...);

+ return 0;

With some additional reformatting and typing post conversion.

Miscellanea:

o convert an rc++ use to a bool
o use seq_puts with fixed strings where appropriate

Signed-off-by: Joe Perches <joe@perches.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: Andreas Dilger <andreas.dilger@intel.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/lustre/lustre/libcfs/hash.c
drivers/staging/lustre/lustre/llite/lproc_llite.c
drivers/staging/lustre/lustre/lmv/lproc_lmv.c
drivers/staging/lustre/lustre/lov/lproc_lov.c
drivers/staging/lustre/lustre/obdclass/linux/linux-module.c
drivers/staging/lustre/lustre/obdclass/lprocfs_status.c
drivers/staging/lustre/lustre/obdclass/lu_object.c
drivers/staging/lustre/lustre/osc/lproc_osc.c
drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c