<div dir="ltr"><div class="gmail_default" style="font-family:tahoma,sans-serif">And taken in for the submission</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Mar 10, 2014 at 6:03 PM, Barros Pena, Belen <span dir="ltr"><<a href="mailto:belen.barros.pena@intel.com" target="_blank">belen.barros.pena@intel.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 10/03/2014 17:25, "Ravi Chintakunta" <<a href="mailto:ravi.chintakunta@timesys.com">ravi.chintakunta@timesys.com</a>><br>


wrote:<br>
<div class=""><br>
>Hi Belen,<br>
><br>
>Thanks for the quick review. I have pushed my changes to the<br>
>ravi/bug-fixes branch.<br>
<br>
</div>Looking good for me.<br>
<br>
Thanks!<br>
<br>
Belén<br>
<div class="HOEnZb"><div class="h5"><br>
><br>
>-Ravi<br>
><br>
>> Â -----Original Message-----<br>
>> Â From: Barros Pena, Belen [mailto:<a href="mailto:belen.barros.pena@intel.com">belen.barros.pena@intel.com</a>]<br>
>> Â Sent: Monday, March 10, 2014 1:03 PM<br>
>> Â To: Ravi Chintakunta; '<a href="mailto:toaster@yoctoproject.org">toaster@yoctoproject.org</a>'<br>
>> Â Subject: Re: [Toaster] review request for counters for filters<br>
>><br>
>> Â On 10/03/2014 16:41, "Ravi Chintakunta" <<a href="mailto:ravi.chintakunta@timesys.com">ravi.chintakunta@timesys.com</a>><br>
>> Â wrote:<br>
>><br>
>> Â >I have pushed the implementation for adding counters to filters<br>
>> Â ><br>
>> Â ><a href="https://bugzilla.yoctoproject.org/show_bug.cgi?id=5821" target="_blank">https://bugzilla.yoctoproject.org/show_bug.cgi?id=5821</a><br>
>> Â ><br>
>> Â >to my branch: ravi/bug-fixes<br>
>> Â ><br>
>> Â ><a href="http://git.yoctoproject.org/cgit/cgit.cgi/poky-" target="_blank">http://git.yoctoproject.org/cgit/cgit.cgi/poky-</a><br>
>> Â contrib/commit/?h=ravi/bug-<br>
>> Â >fixes&id=6da9e0124b2e749755714d8d8320cfd3d7884f5c<br>
>><br>
>> Â This is pretty neat functionality :) Just 2 tiny things:<br>
>><br>
>> Â * When the counter is 0, we are missing a space between the end of the<br>
>> Â option name and the opening parenthesis of the counter<br>
>> Â * When the filter is applied on top of search results, we are missing<br>
>> Â ' '<br>
>> Â around the search string<br>
>><br>
>> Â ><br>
>> Â >Please note that the counts for the filters in config variables is<br>
>> Â not<br>
>> Â >adding up to the total count. Looks like there are config variables<br>
>> Â that<br>
>> Â >are set in other files and that are not displayed as filter option.<br>
>><br>
>> Â Yed, this is indeed the case. Our "set in file" filters do not cover<br>
>> Â all<br>
>> Â the files where variables can be set. Also, a variable might be set by<br>
>> Â more than one file.<br>
>><br>
>> Â Cheers<br>
>> Â ><br>
>> Â >Thanks,<br>
>> Â >Ravi<br>
>> Â ><br>
>> Â ><br>
>> Â >--<br>
>> Â >_______________________________________________<br>
>> Â >toaster mailing list<br>
>> Â ><a href="mailto:toaster@yoctoproject.org">toaster@yoctoproject.org</a><br>
>> Â ><a href="https://lists.yoctoproject.org/listinfo/toaster" target="_blank">https://lists.yoctoproject.org/listinfo/toaster</a><br>
><br>
<br>
--<br>
_______________________________________________<br>
toaster mailing list<br>
<a href="mailto:toaster@yoctoproject.org">toaster@yoctoproject.org</a><br>
<a href="https://lists.yoctoproject.org/listinfo/toaster" target="_blank">https://lists.yoctoproject.org/listinfo/toaster</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">Alex Damian<div>Yocto Project<br></div><div>SSG / OTC </div></div>
</div>