Can anyone please comment on it? :)
BR
imran
On Wed, Dec 17, 2014 at 4:11 PM, Imran Zaman imran.za...@gmail.com wrote:
Nested westons (with wayland backend as child weston) is one possible
use case for the needed touch support.
Signed-off-by: Imran Zaman imran.za...@gmail.com
---
src
Hi Ekstrand!
I have tested it in nested compositors case; unfortunately it is not enough
and doesn't work :-)
The reason is that drm compositor (choose_mode) function rejects the mode
if width and height are swapped so similar patch is needed for choose_mode
in drm compositor based on transform.
Nested westons (with wayland backend as child weston) is one possible
use case for the needed touch support.
Signed-off-by: Imran Zaman imran.za...@gmail.com
---
src/compositor-wayland.c | 59
1 file changed, 59 insertions(+)
diff --git a/src
Interesting read.. have some arguments about the raised points.. but
lets not go in that direction..
Can you guys suggest to do some changes or shall i drop the patch
altogether? if you are happy with current implementation lets live
with it.. I noticed bug when weston was not working properly due
to
not allow other negative numbers however.
[IZ] IMO its good to keep the APIs simple and homogeneous with
appropriate data structures needed for the input
BR
imran
On 11/21/2014 07:38 AM, Imran Zaman wrote:
+static bool
+convert_strtol(const char *str, char **endptr, int base, long *val
Hi
Somehow I dont see the patch below in patch work:
http://lists.freedesktop.org/archives/wayland-devel/2014-November/018410.html
Any idea what is wrong with it or there is some sort of filtering or ?
BR
imran
On Thu, Nov 20, 2014 at 12:47 AM, Bill Spitzak spit...@gmail.com wrote:
On
:22 PM, Pekka Paalanen ppaala...@gmail.com wrote:
On Wed, 15 Oct 2014 16:43:34 +0300
Imran Zaman imran.za...@gmail.com wrote:
Hi
This will allow more than 1 simultaneous client connections to the server
without the possibility of connection refused error. possible use case
is multiple session
:33:51 +0200
Imran Zaman imran.za...@gmail.com wrote:
Hi
Somehow I dont see the patch below in patch work:
http://lists.freedesktop.org/archives/wayland-devel/2014-November/018410.html
Any idea what is wrong with it or there is some sort of filtering or ?
Hi, yeah, I tried to ask you about
This will allow more than 1 simultaneous client connections to the server
without the possibility of connection refused error.
Signed-off-by: Imran Zaman imran.za...@gmail.com
---
src/wayland-server.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/wayland-server.c b/src
On Tue, Nov 25, 2014 at 1:15 AM, Bill Spitzak spit...@gmail.com wrote:
On 11/24/2014 11:32 AM, Imran Zaman wrote:
[IZ2] This is the case where endptr is used in patch. I can remove
endptr but it seems its used so i have to keep the existing
functionality in place.
+ if (!weston_strtoi(pid
Change-Id: I50900852311604a8c31313bbfb1d137c495d2269
Signed-off-by: Imran Zaman imran.za...@gmail.com
---
Makefile.am | 14 +-
clients/multi-resource.c| 8 +-
clients/terminal.c | 8 +-
clients/wscreensaver-glue.c | 6 +-
shared/config-parser.c | 10
... corrected jussi email address..
BR
irman
On Wed, Nov 19, 2014 at 12:56 PM, Pekka Paalanen ppaala...@gmail.com wrote:
On Wed, 15 Oct 2014 17:36:27 +0300
Imran Zaman imran.za...@gmail.com wrote:
Hi
support for adjusting socket access rights to allow group of users to
connect
Imran Zaman imran.za...@gmail.com wrote:
The reason is that strtol is used at many places in weston/wayland..
and its not covering all the error cases everywhere (i.e. its buggy)..
so its better to
encapsulate it in a function with all the input and output checks...
it can be moved
Signed-off-by: Imran Zaman imran.za...@gmail.com
---
src/scanner.c| 5 -
src/wayland-client.c | 6 +-
2 files changed, 9 insertions(+), 2 deletions(-)
diff --git a/src/scanner.c b/src/scanner.c
index 5e5152b..9ba34e8 100644
--- a/src/scanner.c
+++ b/src/scanner.c
@@ -405,9
Signed-off-by: Imran Zaman imran.za...@gmail.com
---
src/scanner.c| 4 +++-
src/wayland-client.c | 5 -
2 files changed, 7 insertions(+), 2 deletions(-)
diff --git a/src/scanner.c b/src/scanner.c
index 5e5152b..fa8e0c0 100644
--- a/src/scanner.c
+++ b/src/scanner.c
@@ -405,11 +405,13
pushed v3 to patchwork with the change mentioned and marked the older
versions as superceeded
http://patchwork.freedesktop.org/patch/36297/
BR
imran
On Wed, Nov 5, 2014 at 5:27 PM, Pekka Paalanen ppaala...@gmail.com wrote:
On Tue, 4 Nov 2014 15:55:06 +0200
Imran Zaman imran.za...@gmail.com
Pekka, thanks a lot for the detailed explanation. Lets see which way we go.
BR
imran
On Wed, Nov 5, 2014 at 6:32 PM, Pekka Paalanen ppaala...@gmail.com wrote:
On Fri, 17 Oct 2014 12:37:37 +0300
Imran Zaman imran.za...@gmail.com wrote:
In a multi-seat configuration, clients may need
Thats really good initiative Pekka/Daniel. Thanks.
BR
imran
On Tue, Nov 4, 2014 at 1:14 PM, Pekka Paalanen ppaala...@gmail.com wrote:
Hi all,
some of you already know that we now have
http://patchwork.freedesktop.org/project/wayland/list/
to keep track of the patches sent to wayland-devel
I will push new patch with minor fix to the strtol function in wayland and
move this old patch (after segfault fix) to weston so that it does not end
up in libwayland APIs.
Consequently I changed its property in patchwork
BR
imran
On Wed, Oct 29, 2014 at 10:27 AM, Imran Zaman imran.za
Signed-off-by: Imran Zaman imran.za...@gmail.com
---
src/scanner.c| 2 +-
src/wayland-client.c | 3 ++-
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/scanner.c b/src/scanner.c
index 5e5152b..2ed9775 100644
--- a/src/scanner.c
+++ b/src/scanner.c
@@ -407,7 +407,7
till now.
http://lists.freedesktop.org/archives/wayland-devel/2014-October/017833.html
BR
imran
On Tue, Oct 28, 2014 at 6:36 PM, Daniel Stone dan...@fooishbar.org wrote:
Hi,
On 28 October 2014 15:40, Imran Zaman imran.za...@gmail.com wrote:
You guys should check the reason why the patch
On Wed, Oct 29, 2014 at 10:09 AM, Giulio Camuffo giuliocamu...@gmail.com
wrote:
2014-10-29 8:45 GMT+02:00 Imran Zaman imran.za...@gmail.com:
Daniel!
As per your logic, I see wl_list APIs exposed etc, which shouldn't be
part
of libwayland as well.
similarly, wl_fixed_to_double
You guys should check the reason why the patch is there rather than
throwing out random thoughts or blunt comments.
I did this patch mainly because weston/wayland has been using
strtol/strtoul functions in number of places with buggy error checks, and
duplicate code everywhere. Weston and wayland
Hi
This will allow more than 1 simultaneous client connections to the server
without the possibility of connection refused error. possible use case
is multiple session
compositors can connect to the system compositor simultaneously.
diff --git a/src/wayland-server.c b/src/wayland-server.c
strtol/strtoul utility functions are used extensively in
weston/wayland, and are not bug-free in their current form.
To avoid definition in weston and wayland, its wrapped
in functions with appropriate input and output checks.
Test cases are also updated.
Signed-off-by: Imran Zaman imran.za
mentioned that),
but I understand there are reasons for them to be public and maybe in the
end it will have more pros..
Anyway, I have few nitpicks and a questions - see below.
On 16 October 2014 18:11, Imran Zaman imran.za...@gmail.com wrote:
strtol/strtoul utility functions are used extensively
of the output (e.g. its very
similar that input which has the name property attached to
it).
Signed-off-by: Imran Zaman imran.za...@gmail.com
---
protocol/wayland.xml | 22 +-
1 file changed, 21 insertions(+), 1 deletion(-)
diff --git a/protocol/wayland.xml b/protocol
the updated patch after the changes..
On Thu, Oct 16, 2014 at 4:05 AM, Bryce Harrington br...@osg.samsung.com wrote:
On Wed, Oct 15, 2014 at 04:18:59PM +0300, Imran Zaman wrote:
Hi
The patch is used to replace strtol and strtoul with wl_strtol and
wl_strtoul with inputs and result checks
strtol/strtoul utility functions are used extensively in
weston/wayland, and are not bug-free in their current form.
To avoid definition in weston and wayland, its wrapped
in functions with appropriate input and output checks.
Test cases are also updated.
Signed-off-by: Imran Zaman imran.za
This is in order to support nested compositor architectures
where system compositor using drm-backend is shared among
multiple child compositors using wayland-backend.
Signed-off-by: Imran Zaman imran.za...@gmail.com
---
src/wayland-client.c | 5 -
src/wayland-server.c | 5 -
2 files
This is used for nested compositor architectures.
Signed-off-by: Imran Zaman imran.za...@gmail.com
---
src/wayland-server.c | 27 +++
1 file changed, 27 insertions(+)
diff --git a/src/wayland-server.c b/src/wayland-server.c
index 09e8903..721fabe 100644
--- a/src/wayland
imran
commit 6be79948c9d408bb4f61cec5fff391f7ed7beb7b
Author: Imran Zaman imran.za...@intel.com
Date: Wed Oct 15 16:02:16 2014 +0300
wayland-util: added wl_strtol/wl_strtoul utility functions
The utility functions are used extensively so added appropriate
checks and test cases
Hi
The patch is used to replace strtol and strtoul with wl_strtol and
wl_strtoul with inputs and result checks.
The utility functions are used extensively in wayland and weston so added
appropriate
input and output checks; test cases are also updated; will push the patch
for weston as well.
Hi
support specifying custom directories for the client and server
sockets through environment variables.
This is in order to support nested compositor architectures where
system compositor using drm-backend is shared among multiple child
compositors using wayland-backend.
---
diff
Hi
support for adjusting socket access rights to allow group of users to
connect to the socket.
This is used for nested compositor architectures.
-
diff --git a/src/wayland-server.c b/src/wayland-server.c
index ce1eca8..b1ca5e6 100644
--- a/src/wayland-server.c
+++ b/src/wayland-server.c
@@
for this?
On Wed, Oct 15, 2014 at 6:18 AM, Imran Zaman imran.za...@gmail.com wrote:
Hi
The patch is used to replace strtol and strtoul with wl_strtol and
wl_strtoul with inputs and result checks.
The utility functions are used extensively in wayland and weston so added
appropriate
input
...@remlab.net
wrote:
Le 2014-10-15 16:14, Imran Zaman a écrit :
Hi
The patch is used to replace strtol and strtoul with wl_strtol and
wl_strtoul with inputs and result checks.
I don't know where Wayland developers stand on this, but I would rather
the client library function calls
37 matches
Mail list logo
