On 05/19/2015 09:40 PM, Christopher Larson wrote:

On Tue, May 19, 2015 at 3:47 AM, Richard Purdie <richard.purdie@linuxfoundation.org> wrote:
On Tue, 2015-05-19 at 16:29 +0800, Chen Qi wrote:
> Add eventmask for buildstats_summary handler.

Why?

This says what you've done but not why we need this...

Cheers,

Richard

Also, this adds the eventmask, but doesn’t remove the then-redundant isinstance() on the event object.

Hi Richard and Christopher,

Thanks for you advice.
I'll send out V2.

Regards,
Chen Qi

--
Christopher Larson
clarson at kergoth dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Maintainer - Tslib
Senior Software Engineer, Mentor Graphics