[x265] [PATCH] framefilter: comment cleanups, use pixel data type

[Steve Borho]

On Fri, Sep 13, 2013 at 11:07 AM, Derek Buitenhuis <
derek.buitenhuis at gmail.com> wrote:

> On 9/13/2013 4:51 PM, Steve Borho wrote:
> > This is a really minor but unrelated bug fix.  I can move it into a
> separate patch.
> >
> > 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.
>
> LGTM if you segregate this off int off separate patch with that
> description in the commit message.
>

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


>
> - Derek
> _______________________________________________
> x265-devel mailing list
> x265-devel at videolan.org
> https://mailman.videolan.org/listinfo/x265-devel
>



-- 
Steve Borho
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20130913/0e0f2870/attachment.html>