[Spice-devel] [PATCH spice-common] common/log.h: add spice_client_error

2012-07-05 Thread Alon Levy
To be used to mark client protocol violations, more easily greppable. --- common/log.h | 11 +++ 1 file changed, 11 insertions(+) diff --git a/common/log.h b/common/log.h index 37c491f..df8a756 100644 --- a/common/log.h +++ b/common/log.h @@ -28,6 +28,10 @@ SPICE_BEGIN_DECLS #define

Re: [Spice-devel] [PATCH spice-common] common/log.h: add spice_client_error

2012-07-05 Thread Marc-André Lureau
On Thu, Jul 5, 2012 at 11:38 AM, Alon Levy al...@redhat.com wrote: To be used to mark client protocol violations, more easily greppable. --- common/log.h | 11 +++ 1 file changed, 11 insertions(+) diff --git a/common/log.h b/common/log.h index 37c491f..df8a756 100644 ---

Re: [Spice-devel] [PATCH spice-common] common/log.h: add spice_client_error

2012-07-05 Thread Alon Levy
On Thu, Jul 05, 2012 at 01:10:44PM +0200, Marc-André Lureau wrote: On Thu, Jul 5, 2012 at 11:38 AM, Alon Levy al...@redhat.com wrote: To be used to mark client protocol violations, more easily greppable. --- common/log.h | 11 +++ 1 file changed, 11 insertions(+) diff --git

Re: [Spice-devel] [PATCH spice-common] common/log.h: add spice_client_error

2012-07-05 Thread Marc-André Lureau
Hi On Thu, Jul 5, 2012 at 1:17 PM, Alon Levy al...@redhat.com wrote: That doesn't allow me to have in the same compilation unit (red_worker.c) two domains. Also, you can't define macros with macros, so I'm not sure what better way there is. I'd put the macro in red_worker.c in this case. And

Re: [Spice-devel] [PATCH spice-common] common/log.h: add spice_client_error

2012-07-05 Thread Alon Levy
On Thu, Jul 05, 2012 at 01:20:35PM +0200, Marc-André Lureau wrote: Hi On Thu, Jul 5, 2012 at 1:17 PM, Alon Levy al...@redhat.com wrote: That doesn't allow me to have in the same compilation unit (red_worker.c) two domains. Also, you can't define macros with macros, so I'm not sure what