Re: [FFmpeg-devel] [PATCH]lavf/mpegts: Consider 0x0f just a hint towards aac

2017-09-20 Thread Michael Niedermayer
On Wed, Sep 20, 2017 at 02:24:35AM +0200, Carl Eugen Hoyos wrote: > 2017-09-14 23:48 GMT+02:00 Michael Niedermayer : > > On Wed, Sep 13, 2017 at 07:12:17PM +0200, Carl Eugen Hoyos wrote: > >> Hi! > >> > >> Attached patch fixes ticket #6657. > >> > >> Please comment, Carl Eugen > > > >> mpegts.c |

Re: [FFmpeg-devel] [PATCH]lavf/mpegts: Consider 0x0f just a hint towards aac

2017-09-19 Thread Carl Eugen Hoyos
2017-09-14 23:48 GMT+02:00 Michael Niedermayer : > On Wed, Sep 13, 2017 at 07:12:17PM +0200, Carl Eugen Hoyos wrote: >> Hi! >> >> Attached patch fixes ticket #6657. >> >> Please comment, Carl Eugen > >> mpegts.c |2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> 370a433251f563e16b654ee

Re: [FFmpeg-devel] [PATCH]lavf/mpegts: Consider 0x0f just a hint towards aac

2017-09-14 Thread Michael Niedermayer
On Fri, Sep 15, 2017 at 12:13:56AM +0200, Carl Eugen Hoyos wrote: > 2017-09-14 23:48 GMT+02:00 Michael Niedermayer : > > > also please add a fate test, if the file is not too large > > Are 10 bytes ok? with mpeg-ts, i think its ok > The alternative is to simply mux any latm sample into mpe

Re: [FFmpeg-devel] [PATCH]lavf/mpegts: Consider 0x0f just a hint towards aac

2017-09-14 Thread Carl Eugen Hoyos
2017-09-14 23:48 GMT+02:00 Michael Niedermayer : > also please add a fate test, if the file is not too large Are 10 bytes ok? The alternative is to simply mux any latm sample into mpegts with patched FFmpeg to force id 0x0f. Carl Eugen ___ ffmpeg-d

Re: [FFmpeg-devel] [PATCH]lavf/mpegts: Consider 0x0f just a hint towards aac

2017-09-14 Thread Michael Niedermayer
On Wed, Sep 13, 2017 at 07:12:17PM +0200, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #6657. > > Please comment, Carl Eugen > mpegts.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > 370a433251f563e16b654eeca8bb3463e787fd61 > 0001-lavf-mpegts-Consider-stream_typ