On Thu, Feb 27, 2014 at 09:39:41PM +0100, Luca Barbato wrote:
> On 27/02/14 21:00, Diego Biurrun wrote:
> > On Thu, Feb 27, 2014 at 07:06:08PM +0100, Luca Barbato wrote:
> >> They require the full decoder.
> >> ---
> >>  configure           |  4 ++--
> >>  libavcodec/Makefile | 11 ++---------
> >>  2 files changed, 4 insertions(+), 11 deletions(-)
> > 
> > That the parser requires the full decoder seems like the underlying issue
> > to me.
> 
> I'm chopping h264.c in pieces so hopefully the parser might not depend
> in the whole deal anymore.
> 
> Factoring out so I don't have to touch the makefile in 3 places.

Given that you haven't done that yet and the end result is unclear,
I remain sceptical.  Can't you first do the other steps so that we
get to see this as part of a sensible series?  Right now this is
simply incorrect and should be rejected IMO.

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to