[OE-core] [PATCH 1/1] buildstats-summary: add eventmask

ChenQi Qi.Chen at windriver.com
Wed May 20 02:17:52 UTC 2015


On 05/19/2015 09:40 PM, Christopher Larson wrote:
>
> On Tue, May 19, 2015 at 3:47 AM, Richard Purdie 
> <richard.purdie at linuxfoundation.org 
> <mailto:richard.purdie at 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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20150520/6ec77fa7/attachment-0002.html>


More information about the Openembedded-core mailing list