Re: [Qemu-devel] [PATCH 0/4] Fix JSON string formatter

2013-04-13 Thread Blue Swirl
Thanks, applied all. On Thu, Apr 11, 2013 at 4:07 PM, Markus Armbruster wrote: > This should unbreak "make check" on machines where char is unsigned. > Blue, please give it a whirl. > > The JSON parser is still as broken as ever. Left for another day. > > v2: > - Rebased, trivial conflicts in PA

Re: [Qemu-devel] [PATCH 0/4] Fix JSON string formatter

2013-04-11 Thread Markus Armbruster
Rats, forgot --subject-prefix="PATCH v2". My apologies!

Re: [Qemu-devel] [PATCH 0/4] Fix JSON string formatter

2013-04-11 Thread Markus Armbruster
Blue Swirl writes: > On Thu, Mar 14, 2013 at 5:49 PM, Markus Armbruster wrote: >> This should unbreak "make check" on machines where char is unsigned. >> Blue, please give it a whirl. > > Patches no longer apply, please rebase. Sent.

Re: [Qemu-devel] [PATCH 0/4] Fix JSON string formatter

2013-04-11 Thread Laszlo Ersek
On 04/11/13 18:07, Markus Armbruster wrote: > This should unbreak "make check" on machines where char is unsigned. > Blue, please give it a whirl. > > The JSON parser is still as broken as ever. Left for another day. > > v2: > - Rebased, trivial conflicts in PATCH 1/4. > - Make mod_utf8_codepoin

[Qemu-devel] [PATCH 0/4] Fix JSON string formatter

2013-04-11 Thread Markus Armbruster
This should unbreak "make check" on machines where char is unsigned. Blue, please give it a whirl. The JSON parser is still as broken as ever. Left for another day. v2: - Rebased, trivial conflicts in PATCH 1/4. - Make mod_utf8_codepoint() treat empty input as invalid sequence of length zero (

Re: [Qemu-devel] [PATCH 0/4] Fix JSON string formatter

2013-03-23 Thread Blue Swirl
On Thu, Mar 14, 2013 at 5:49 PM, Markus Armbruster wrote: > This should unbreak "make check" on machines where char is unsigned. > Blue, please give it a whirl. Patches no longer apply, please rebase. > > The JSON parser is still as broken as ever. Left for another day. > > Markus Armbruster (4

Re: [Qemu-devel] [PATCH 0/4] Fix JSON string formatter

2013-03-18 Thread Markus Armbruster
Blue Swirl writes: > On Thu, Mar 14, 2013 at 5:49 PM, Markus Armbruster wrote: >> This should unbreak "make check" on machines where char is unsigned. >> Blue, please give it a whirl. > > With the patches applied there are no errors, thanks. > Tested-by: Blue Swirl Thanks! > Though test-corou

Re: [Qemu-devel] [PATCH 0/4] Fix JSON string formatter

2013-03-17 Thread Blue Swirl
On Thu, Mar 14, 2013 at 5:49 PM, Markus Armbruster wrote: > This should unbreak "make check" on machines where char is unsigned. > Blue, please give it a whirl. With the patches applied there are no errors, thanks. Tested-by: Blue Swirl Though test-coroutine seems to hang, maybe fallout from re

[Qemu-devel] [PATCH 0/4] Fix JSON string formatter

2013-03-14 Thread Markus Armbruster
This should unbreak "make check" on machines where char is unsigned. Blue, please give it a whirl. The JSON parser is still as broken as ever. Left for another day. Markus Armbruster (4): unicode: New mod_utf8_codepoint() check-qjson: Fix up a few bogus comments check-qjson: Test noncharac