Re: [Qemu-devel] [PATCH v2 00/60] json: Fixes, error reporting improvements, cleanups

2018-08-20 Thread Markus Armbruster
Fam Zheng writes: > On Mon, 08/20 10:31, Markus Armbruster wrote: >> no-re...@patchew.org writes: >> >> > Hi, >> > >> > This series seems to have some coding style problems. See output below for >> > more information: >> [...] >> > Checking PATCH 4/60: qmp-cmd-test: Split off qmp-test... >> > WA

Re: [Qemu-devel] [PATCH v2 00/60] json: Fixes, error reporting improvements, cleanups

2018-08-20 Thread Fam Zheng
On Mon, 08/20 10:31, Markus Armbruster wrote: > no-re...@patchew.org writes: > > > Hi, > > > > This series seems to have some coding style problems. See output below for > > more information: > [...] > > Checking PATCH 4/60: qmp-cmd-test: Split off qmp-test... > > WARNING: added, moved or deleted

Re: [Qemu-devel] [PATCH v2 00/60] json: Fixes, error reporting improvements, cleanups

2018-08-20 Thread Markus Armbruster
no-re...@patchew.org writes: > Hi, > > This series seems to have some coding style problems. See output below for > more information: [...] > Checking PATCH 4/60: qmp-cmd-test: Split off qmp-test... > WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? > #48: > new file mode

Re: [Qemu-devel] [PATCH v2 00/60] json: Fixes, error reporting improvements, cleanups

2018-08-18 Thread no-reply
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20180817150559.16243-1-arm...@redhat.com Subject: [Qemu-devel] [PATCH v2 00/60] json: Fixes, error reporting improvements, cleanups === TEST SCRIPT BEGIN === #!/bin/bash

[Qemu-devel] [PATCH v2 00/60] json: Fixes, error reporting improvements, cleanups

2018-08-17 Thread Markus Armbruster
JSON is such a simple language, so writing a parser should be easy, shouldn't it? Well, the evidence is in, and it's a lot of patches. Summary of fixes: * Reject ASCII control characters in strings as RFC 7159 specifies * Reject all invalid UTF-8 sequences, not just some * Reject invalid \u