<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Sep 13, 2013 at 11:07 AM, Derek Buitenhuis <span dir="ltr"><<a href="mailto:derek.buitenhuis@gmail.com" target="_blank">derek.buitenhuis@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 9/13/2013 4:51 PM, Steve Borho wrote:<br>
> This is a really minor but unrelated bug fix.  I can move it into a separate patch.<br>
><br>
> The PPAScopeEvent macro causes a bar to appear in our profiler with the given name, associated with the CPU core and start/stop time.  The problem with this particular instance is that the function was copy-pasted from another in this file and this profile scope came along for the ride and since they both have the same name and one calls the other it is essentially a NOP.<br>

<br>
</div>LGTM if you segregate this off int off separate patch with that<br>
description in the commit message.<br></blockquote><div><br></div><div>Ok, I saw this after I resent the series, but I've gone ahead and added this paragraph to the second patch and pushed the pair of them</div><div>
 </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
- Derek<br>
_______________________________________________<br>
x265-devel mailing list<br>
<a href="mailto:x265-devel@videolan.org">x265-devel@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/x265-devel" target="_blank">https://mailman.videolan.org/listinfo/x265-devel</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>Steve Borho
</div></div>