<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">On 05/19/2015 09:40 PM, Christopher
Larson wrote:<br>
</div>
<blockquote
cite="mid:CABcZAN=Ky_TKqC8y3ecYh7Mf_SkQYJ0UUvDXMHnQCKZBpW+1KA@mail.gmail.com"
type="cite">
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<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"><<a
moz-do-not-send="true"
href="mailto:richard.purdie@linuxfoundation.org"
target="_blank">richard.purdie@linuxfoundation.org</a>></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>
> Add eventmask for buildstats_summary handler.<br>
<br>
</span>Why?<br>
<br>
This says what you'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>
</div>
</div>
</blockquote>
<br>
Hi Richard and Christopher,<br>
<br>
Thanks for you advice.<br>
I'll send out V2.<br>
<br>
Regards,<br>
Chen Qi<br>
<br>
<blockquote
cite="mid:CABcZAN=Ky_TKqC8y3ecYh7Mf_SkQYJ0UUvDXMHnQCKZBpW+1KA@mail.gmail.com"
type="cite">
<div dir="ltr">
<div class="gmail_extra">-- <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>
</blockquote>
<br>
</body>
</html>