On Thu, 14 Nov 2013 12:27:22 -0800
Bill Spitzak spit...@gmail.com wrote:
Pekka Paalanen wrote:
Yes, you cannot use non-integer surface sizes. You cannot express a
surface size, that would result in a 1024x1024 area in output pixels,
because 1024 is not divisible by output_scale=3.
I have tested your patch, but it doesn't solve all the bugs occuring in
XWayland because of views (take vlc, go to the menu, crash).
It appears ok to me to set view to NULL at these locations, but there's
probably something more to do.
Axel Davy
On 15/11/2013, Dima Ryazanov wrote :
Ping :)
Oh interesting... I can fix the crash by checking for a NULL pointer,
though I don't know if that's the proper fix. Anyways, I'll send out the
new patches.
On Fri, Nov 15, 2013 at 12:44 AM, Axel Davy d...@clipper.ens.fr wrote:
I have tested your patch, but it doesn't solve all the bugs
Fixes a crash caused by accessing a deleted view in
weston_wm_window_schedule_repaint. It can be easily reproduced by switching
between menus in Firefox.
Signed-off-by: Dima Ryazanov d...@gmail.com
---
src/xwayland/window-manager.c | 2 ++
1 file changed, 2 insertions(+)
diff --git
This fixes crashes caused by popup windows that don't have override_redirect
(e.g., menus in VLC and KDE apps), though I don't know if this is correct.
Signed-off-by: Dima Ryazanov d...@gmail.com
---
src/xwayland/window-manager.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff
Fixes a crash in Firefox when clicking an install plugin popup.
Signed-off-by: Dima Ryazanov d...@gmail.com
---
src/xwayland/window-manager.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/xwayland/window-manager.c b/src/xwayland/window-manager.c
index
I've looked deeply in the code to find the remaining xwayland bugs.
I'll publish later a fix for these. The remaining bugs are due to
accessing to the frame field on
unmapped windows.
Your patch solves all the issues with the view field.
Your patch is tested and
Reviewed-by: Axel Davy
You were faster than me!
I was just about sending this very same patch.
The patch is correct as I was explaining in my commit message:
XWayland: Don't access the frame field for unmapped windows.
There are situations where weston_wm_window_get_frame_size
and
This patch (Again, I had the same), solves the last bugs for XWayland,
but for this one, I'm not sure it is the correct fix.
In fact it would mean than focus_window would be an unmapped windows,
which looks strange to me.
Axel Davy
Le 15/11/2013 11:01, Dima Ryazanov a écrit :
Fixes a crash in
Hi all,
this is a reply to the topic of whether to have information about decorations
in the xdg_shell at all (see [1]). Sorry that I don't reply to the right
message, had not been subscribed to the list.
I want to outline why we in the KDE Plasma team would prefer to have the
possibility to
On Fri, Nov 15, 2013 at 6:17 AM, Martin Gräßlin mgraess...@kde.org wrote:
Hi all,
this is a reply to the topic of whether to have information about
decorations
in the xdg_shell at all (see [1]). Sorry that I don't reply to the right
message, had not been subscribed to the list.
I want to
An unmapped window shouldn't be the input focus.
This solves some remaining Weston crashes with XWayland,
because we assume wm-focus_window has a frame.
Signed-off-by: Axel Davy axel.d...@ens.fr
---
src/xwayland/window-manager.c | 2 ++
1 file changed, 2 insertions(+)
diff --git
I've posted a patch, which prevents wm-focus_window to be a window
without frame.
This solves the same bug than this patch, but looks better to me.
http://lists.freedesktop.org/archives/wayland-devel/2013-November/012008.html
Axel Davy
Le 15/11/2013 11:25, Axel Davy a écrit :
This patch
Jasper St. Pierre wrote:
On Fri, Nov 15, 2013 at 6:17 AM, Martin Gräßlin
mgraess...@kde.org wrote:
snip, because gmane
I think the trap that I personally don't want to run into is the case
where we have a compositor that doesn't support SSD coupled to a client
that doesn't support CSD.
On Friday 15 November 2013 07:50:55 Jasper St. Pierre wrote:
Ah, whoops, I read your email wrong. It sounds like you're also in favor of
requiring CSD support from the client.
Yep, I'm fine with ensuring that CSD is the required fallback. As said: I don't
want to force anybody to SSD :-)
Otherwise the tail of fds_in buffer would just shift beyond the beginning.
That confuses the actual request handler and results in a crash further on
due to corrupted tail.
Signal the lack of file descriptor with -1, so that the request handler
can determine that no valid file descriptor was
This is just for the sake of cleanliness and correct error handling.
Without this the mmap would fail emitting an error message about failed
mapping to the client and a close of bogus file descriptor would be
attempted, which is harmless since it can not fail.
Signed-off-by: Lubomir Rintel
This allows specifying a particular backend to load in a manner similar to
modules.
Signed-off-by: Lubomir Rintel lkund...@v3.sk
---
man/weston.ini.man | 17 +
src/compositor.c | 24 +++-
2 files changed, 32 insertions(+), 9 deletions(-)
diff --git
Hi,
Amed assrs a result of previous mailing I've merged
wayland-integration-tests into wayland. And, with a tear in my eye, renaead
and asswrite xD (actually, I renamed all functions. I removed prefixes).
Here's repo with the first version:
https://github.com/mchalupa/wayland/tree/wits
Regards,
On Mon, 21 Oct 2013 11:09:46 +0200
Marek Ch mchqwe...@gmail.com wrote:
Hi,
I have got one question. In documentation of wl_registry's global_remove
event is:
This event notifies the client that the global identified by name is no
longer available. If the client bound to the global using
As per Kristian's suggestion we can avoid the problem of effectively
disabling the event queuing mechanism by only doing the sync request
when the swap interval is zero. To fix the bug of using three buffers
we can just block for the frame callback in get_back_bo instead of
swap_buffers. I was
The Wayland EGL platform now respects the eglSwapInterval value. The value is
clamped to either 0 or 1 because it is difficult (and probably not useful) to
sync to more than 1 redraw.
The main change is that if the swap interval is 0 then Mesa won't install a
frame callback so that eglSwapBuffers
On 11/15/2013 01:50 PM, Jasper St. Pierre wrote:
Ah, whoops, I read your email wrong. It sounds like you're also in favor
of requiring CSD support from the client.
That said, I'm curious about cases like Nautilus under SSD. Should we
hide the close button, or do anything like that to make it
From: Pekka Paalanen pekka.paala...@collabora.co.uk
The sub-surface protocol was originally committed into Weston on May
10th, 2013, in commit 2396aec6842c709a714f3825dbad9fd88478f2e6. The
design for the protocol had started in the beginning of December 2012. I
think it is high time to move this
From: Pekka Paalanen pekka.paala...@collabora.co.uk
This reverts commit 2396aec6842c709a714f3825dbad9fd88478f2e6.
This exact version of the sub-surface protocol has been copied into
Wayland core. Therefore it must be removed from here to avoid build
conflicts and useless duplication.
No other
I'm not sure waiting 10ms when all the buffers aren't released is the
best solution.
In fact, I experimented a eglSwapInterval(0) behaviour with a simple
patch to enable it:
having 4 back buffers and using a roundtrip after commiting (that is: a
bit similar to what you do, but without the
Got the chorme-ozone working for ARM with proprietary GPU drivers. It is
neat!
WebGL also working very good.
I recommend to use gold linker and latest ninja(1.4).
I have to make only the following change
@@ -117,7 +117,7 @@ bool GLSurfaceEGL::InitializeOneOff() {
LOG(ERROR) OZONE failed
Is libinput supposed to be sort of replacement for libxcbcommon?
Thanks.
On Thu, Nov 14, 2013 at 4:58 PM, Artsiom Anikeyenka
arty.ani...@gmail.comwrote:
I like the generalized architecture of any input handling as well. I
didn't look into the code because I will hardly understand it yet. I
If by libxcbcommon you mean libxkbcommon, no.
libxkbcommon is about the complex ways of interpreting key events, with
client-side key maps, modifiers, accessibility features (latched keys,
sticky keys, etc.). It's meant to be used from both the compositor and the
client.
libinput is about
Thanks! That's strange... I mean... It sounds like there are extra levels
of responsibility. What I would expect is one way flow with 1 component
processing all the input. 1 person will hardly deal with more then 1 output
(though it may use several inputs at a time). That's y the concept of 1
It's because the keyboard is a surprisingly complicated tool with multiple
modes of operating, probably without you ever realizing it, and clients
need all of that power:
If I'm typing text, the *labels* of the keys matter, and input methods
allow us to translate sequences of key presses and
From: Daniel Stone dan...@fooishbar.org
Add the ability to bind to modifiers; the binding is armed when a key
which sets the requested modifier is pressed, and triggered if the key
is released with no other keys having been pressed in the meantime, as
well as mouse buttons or scroll axes.
This
From: Louis-Francis Ratté-Boulianne l...@collabora.com
When enabled, this will make all but the keyboard-focused window dim.
Also the background gets dimmed, if there are any windows open. The
panel is not dimmed.
When the keyboard focus changes, the change in dimming is animated.
The dimming
The following series adds exposay as previously submitted on the list
but rebased on master. It also adds an alt-tab switcher which is a
nice demonstration of weston_views. In addition there's a bug fix
in the input code that is easily triggered by exposay.
Daniel Stone (3):
Add modifier-only
This causes the surface to get the keyboard focus, which in turn
causes focus-animation to nicely work with exposay, making the
not focused surfaces to be dimmed.
Signed-off-by: Emilio Pozuelo Monfort emilio.pozu...@collabora.co.uk
---
src/shell.c | 3 +--
1 file changed, 1 insertion(+), 2
From: Daniel Stone dan...@fooishbar.org
Exposay provides window overview functions which, when a key which
produces the binding modifier is pressed on its own, scales all
currently-open windows down to be shown overlaid on the desktop,
providing keyboard and mouse navigation to be able to switch
with the previous patch the resource isn't used inside weston_surface_destroy()
anymore (aside sending events unuseful for a closing client), so we can safely
reset it.
Reviewed-by: Jason Ekstrand ja...@jlekstrand.net
---
src/compositor.c | 4
1 file changed, 4 insertions(+)
diff --git
On Thu, Nov 14, 2013 at 12:28:51PM +, Neil Roberts wrote:
It's not obvious that these functions are needed so it would be good
to have some documentation for them.
---
doc/doxygen/Makefile.am | 3 ++-
src/wayland-shm.c | 64
+
2
On Thu, Nov 14, 2013 at 11:42:51PM +0100, Giulio Camuffo wrote:
---
src/compositor.h | 3 +++
src/input.c | 8
src/shell.c | 3 +--
src/zoom.c | 27 +++
4 files changed, 31 insertions(+), 10 deletions(-)
That is nice, I like it a lot.
On Thu, Nov 14, 2013 at 11:42:54PM +0100, Giulio Camuffo wrote:
when a fullscreen surface hides the cursor, such as mpv, we must
make sure that the cursor gets hidden also when it is over the
underlying black surface. so check if the surface under the pointer
is a black surface and focus on
I agree. I wondered indeed if it made more sense to remove the default
implementations in input.c and mandate all the shells to have a
default grab.
I'm not sure how a flag would work given it not only does need to not
pick a surface, but also pick another one. A don't pick and pick its
On Fri, Nov 15, 2013 at 02:18:15PM +0100, Lubomir Rintel wrote:
This allows specifying a particular backend to load in a manner similar to
modules.
Signed-off-by: Lubomir Rintel lkund...@v3.sk
Yes, that's a good addition, committed.
Kristian
---
man/weston.ini.man | 17 +
On Fri, Nov 15, 2013 at 02:17:54PM +0100, Lubomir Rintel wrote:
Otherwise the tail of fds_in buffer would just shift beyond the beginning.
That confuses the actual request handler and results in a crash further on
due to corrupted tail.
Signal the lack of file descriptor with -1, so that the
On Fri, Nov 15, 2013 at 02:17:55PM +0100, Lubomir Rintel wrote:
This is just for the sake of cleanliness and correct error handling.
Without this the mmap would fail emitting an error message about failed
mapping to the client and a close of bogus file descriptor would be
attempted, which is
On Fri, Nov 15, 2013 at 02:17:56PM +0100, Lubomir Rintel wrote:
It would be possible to make the compositor leak file descriptors by
passing descriptors of open unmmapable files to it, such as /dev/null.
Signed-off-by: Lubomir Rintel lkund...@v3.sk
---
src/wayland-shm.c | 2 +-
1 file
The server requires clients to only allocate one ID ahead of the previously
highest ID in order to keep the ID range tight. Failure to do so will
make the server close the client connection. However, the way we allocate
new IDs is racy. The generated code looks like:
new_proxy =
On Fri, Nov 15, 2013 at 04:12:13PM +0200, ppaala...@gmail.com wrote:
From: Pekka Paalanen pekka.paala...@collabora.co.uk
This reverts commit 2396aec6842c709a714f3825dbad9fd88478f2e6.
This exact version of the sub-surface protocol has been copied into
Wayland core. Therefore it must be
47 matches
Mail list logo