On Thu, Aug 28, 2014 at 03:07:26PM +0200, Hans de Goede wrote:
> Hi,
>
> On 08/27/2014 06:31 AM, Peter Hutterer wrote:
> > The big change here is the requirement to have the translation component in
> > a
> > device-normalized coordinate space. Without that, we cannot reliably rotate
> > as
> >
On Thu, Aug 28, 2014 at 03:04:30PM +0200, Hans de Goede wrote:
> Hi,
>
> On 08/27/2014 06:31 AM, Peter Hutterer wrote:
> > Signed-off-by: Peter Hutterer
> > ---
> > src/libinput-util.h | 100
> >
> > test/misc.c | 78
On Thu, Aug 28, 2014 at 03:02:33PM +0200, Hans de Goede wrote:
> Hi,
>
> On 08/27/2014 06:31 AM, Peter Hutterer wrote:
> > We apply calibration to single-touch and absolute devices, but we might as
> > well do so for multitouch events.
> >
> > Signed-off-by: Peter Hutterer
> > ---
> > src/evdev
Jasper,
I looked through your xdg-shell-rewrite branch and, other than some things
that need better docs/rewording, I think it looks OK. In particular, I
think we're probably OK removing set_unstable_version in 1.6.
Things that need attention (some of this may be a repeat of what Pekka
said):
-
Really, do we need a whole 30 seconds? You can easily do enough
interaction to crash something in 30 seconds. Other than that, this looks
fine.
Reviewed-by: Jason Ekstrand
On Thu, Aug 28, 2014 at 1:48 AM, Pekka Paalanen wrote:
> From: Pekka Paalanen
>
> If weston-desktop-shell dies soon aft
Add a new "numlock-on" option in the [keyboard] section of weston.ini
which, if set to true, is used to enable the numlock of the keyboards
attached at startup.
---
man/weston.ini.man | 6 ++
src/compositor.c | 13 +
2 files changed, 19 insertions(+)
diff --git a/man/weston.ini
This adds a function weston_keyboard_set_locks() which can be used to
change the state of the num lock and the caps locks, changing the leds too.
Only the evdev and libinput backends supports this, since it doesn't make
sense for embedded sessions.
---
src/compositor.h | 12
src/input
On 28 August 2014 17:52, Pekka Paalanen wrote:
> On Thu, 28 Aug 2014 15:59:37 +0200
> Marek Chalupa wrote:
>
> > On 28 August 2014 12:02, Pekka Paalanen wrote:
> >
> > > On Mon, 4 Aug 2014 11:30:46 +0200
> > > Marek Chalupa wrote:
> > >
> > > > From the doc comment I get the feeling, that aft
On Thu, 28 Aug 2014 15:59:37 +0200
Marek Chalupa wrote:
> On 28 August 2014 12:02, Pekka Paalanen wrote:
>
> > On Mon, 4 Aug 2014 11:30:46 +0200
> > Marek Chalupa wrote:
> >
> > > From the doc comment I get the feeling, that after successfull call to
> > > wl_display_prepare_read(), the threa
On 28 August 2014 12:02, Pekka Paalanen wrote:
> On Mon, 4 Aug 2014 11:30:46 +0200
> Marek Chalupa wrote:
>
> > From the doc comment I get the feeling, that after successfull call to
> > wl_display_prepare_read(), the thread gains exclusive access to the fd.
> > That is not true. It only ensure
Hi,
On 08/28/2014 03:12 PM, Hans de Goede wrote:
> Hi,
>
> On 08/27/2014 06:31 AM, Peter Hutterer wrote:
>> WL_CALIBRATION is a weston-specific property that required the translation
>> component of the matrix to be in pixels. libinput can't provide calibration
>> based on unknown outputs, so dro
Hi,
On 08/27/2014 06:31 AM, Peter Hutterer wrote:
> Signed-off-by: Peter Hutterer
Looks good:
Reviewed-by: Hans de Goede
> ---
> See patch 08/11, not sure we really want that configuration back-channel.
I think it is useful to have.
Regards,
Hans
>
> src/evdev.c | 19 --
Hi,
On 08/27/2014 06:31 AM, Peter Hutterer wrote:
> New configuration API:
> libinput_device_config_calibration_has_matrix()
> libinput_device_config_calibration_set_matrix()
> libinput_device_config_calibration_get_matrix()
> libinput_device_config_calibration_get_default_
Hi,
On 08/27/2014 06:31 AM, Peter Hutterer wrote:
> Basic tests for rotation, translation and scaling events.
>
> Note that tests need to be added separately for single-touch and touch
> devices, this is a restriction of the litest framework.
>
> Signed-off-by: Peter Hutterer
Looks good:
Revi
Hi,
On 08/27/2014 06:31 AM, Peter Hutterer wrote:
> WL_CALIBRATION is a weston-specific property that required the translation
> component of the matrix to be in pixels. libinput can't provide calibration
> based on unknown outputs, so drop support for this property.
>
> The basic functionality i
Hi,
On 08/27/2014 06:31 AM, Peter Hutterer wrote:
> The big change here is the requirement to have the translation component in a
> device-normalized coordinate space. Without that, we cannot reliably rotate as
> the coordinate space is effectively unknown and may differ between the axes.
> This a
Hi,
On 08/27/2014 06:31 AM, Peter Hutterer wrote:
> Signed-off-by: Peter Hutterer
> ---
> src/libinput-util.h | 100
>
> test/misc.c | 78
> 2 files changed, 178 insertions(+)
>
> diff --git
Hi,
On 08/27/2014 06:31 AM, Peter Hutterer wrote:
> We apply calibration to single-touch and absolute devices, but we might as
> well do so for multitouch events.
>
> Signed-off-by: Peter Hutterer
> ---
> src/evdev.c | 31 +++
> 1 file changed, 19 insertions(+), 12 d
Hi,
On 08/27/2014 06:31 AM, Peter Hutterer wrote:
> Signed-off-by: Peter Hutterer
Looks good:
Reviewed-by: Hans de Goede
Regards,
Hans
> ---
> src/evdev.c | 3 ++-
> src/evdev.h | 3 ++-
> 2 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/src/evdev.c b/src/evdev.c
> index
Hi,
On 08/27/2014 06:31 AM, Peter Hutterer wrote:
> We have a wrapper, use it.
>
> Signed-off-by: Peter Hutterer
Looks good:
Reviewed-by: Hans de Goede
Regards,
Hans
> ---
> src/udev-seat.c | 27 ++-
> 1 file changed, 14 insertions(+), 13 deletions(-)
>
> diff --
Hi,
On 08/27/2014 06:31 AM, Peter Hutterer wrote:
> With a non-zero absmin for both axes and different ranges for x/y, just to
> detect those errors.
>
> Signed-off-by: Peter Hutterer
Looks good:
Reviewed-by: Hans de Goede
Regards,
Hans
> ---
> test/Makefile.am | 1 +
>
Hi,
On 08/27/2014 06:31 AM, Peter Hutterer wrote:
> litest_wait_for_event() returns if any event is available.
> litest_wait_for_event_of_type(... type, type, type, -1) returns if any of the
> given event types is availble. All other events are discarded.
>
> Signed-off-by: Peter Hutterer
Looks
cairo-util has used sans font family for title fonts so far.
But sans doesn't support non-ascii characters.
So now let users choose a font family for titles.
The entry name is generic, 'font' so that other places
where draw texts can use this entry too.
Signed-off-by: Ryo Munakata
---
man/westo
On Thu, 28 Aug 2014 12:12:59 +0300
Pekka Paalanen wrote:
> On Fri, 22 Aug 2014 13:22:27 +0300
> Giulio Camuffo wrote:
>
> > This adds a function weston_keyboard_set_locks() which can be used to
> > change the state of the num lock and the caps locks, changing the leds too.
> > Only the evdev an
On Mon, 25 Aug 2014 20:56:52 +0200
raw...@gmail.com wrote:
> From: Arnaud Vrac
>
> ---
> src/pixman-renderer.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/src/pixman-renderer.c b/src/pixman-renderer.c
> index a816d02..4fdcb05 100644
> --- a/src/pixman-renderer.c
> +++ b/src/pix
On Mon, 25 Aug 2014 20:56:51 +0200
raw...@gmail.com wrote:
> From: Arnaud Vrac
>
> ---
> clients/keyboard.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/clients/keyboard.c b/clients/keyboard.c
> index 7c11cec..750f58e 100644
> --- a/clients/keyboard.c
> +++ b/clients/keyboard.c
>
On Mon, 25 Aug 2014 20:56:50 +0200
raw...@gmail.com wrote:
> From: Arnaud Vrac
>
> ---
> tests/weston-test.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/tests/weston-test.c b/tests/weston-test.c
> index f1e45c1..5c6fd47 100644
> --- a/tests/weston-test.c
> +++ b/tests/weston-test.
Hmm, thinking about that... Maybe this is not the right solution.
Programmer should probably do something like:
do {
ret = wl_display_read_events(display);
} while (ret == 0 && errno == EAGAIN);
to be sure he read the events (or got an error). This patch would force him
to re-run wl_display_pre
On Thu, 28 Aug 2014 18:35:27 +0900
Ryo Munakata wrote:
> Signed-off-by: Ryo Munakata
> ---
> clients/multi-resource.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/clients/multi-resource.c b/clients/multi-resource.c
> index c4a0c18..0dc2c74 100644
> --- a/clients/multi-resource.c
>
On Mon, 4 Aug 2014 11:30:46 +0200
Marek Chalupa wrote:
> From the doc comment I get the feeling, that after successfull call to
> wl_display_prepare_read(), the thread gains exclusive access to the fd.
> That is not true. It only ensures that _one_ of the threads, that called
> this function, wi
Hi,
On 28 August 2014 11:30, Pekka Paalanen wrote:
> On Wed, 16 Jul 2014 15:52:27 +0200
> Marek Chalupa wrote:
>
> > Hi,
> >
> > in this series of patches I added support for xdg-shell/surface
> > to the test-suite.
> >
> > The thing that led me to this is captured in the last patch, that is
>
I found another one in multi-resouce.
Patch sent.
Thanks.
On Thu, 28 Aug 2014 11:59:43 +0300
Pekka Paalanen wrote:
> On Tue, 26 Aug 2014 21:25:22 +0200
> "Nils Chr. Brause" wrote:
>
> > Since wl_display_get_fd() doesn't have any side effects, this is truly
> > useless.
> >
> > Reviewed-by: N
Signed-off-by: Ryo Munakata
---
clients/multi-resource.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/clients/multi-resource.c b/clients/multi-resource.c
index c4a0c18..0dc2c74 100644
--- a/clients/multi-resource.c
+++ b/clients/multi-resource.c
@@ -258,8 +258,6 @@ create_display(void)
In the first place, create a function display_wakeup_threads that
contains always-repeated pattern: broadcast + increase serial.
Replace the pattern with this function and call this function also
on return from read_events when wl_connection_read fails with
EAGAIN (this solves the bug from the tes
This test creates three threads (and main one) and in each does
preapare_read and read_events. This is normal workflow, but
since we have no data on socket, the read_events will return
with EAGAIN. On this path we do not wake up threads and
this test blocks until timeout.
Signed-off-by: Marek Chal
On Wed, 16 Jul 2014 15:52:27 +0200
Marek Chalupa wrote:
> Hi,
>
> in this series of patches I added support for xdg-shell/surface
> to the test-suite.
>
> The thing that led me to this is captured in the last patch, that is
> that maximizing and fullscreening didn't work as expected for really
On Fri, 22 Aug 2014 11:48:59 +0300
Giulio Camuffo wrote:
> Add a new "numlock-on" option in the [keyboard] section of weston.ini
> which, if set to true, is used to enable the numlock of the keyboards
> attached at startup.
> ---
> man/weston.ini.man | 6 ++
> src/compositor.c | 13 ++
On Fri, 22 Aug 2014 13:22:27 +0300
Giulio Camuffo wrote:
> This adds a function weston_keyboard_set_locks() which can be used to
> change the state of the num lock and the caps locks, changing the leds too.
> Only the evdev and libinput backends supports this, since it doesn't make
> sense for em
On Tue, 26 Aug 2014 21:25:22 +0200
"Nils Chr. Brause" wrote:
> Since wl_display_get_fd() doesn't have any side effects, this is truly
> useless.
>
> Reviewed-by: Nils Chr. Brause
>
>
>
> On Tue, Aug 26, 2014 at 2:59 PM, Ryo Munakata wrote:
>
> > ---
> > clients/simple-shm.c | 2 --
> > 1
From: Pekka Paalanen
If weston-desktop-shell dies soon after launch, or maybe cannot be
executed at all, let weston exit rather than letting the user stare at a
black screen.
But, do not exit weston, if weston-desktop-shell dies later, as the user
may already have apps open, and those apps would
On Wed, 27 Aug 2014 13:16:56 -0500
Derek Foreman wrote:
> On 27/08/14 05:41 AM, Pekka Paalanen wrote:
> > From: Pekka Paalanen
> >
> > The desktop shell plugin registers both a wl_client destroy signal
> > listener, and a sigchld handler, when launching weston-desktop-shell.
> > However, nothin
On Wed, 27 Aug 2014 11:31:18 -0500
Derek Foreman wrote:
> On 27/08/14 05:41 AM, Pekka Paalanen wrote:
> > From: Pekka Paalanen
> >
> > weston_client_start() is a new wrapper around weston_client_launch(),
> > that does the process tracking on its own, and logs the process exit
> > status.
> >
42 matches
Mail list logo