diff options
| author | Markus Armbruster <armbru@redhat.com> | 2016-08-03 13:37:51 +0200 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2016-08-08 09:00:44 +0200 |
| commit | df3c286c53ac51e7267f2761c7a0c62e11b6e815 (patch) | |
| tree | 3871ea9f31a2b17dd3d5b146c20fe09a00fe2f63 /hw/ppc | |
| parent | 51009170d8fc263cfdcd5a60fe3ba213daa3d15b (diff) | |
| download | qemu-df3c286c53ac51e7267f2761c7a0c62e11b6e815.tar.gz qemu-df3c286c53ac51e7267f2761c7a0c62e11b6e815.tar.bz2 qemu-df3c286c53ac51e7267f2761c7a0c62e11b6e815.zip | |
error: Strip trailing '\n' from error string arguments (again)
Commit 9af9e0f, 6daf194d, be62a2eb and 312fd5f got rid of a bunch, but
they keep coming back. checkpatch.pl tries to flag them since commit
5d596c2, but it's not very good at it. Offenders tracked down with
Coccinelle script scripts/coccinelle/err-bad-newline.cocci, an updated
version of the script from commit 312fd5f.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <1470224274-31522-2-git-send-email-armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Diffstat (limited to 'hw/ppc')
| -rw-r--r-- | hw/ppc/spapr_cpu_core.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/hw/ppc/spapr_cpu_core.c b/hw/ppc/spapr_cpu_core.c index 170ed154a6..3ec47b7954 100644 --- a/hw/ppc/spapr_cpu_core.c +++ b/hw/ppc/spapr_cpu_core.c @@ -234,7 +234,7 @@ void spapr_core_pre_plug(HotplugHandler *hotplug_dev, DeviceState *dev, } if (cc->core_id % smp_threads) { - error_setg(&local_err, "invalid core id %d\n", cc->core_id); + error_setg(&local_err, "invalid core id %d", cc->core_id); goto out; } |