<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Sep 13, 2013 at 5:01 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 7:29 AM, Steve Borho wrote:<br>
> # HG changeset patch<br>
> # User Steve Borho <<a href="mailto:steve@borho.org">steve@borho.org</a>><br>
> # Date 1379053732 18000<br>
> #      Fri Sep 13 01:28:52 2013 -0500<br>
> # Node ID 42faf1cb4d7b1cf6b0323fa4abc82a31c8eed270<br>
> # Parent  17d41a7563aaa090b47377dac317437d7733844a<br>
> framefilter: comment cleanups, use pixel data type<br>
<br>
</div>[...]<br>
<div class="im"><br>
> -            // TODO: this code is NOT VERIFY because TransformSkip and PCM mode have some bugs, they always not active!<br>
> +            // TODO: this code is NOT VERIFY because TransformSkip and PCM modes have some bugs, they are never enabled<br>
<br>
</div>s/VERIFY/VERIFIED/<br></blockquote><div><br></div><div style>yeah, I should improve the english there a bit more</div><div style> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im"><br>
> -        // TODO: this code is NOT VERIFY because TransformSkip and PCM mode have some bugs, they always not active!<br>
> +        // TODO: this code is NOT VERIFY because TransformSkip and PCM modes have some bugs, they are never enabled<br>
<br>
</div>Ditto.<br>
<br>
> -    PPAScopeEvent(Thread_filterCU);<br>
> -<br>
<br>
This is the only change I'm not sure about. How is it related?<br></blockquote><div><br></div><div style>This is a really minor but unrelated bug fix.  I can move it into a separate patch.</div><div style><br></div><div style>
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.</div>
<div><br></div></div>-- <br>Steve Borho
</div></div>