<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 19, 2015 at 3:47 AM, Richard Purdie <span dir="ltr">&lt;<a href="mailto:richard.purdie@linuxfoundation.org" target="_blank">richard.purdie@linuxfoundation.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, 2015-05-19 at 16:29 +0800, Chen Qi wrote:<br>
&gt; Add eventmask for buildstats_summary handler.<br>
<br>
</span>Why?<br>
<br>
This says what you&#39;ve done but not why we need this...<br>
<br>
Cheers,<br>
<br>
Richard</blockquote></div><br>Also, this adds the eventmask, but doesn’t remove the then-redundant isinstance() on the event object.<br>-- <br><div class="gmail_signature">Christopher Larson<br>clarson at kergoth dot com<br>Founder - BitBake, OpenEmbedded, OpenZaurus<br>Maintainer - Tslib<br>Senior Software Engineer, Mentor Graphics</div>
</div></div>