Re: [Spice-devel] [PATCH 1/7] Fix line length errors in main_channel

2012-01-20 Thread Hans de Goede
Hi, First of all many thanks for the patches, and welcome to the Spice community! As for this patch: ACK. I realize you cannot push patches yourself, so I (or some other spice team member) will take care of getting these into git, but first lets get the entire set reviewed. Regards, Hans On

Re: [Spice-devel] [PATCH 1/7] Fix line length errors in main_channel

2012-01-19 Thread Dan McGee
On Thu, Jan 19, 2012 at 3:14 PM, Daniel P. Berrange wrote: > On Thu, Jan 19, 2012 at 02:09:58PM -0600, Dan McGee wrote: >> These are all existing errors; fix them so they don't block future >> commits in this file unnecessarily. >> >>     error (1): length @ server/main_channel.c +369 >>     error

Re: [Spice-devel] [PATCH 1/7] Fix line length errors in main_channel

2012-01-19 Thread Daniel P. Berrange
On Thu, Jan 19, 2012 at 02:09:58PM -0600, Dan McGee wrote: > These are all existing errors; fix them so they don't block future > commits in this file unnecessarily. > > error (1): length @ server/main_channel.c +369 > error (2): length @ server/main_channel.c +444 > error (3): length

[Spice-devel] [PATCH 1/7] Fix line length errors in main_channel

2012-01-19 Thread Dan McGee
These are all existing errors; fix them so they don't block future commits in this file unnecessarily. error (1): length @ server/main_channel.c +369 error (2): length @ server/main_channel.c +444 error (3): length @ server/main_channel.c +764 error (4): length @ server/main_channe