Acked-by: Jonathon Jongsma
On Wed, 2017-09-20 at 08:51 +0100, Frediano Ziglio wrote:
> Signed-off-by: Frediano Ziglio
> ---
> server/display-channel.c | 2 +-
> server/red-parse-qxl.c | 46 +++---
>
> 2 files changed, 24 insertions(+), 24 deletions(-
Acked-by: Jonathon Jongsma
On Mon, 2017-09-25 at 13:20 +0100, Frediano Ziglio wrote:
> Signed-off-by: Frediano Ziglio
> Acked-by: Jonathon Jongsma
> ---
> server/cache-item.tmpl.c | 6 +++---
> server/dcc.c | 10 +-
> server/inputs-channel.c |
Acked-by: Jonathon Jongsma
On Mon, 2017-09-25 at 09:25 +0100, Frediano Ziglio wrote:
> Signed-off-by: Frediano Ziglio
> ---
> server/spicevmc.c | 14 +++---
> 1 file changed, 7 insertions(+), 7 deletions(-)
>
> Changes since v2:
> - move RedPipeItem stuff in proper patch.
>
> diff --g
On Mon, 2017-09-25 at 03:56 -0400, Frediano Ziglio wrote:
> >
> > On Wed, 2017-09-20 at 08:50 +0100, Frediano Ziglio wrote:
> > > Signed-off-by: Frediano Ziglio
> > > ---
> > > server/mjpeg-encoder.c | 28 ++--
> > > 1 file changed, 14 insertions(+), 14 deletions(-)
> > >
On Mon, Sep 25, 2017 at 04:37:43PM +0200, Christophe Fergeau wrote:
> On Mon, Sep 25, 2017 at 10:22:57AM -0400, Frediano Ziglio wrote:
> > >
> > > On Mon, Sep 25, 2017 at 09:04:16AM -0400, Frediano Ziglio wrote:
> > > > Yes, intentional and consistent with previous code.
> > > > The usage of unsig
On Mon, Sep 25, 2017 at 10:22:57AM -0400, Frediano Ziglio wrote:
> >
> > On Mon, Sep 25, 2017 at 09:04:16AM -0400, Frediano Ziglio wrote:
> > > Yes, intentional and consistent with previous code.
> > > The usage of unsigned for BYTESWAP avoid sign extension during arithmetic.
> >
> > If BYTESWAP
>
> On Mon, Sep 25, 2017 at 09:04:16AM -0400, Frediano Ziglio wrote:
> > Yes, intentional and consistent with previous code.
> > The usage of unsigned for BYTESWAP avoid sign extension during arithmetic.
>
> If BYTESWAP has such issues (and I don't think it does), then this should be
> fixed.
> a
On Mon, Sep 25, 2017 at 09:04:16AM -0400, Frediano Ziglio wrote:
> Yes, intentional and consistent with previous code.
> The usage of unsigned for BYTESWAP avoid sign extension during arithmetic.
If BYTESWAP has such issues (and I don't think it does), then this should be
fixed.
ack on being cons
>
> On Fri, Sep 22, 2017 at 11:10:02AM +0100, Frediano Ziglio wrote:
> > Instead of assuming that the system can safely do unaligned access
> > to memory use packed structures to allow the compiler generate
> > best code possible.
> > A packed structure tells the compiler to not leave padding insi
On Fri, Sep 22, 2017 at 11:10:02AM +0100, Frediano Ziglio wrote:
> Instead of assuming that the system can safely do unaligned access
> to memory use packed structures to allow the compiler generate
> best code possible.
> A packed structure tells the compiler to not leave padding inside it
> and t
Signed-off-by: Frediano Ziglio
Acked-by: Jonathon Jongsma
---
server/cache-item.tmpl.c | 6 +++---
server/dcc.c | 10 +-
server/inputs-channel.c | 4 ++--
server/main-channel-client.c | 14 +++---
server/red-channel-client.c |
Signed-off-by: Frediano Ziglio
---
server/spicevmc.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
Changes since v2:
- move RedPipeItem stuff in proper patch.
diff --git a/server/spicevmc.c b/server/spicevmc.c
index 6b9b96fc8..f56f50eda 100644
--- a/server/spicevmc.c
+++
>
> On Wed, 2017-09-20 at 08:50 +0100, Frediano Ziglio wrote:
> > Signed-off-by: Frediano Ziglio
> > ---
> > server/mjpeg-encoder.c | 28 ++--
> > 1 file changed, 14 insertions(+), 14 deletions(-)
> >
> > diff --git a/server/mjpeg-encoder.c b/server/mjpeg-encoder.c
> > i
13 matches
Mail list logo