Re: [Spice-devel] [PATCH 11/11] dcc: remove not necessary volatile specifications

2016-02-12 Thread Christophe Fergeau
On Fri, Feb 12, 2016 at 06:14:06AM -0500, Frediano Ziglio wrote: > > > > [NB: this did not go to the ML] > > On Fri, Feb 12, 2016 at 05:12:57AM -0500, Frediano Ziglio wrote: > > > > > > > > This causes: > > > > make[4]: Entering directory '/home/teuf/redhat/spice/server' > > > > CC dcc.lo

Re: [Spice-devel] [PATCH 11/11] dcc: remove not necessary volatile specifications

2016-02-12 Thread Frediano Ziglio
> > [NB: this did not go to the ML] > On Fri, Feb 12, 2016 at 05:12:57AM -0500, Frediano Ziglio wrote: > > > > > > This causes: > > > make[4]: Entering directory '/home/teuf/redhat/spice/server' > > > CC dcc.lo > > > dcc.c: In function 'dcc_compress_image_jpeg': > > > dcc.c:852:26: error:

Re: [Spice-devel] [PATCH 11/11] dcc: remove not necessary volatile specifications

2016-02-12 Thread Christophe Fergeau
This causes: make[4]: Entering directory '/home/teuf/redhat/spice/server' CC dcc.lo dcc.c: In function 'dcc_compress_image_jpeg': dcc.c:852:26: error: variable 'jpeg_in_type' might be clobbered by 'longjmp' or 'vfork' [-Werror=clobbered] JpegEncoderImageType jpeg_in_type;

Re: [Spice-devel] [PATCH 11/11] dcc: remove not necessary volatile specifications

2016-02-11 Thread Jonathon Jongsma
Acked-by: Jonathon Jongsma On Tue, 2016-02-09 at 10:28 +, Frediano Ziglio wrote: > These automatic variable are used just inside the function. > > Signed-off-by: Frediano Ziglio > --- > server/dcc.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/server/dc

[Spice-devel] [PATCH 11/11] dcc: remove not necessary volatile specifications

2016-02-09 Thread Frediano Ziglio
These automatic variable are used just inside the function. Signed-off-by: Frediano Ziglio --- server/dcc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/server/dcc.c b/server/dcc.c index 99aadf0..4f458ec 100644 --- a/server/dcc.c +++ b/server/dcc.c @@ -792,9 +792,9 @