Re: [PATCH weston] weston-resizor: Don't add new frame callbacks every click

2017-02-17 Thread Jonas Ådahl
On Wed, Feb 08, 2017 at 03:05:54PM -0600, Derek Foreman wrote: > The frame callback added on button click re-adds itself when done, > so adding a new one every click resulted in an ever increasing > number of callbacks. > > Signed-off-by: Derek Foreman > --- > clients/resizor.c | 5 + > 1 fi

Re: [PATCH weston] Update COPYING

2017-02-17 Thread Bryce Harrington
On Fri, Feb 17, 2017 at 05:19:55PM +, Daniel Stone wrote: > Hi, > > On 17 February 2017 at 17:12, Bryce Harrington wrote: > > On Fri, Feb 17, 2017 at 07:50:45AM -0600, Derek Foreman wrote: > >> How about we just remove all the copyrights from COPYING and let > >> anyone that needs to collate

Student project

2017-02-17 Thread Sergi Granell
Hi everyone! My name is Sergi Granell and I am a last year student at the Facultat d'Informàtica de Barcelona in the Universitat Politècnica de Catalunya [1]. As this is my last semester there, I have to do my end of degree project. I have been interested in Wayland for a long time, and also follo

Re: [PATCH weston] Update COPYING

2017-02-17 Thread Daniel Stone
Hi, On 17 February 2017 at 17:12, Bryce Harrington wrote: > On Fri, Feb 17, 2017 at 07:50:45AM -0600, Derek Foreman wrote: >> How about we just remove all the copyrights from COPYING and let >> anyone that needs to collate this list do it themselves... >> >> That way we don't need to remember to

Re: [PATCH weston] Update COPYING

2017-02-17 Thread Bryce Harrington
On Fri, Feb 17, 2017 at 07:50:45AM -0600, Derek Foreman wrote: > On 17/02/17 07:47 AM, Pekka Paalanen wrote: > >On Mon, 13 Feb 2017 21:45:58 + > >Daniel Stone wrote: > > > >>Include a list of every copyright statement within the COPYING file. > >> > >>Signed-off-by: Daniel Stone > >>--- > >>

Re: [PATCH weston 4/7] Calculate next-frame target time in absolute space

2017-02-17 Thread Daniel Stone
Hi, On 16 February 2017 at 10:23, Pekka Paalanen wrote: > On Tue, 14 Feb 2017 13:18:04 + > Daniel Stone wrote: >> @@ -2389,34 +2390,35 @@ weston_output_finish_frame(struct weston_output >> *output, >> presented_flags); >> >> output->fram

Re: [PATCH weston 1/7] timespec: Add timespec_add_nsec helper

2017-02-17 Thread Daniel Stone
Hi, On 14 February 2017 at 14:09, Pekka Paalanen wrote: > On Tue, 14 Feb 2017 13:18:01 + > Daniel Stone wrote: >> +ZUC_TEST(timespec_test, timespec_add_nsec) >> +{ >> + struct timespec a, r; > > An array and a loop to ease readability? :-) I had a flash of doing this, and remembered why

Re: [PATCH weston 5/7] Don't delay initial output paint

2017-02-17 Thread Daniel Stone
Hi, On 17 February 2017 at 14:22, Pekka Paalanen wrote: > On Thu, 16 Feb 2017 14:53:39 + > Daniel Stone wrote: >> I was about to say 'this doesn't gain us anything', except if all >> clients do submit and trigger repaint immediately, there's a 7ms grace >> period where we could potentially g

Re: [PATCH weston 5/7] Don't delay initial output paint

2017-02-17 Thread Pekka Paalanen
On Thu, 16 Feb 2017 14:53:39 + Daniel Stone wrote: > Hi, > > On 16 February 2017 at 14:31, Pekka Paalanen wrote: > > On Tue, 14 Feb 2017 13:18:05 + > > Daniel Stone wrote: > >> On startup, we cannot lock on to the repaint timer because it is unknown > >> to us. We deal with this by c

Re: [PATCH weston] terminal: Fix crash due to race condition in init

2017-02-17 Thread Mike Blumenkrantz
Hi, Reverting this causes a 100% reproducible crash on startup in one of my CI tests for weston-terminal. Here is some of the debugging info: Program terminated with signal SIGSEGV, Segmentation fault. #0 0x0040ab93 in handle_char (terminal=0x6348a0, utf8=...) at clients/terminal.c:1991

Re: [PATCH weston v6] compositor-drm: Ignore non-KMS devices

2017-02-17 Thread Pekka Paalanen
On Fri, 17 Feb 2017 07:43:24 + "Ucan, Emre (ADITG/SW1)" wrote: > Hi Daniel, > > It would be great to have this patch in weston 2.0. > We made a similar patch to fix this issue for Renesas Gen3 boards. Hi Emre, it seems too risky to land now when we are in "release critical bug fixes only"

Re: [PATCH weston] Update COPYING

2017-02-17 Thread Derek Foreman
On 17/02/17 07:47 AM, Pekka Paalanen wrote: On Mon, 13 Feb 2017 21:45:58 + Daniel Stone wrote: Include a list of every copyright statement within the COPYING file. Signed-off-by: Daniel Stone --- COPYING | 37 - 1 file changed, 32 insertions(+), 5 del

Re: [PATCH weston] Update COPYING

2017-02-17 Thread Pekka Paalanen
On Mon, 13 Feb 2017 21:45:58 + Daniel Stone wrote: > Include a list of every copyright statement within the COPYING file. > > Signed-off-by: Daniel Stone > --- > COPYING | 37 - > 1 file changed, 32 insertions(+), 5 deletions(-) > > diff --git a/COPYING