Signed-off-by: Jeremy White
---
src/scan.c| 6 +++---
src/session.c | 8
src/spice.c | 18 +++---
3 files changed, 18 insertions(+), 14 deletions(-)
diff --git a/src/scan.c b/src/scan.c
index bb020de..ebadaf8 100644
--- a/src/scan.c
+++ b/src/scan.c
@@ -70,7 +70,7
On 6/13/19 2:47 PM, Frediano Ziglio wrote:
On 6/13/19 1:23 PM, Frediano Ziglio wrote:
This constant fits in a 32 bit signed integer so it does not need the
suffix. However some of the derived constants don't so use an uint64_t
cast to avoid the long vs long long confusion (such as in print
>
> On 6/13/19 1:23 PM, Frediano Ziglio wrote:
> >>
> >> This constant fits in a 32 bit signed integer so it does not need the
> >> suffix. However some of the derived constants don't so use an uint64_t
> >> cast to avoid the long vs long long confusion (such as in print
> >> statements).
> >>
On 6/13/19 12:57 PM, Frediano Ziglio wrote:
On 6/12/19 1:53 PM, Frediano Ziglio wrote:
On Windows Fedora 30 reports these errors:
In file included from
/usr/x86_64-w64-mingw32/sys-root/mingw/include/openssl/crypto.h:29,
from
On 6/13/19 1:23 PM, Frediano Ziglio wrote:
This constant fits in a 32 bit signed integer so it does not need the
suffix. However some of the derived constants don't so use an uint64_t
cast to avoid the long vs long long confusion (such as in print
statements).
Also some of the expressions these
>
> This constant fits in a 32 bit signed integer so it does not need the
> suffix. However some of the derived constants don't so use an uint64_t
> cast to avoid the long vs long long confusion (such as in print
> statements).
> Also some of the expressions these constants are used in may
> On 6/12/19 1:53 PM, Frediano Ziglio wrote:
> > On Windows Fedora 30 reports these errors:
> >
> > In file included from
> > /usr/x86_64-w64-mingw32/sys-root/mingw/include/openssl/crypto.h:29,
> > from
> >
> >
On Wed, 12 Jun 2019, Snir Sheriber wrote:
> Hi,
>
> On 6/11/19 9:42 PM, Francois Gouget wrote:
> > schedule_frame() only pulls frames out of GStreamer's pipeline once all
> > previous decoded frames have been displayed. Thus when the video delay
>
>
> IIRC we used to pull when samples arrived
On 6/12/19 1:53 PM, Frediano Ziglio wrote:
On Windows Fedora 30 reports these errors:
In file included from
/usr/x86_64-w64-mingw32/sys-root/mingw/include/openssl/crypto.h:29,
from
/usr/x86_64-w64-mingw32/sys-root/mingw/include/openssl/bio.h:20,
from
This constant fits in a 32 bit signed integer so it does not need the
suffix. However some of the derived constants don't so use an uint64_t
cast to avoid the long vs long long confusion (such as in print
statements).
Also some of the expressions these constants are used in may overflow so
perform
This constant fits in a regular 32 bit signed integer so it does not
need the suffix.
It is also not used in expressions that may overflow.
Signed-off-by: Francois Gouget
---
v2: No changes.
v1: https://lists.freedesktop.org/archives/spice-devel/2019-May/049193.html
server/utils.h | 2 +-
1
Hey,
yup fine with me too.
Christophe
On Thu, Jun 13, 2019 at 03:40:17AM -0400, Frediano Ziglio wrote:
> >
> > Hi,
> >
> > Fine with me.
> >
>
> This is also related to
> https://gitlab.freedesktop.org/spice/spice/merge_requests/2,
> I talked with Christophe and he agreed would be sensible
On 6/11/19 9:43 PM, Frediano Ziglio wrote:
Make sure the project compile and pass tests without problems
on a 32 bit architecture.
Signed-off-by: Frediano Ziglio
Ack.
Uri
---
.gitlab-ci.yml | 22 ++
1 file changed, 22 insertions(+)
diff --git a/.gitlab-ci.yml
On 6/12/19 9:30 AM, Frediano Ziglio wrote:
Some typos/improves (just commit message)
Subject: [PATCH spice-server 1/2] Fix compile warnings on Linux 32bit system
Better "Remove compile warnings ..."
Both are OK.
Based on a patch from Hongzhi.Song .
There are following compile errors
We started disabling Celt support making the option required.
After 2 releases start making it disabled unless explicitly
enabled.
This makes https://gitlab.freedesktop.org/spice/spice/merge_requests/2
obsolete.
Signed-off-by: Frediano Ziglio
---
meson_options.txt| 1 +
>
> Hi,
>
> Fine with me.
>
This is also related to
https://gitlab.freedesktop.org/spice/spice/merge_requests/2,
I talked with Christophe and he agreed would be sensible to disable by default.
> On 6/12/19 3:06 PM, Frediano Ziglio wrote:
> > We started disabling Celt support making the
>
> Assuming it passed the CI, Ack.
>
Yes and not. Yes as the specific job passes.
No as there are some pending patches which are required in order to
compiler for Fedora 30 (one for Windows that I sent yesterday, the others
some weeks ago about alignment).
> Acked-by: Snir Sheriber
>
> On
Hi,
Fine with me.
On 6/12/19 3:06 PM, Frediano Ziglio wrote:
We started disabling Celt support making the option required.
After 2 releases start making it disabled unless explicitly
enabled.
Signed-off-by: Frediano Ziglio
---
m4/spice-deps.m4 | 14 ++
meson_options.txt | 1
It seems this was pushed already some time ago
On 6/12/19 2:53 PM, Frediano Ziglio wrote:
This brings in the following changes:
Frediano Ziglio (4):
gitignore: Ignore auto generated generated_messages.h file
Do not check for HAVE_CONFIG_H
build: Remove unused
Based on a patch from Hongzhi.Song .
There are following compile errors on Linux 32bit system with -Werror
for gcc.
red-channel.c:207:73: error: format '%x' expects argument of type
'unsigned int', but argument 7 has type 'long unsigned int' [-Werror=format=]
|207| red_channel_debug(self,
20 matches
Mail list logo