Messages in this thread | | | From | Namhyung Kim <> | Subject | Re: [PATCH 06/17] tools lib traceevent: Add pevent_unregister_event_handler() | Date | Thu, 16 Jan 2014 10:58:05 +0900 |
| |
On Wed, 15 Jan 2014 20:52:59 -0500, Steven Rostedt wrote: > On Thu, 16 Jan 2014 09:00:28 +0900 > Namhyung Kim <namhyung.kim@lge.com> wrote: > >> Hi Steve, >> >> 2014-01-16 AM 1:18, Steven Rostedt wrote: >> > On Wed, 15 Jan 2014 10:45:29 +0900 >> > Namhyung Kim <namhyung@kernel.org> wrote: >> >> + if (event->handler == func && event->context == context) { >> >> + pr_stat("overriding event (%d) %s:%s with default print handler", >> > >> > Should we use the word "overriding" again? Perhaps saying: >> > >> > "removing override print handler (%d) %s:%s. Going back to default handler." >> > >> > Or something? >> >> What about this: >> >> "removing override handler for event (%d) %s:%s. Going back to default >> handler." >> > > Sounds good.
Thanks, can I add your ack with this change?
Namhyung
| |