Messages in this thread | | | Subject | Re: [PATCH] Staging: anrdroid: sync.c: fixed all checkpatch errors and warnings | From | Joe Perches <> | Date | Wed, 05 Jun 2013 16:14:03 -0700 |
| |
On Wed, 2013-06-05 at 23:23 +0300, Aldo Iljazi wrote: > diff --git a/drivers/staging/android/sync.c b/drivers/staging/android/sync.c [] > @@ -893,13 +892,13 @@ static void sync_print_obj(struct seq_file *s, struct sync_timeline *obj) > if (obj->ops->timeline_value_str) { > char value[64]; > obj->ops->timeline_value_str(obj, value, sizeof(value)); > - seq_printf(s, ": %s", value); > + seq_puts(s, ": %s", value);
This is wrong and shouldn't compile without warning.
Is this an over aggressive conversion or did checkpatch really complain on this line?
| |