Re: PATCH] Fix new_id(n) arg parse from va list

2014-09-30 Thread Jason Ekstrand
While that looks correct, I'm sure it breaks something. As the person who wrote that code, that choice was very intentional. Even though the protocol type is uint32_t, new_id arguments are always passed into libwayland as either a wl_proxy or wl_resource and that's what this code is designed to h

Re: PATCH] Fix new_id(n) arg parse from va list

2014-09-30 Thread Jasper St. Pierre
What is this trying to solve? On Mon, Aug 18, 2014 at 11:12 AM, Maks Naumov wrote: > Signed-off-by: Maks Naumov > --- > src/connection.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/connection.c b/src/connection.c > index f292853..13e3a3e 100644 > --- a/src/conn

PATCH] Fix new_id(n) arg parse from va list

2014-09-30 Thread Maks Naumov
Signed-off-by: Maks Naumov --- src/connection.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/connection.c b/src/connection.c index f292853..13e3a3e 100644 --- a/src/connection.c +++ b/src/connection.c @@ -492,7 +492,7 @@ wl_argument_from_va_list(const char *signature, u