Re: [Spice-devel] [PATCH v2 x11spice 2/4] Apply Spice style to the spice-video-dummy

2019-05-02 Thread Jeremy White
Hi Frediano, Thanks for the review. I've condensed the thread to just a few interesting points; for the most part, I was able to get indent to make the changes you requested, and will resubmit with that. +if (!infoPtr) +return FALSE; What we usually want is always brackets

Re: [Spice-devel] [PATCH v2 x11spice 2/4] Apply Spice style to the spice-video-dummy

2019-05-02 Thread Frediano Ziglio
I suppose from IRC chat you used indent tool. > Signed-off-by: Jeremy White > --- > spice-video-dummy/src/dummy.h| 9 +- > spice-video-dummy/src/dummy_cursor.c | 42 +-- > spice-video-dummy/src/dummy_driver.c | 558 > --- > 3 files changed, 281 inser

[Spice-devel] [PATCH v2 x11spice 2/4] Apply Spice style to the spice-video-dummy

2019-05-01 Thread Jeremy White
Signed-off-by: Jeremy White --- spice-video-dummy/src/dummy.h| 9 +- spice-video-dummy/src/dummy_cursor.c | 42 +-- spice-video-dummy/src/dummy_driver.c | 558 --- 3 files changed, 281 insertions(+), 328 deletions(-) diff --git a/spice-video-dummy/src/d