[Spice-devel] on mjpeg stream of spice-gtk

2014-05-21 Thread lig.f...@cn.fujitsu.com
Hi, Marc, and all I'm trying to use hardware-decoder to help mjpeg decode, But when I dumped stream from server, I found, the frame is not jpeg, is it normal? Or I just dumped wrong data? st->mjpeg_cinfo->src->next_input_byte in stream_mjpeg_data, is it? Thanks! __

Re: [Spice-devel] streaming assertion

2014-05-21 Thread Uri Lublin
On 05/20/2014 11:19 PM, Jonathon Jongsma wrote: I've been investigating bug 1086820 that results in a failed assertion in spice-server (causing the guest to exit) related to mjpeg streaming. This is the debug output when we hit this issue: (/usr/bin/qemu-kvm:3165): Spice-ERROR **: mjpeg_encod

Re: [Spice-devel] [spice-html5] Tweak hexdump to not generate new lines.

2014-05-21 Thread Jeremy White
Anyway, it's just a suggestion. Since you added a check for == 8 (backspace) it checks more than newlines, and I thought you can protect against other non-printable characters (such as DEL == 127). Sure. Note that the impetus for the change was enabling me to feed the dumps into xxd -r, and it

Re: [Spice-devel] [spice-html5] Tweak hexdump to not generate new lines.

2014-05-21 Thread Uri Lublin
On 05/21/2014 04:42 PM, Jeremy White wrote: Hi, Looks good but maybe it's better to use isprint I'm not aware of an isprint Javascript function; do you have a pointer to that, or to a similar function? I too am not aware of isprint javascript function. I found that Java has Java.lang.Chara

Re: [Spice-devel] [spice-html5] Tweak hexdump to not generate new lines.

2014-05-21 Thread Jeremy White
Hi, Looks good but maybe it's better to use isprint I'm not aware of an isprint Javascript function; do you have a pointer to that, or to a similar function? Cheers, Jeremy ___ Spice-devel mailing list Spice-devel@lists.freedesktop.org http://list

Re: [Spice-devel] [spice-html5] Tweak hexdump to not generate new lines.

2014-05-21 Thread Uri Lublin
On 05/20/2014 09:03 PM, Jeremy White wrote: Signed-off-by: Jeremy White --- utils.js |5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/utils.js b/utils.js index 167f184..9eb42ff 100644 --- a/utils.js +++ b/utils.js @@ -63,7 +63,10 @@ function hexdump_buffer(a)

Re: [Spice-devel] [PATCH v2 1/2] Introduce --enable/disable-webdav option

2014-05-21 Thread Marc-André Lureau
thanks, applied both On Wed, May 21, 2014 at 8:07 AM, Tiziano Müller < tiziano.muel...@stepping-stone.ch> wrote: > This makes the phodav dependency configureable. > And name it after the corresponding channel. > --- > configure.ac | 21 + > 1 file changed, 17 insertions(+),