Re: [Qemu-devel] [PATCH 1/6] trace: [tracetool] Do not rebuild event list in backend code

2012-01-18 Thread Lluís Vilanova
Lluís Vilanova writes: > Harsh Bora writes: >> Hi Lluis, >> This looks really nice. I can include your fixes while addressing other >> review >> comments. Shall I fold your patches with mine or do you want to keep them >> separate? > Whatever works best for you. BTW, I did some more changes, b

Re: [Qemu-devel] [PATCH 1/6] trace: [tracetool] Do not rebuild event list in backend code

2012-01-18 Thread Lluís Vilanova
Harsh Bora writes: > On 01/11/2012 11:35 PM, Lluís Vilanova wrote: >> Signed-off-by: Lluís Vilanova >> --- >> scripts/tracetool.py | 14 +++--- >> 1 files changed, 7 insertions(+), 7 deletions(-) >> >> diff --git a/scripts/tracetool.py b/scripts/tracetool.py >> index 6874f66..80e5684 100

Re: [Qemu-devel] [PATCH 1/6] trace: [tracetool] Do not rebuild event list in backend code

2012-01-18 Thread Harsh Bora
On 01/11/2012 11:35 PM, Lluís Vilanova wrote: Signed-off-by: Lluís Vilanova --- scripts/tracetool.py | 14 +++--- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/scripts/tracetool.py b/scripts/tracetool.py index 6874f66..80e5684 100755 --- a/scripts/tracetool.py +++ b/

[Qemu-devel] [PATCH 1/6] trace: [tracetool] Do not rebuild event list in backend code

2012-01-11 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova --- scripts/tracetool.py | 14 +++--- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/scripts/tracetool.py b/scripts/tracetool.py index 6874f66..80e5684 100755 --- a/scripts/tracetool.py +++ b/scripts/tracetool.py @@ -171,15 +171,14 @@ def si