Re: [PATCH wayland] add apis to enable/disable wayland debug at runtime

2015-12-07 Thread Marek Chalupa
On 12/07/2015 01:05 PM, Pekka Paalanen wrote: On Mon, 7 Dec 2015 11:21:15 +0100 Marek Chalupa wrote: Hi, I like this idea, it can be handy when debugging just a part of code (I wrote similar patch once actually, but then I forgot about it and never sent it to the list). However, the impleme

Re: [PATCH wayland] add apis to enable/disable wayland debug at runtime

2015-12-07 Thread Pekka Paalanen
On Mon, 7 Dec 2015 11:21:15 +0100 Marek Chalupa wrote: > Hi, > > I like this idea, it can be handy when debugging just a part of code (I > wrote similar patch once actually, but then I forgot about it and never > sent it to the list). However, the implementation could be better IMHO: > > I do

Re: [PATCH wayland] add apis to enable/disable wayland debug at runtime

2015-12-07 Thread Marek Chalupa
Hi, I like this idea, it can be handy when debugging just a part of code (I wrote similar patch once actually, but then I forgot about it and never sent it to the list). However, the implementation could be better IMHO: I don't think we should move the variables to wayland-util.c. You don't

RE: [PATCH wayland] add apis to enable/disable wayland debug at runtime

2015-11-13 Thread 박성진
Sorry, I attached the patch at the end of the following email. :) Thanks, Sung-Jin Park -Original Message- From: 박성진 [mailto:sj76.p...@samsung.com] Sent: Friday, November 13, 2015 7:33 PM To: wayland-devel@lists.freedesktop.org Cc: Sung-Jin Park/Common Platform Lab(S/W Center)/S5/Samsung