On 27/02/14 21:52, Diego Biurrun wrote: > 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. >
It is _exactly_ what we have in the makefile, just factored so shuffling the code easier, feel free to peek at my repo, I have a branch with the wip changes. I can try to untangle the parser first but doesn't change the fact that the parser currently depends on the decoder (all of it). lu _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
