Re: [FFmpeg-devel] [PATCH 1/3] Add muxing/demuxing of RMHD

2018-01-19 Thread Michael Niedermayer
On Fri, Jan 19, 2018 at 09:45:44PM +0100, Moritz Barsnick wrote: > On Fri, Jan 19, 2018 at 18:16:01 +0100, Michael Niedermayer wrote: > > > Patches are not made to be applyable on their own. If that is the only > > > thing > > > missing I might move it to 1/3. > > > > code must build after each c

Re: [FFmpeg-devel] [PATCH 1/3] Add muxing/demuxing of RMHD

2018-01-19 Thread Moritz Barsnick
On Fri, Jan 19, 2018 at 18:16:01 +0100, Michael Niedermayer wrote: > > Patches are not made to be applyable on their own. If that is the only thing > > missing I might move it to 1/3. > > code must build after each commit when its pushed. > so if the patches represent future commits 1:1 then pleas

Re: [FFmpeg-devel] [PATCH 1/3] Add muxing/demuxing of RMHD

2018-01-19 Thread Michael Niedermayer
On Fri, Jan 19, 2018 at 01:39:31PM +0100, Thilo Borgmann wrote: > Am 19.01.18 um 01:02 schrieb Michael Niedermayer: > > On Thu, Jan 18, 2018 at 07:39:07PM +0100, Thilo Borgmann wrote: > >> This time including the patch... > > > >> fftools/ffmpeg.c | 28 +++ > >> fftools/ffmpeg_opt.c

Re: [FFmpeg-devel] [PATCH 1/3] Add muxing/demuxing of RMHD

2018-01-19 Thread Thilo Borgmann
Am 19.01.18 um 01:02 schrieb Michael Niedermayer: > On Thu, Jan 18, 2018 at 07:39:07PM +0100, Thilo Borgmann wrote: >> This time including the patch... > >> fftools/ffmpeg.c | 28 +++ >> fftools/ffmpeg_opt.c |2 >> libavformat/Makefile |2 >> libavformat/allformats.c |

Re: [FFmpeg-devel] [PATCH 1/3] Add muxing/demuxing of RMHD

2018-01-18 Thread Michael Niedermayer
On Thu, Jan 18, 2018 at 07:39:07PM +0100, Thilo Borgmann wrote: > This time including the patch... > fftools/ffmpeg.c | 28 +++ > fftools/ffmpeg_opt.c |2 > libavformat/Makefile |2 > libavformat/allformats.c |1 > libavformat/rm.c |1 > libavformat/r

Re: [FFmpeg-devel] [PATCH 1/3] Add muxing/demuxing of RMHD

2018-01-18 Thread Carl Eugen Hoyos
2018-01-18 19:39 GMT+01:00 Thilo Borgmann : > This time including the patch... Why do you add a new demuxer? Isn't it sufficient to add the new descriptor (with AV_RB32()) to the existing probe function? Please split this patch: From a (very) quick look, I saw: Demuxing MLTI, muxing aac, muxing R

Re: [FFmpeg-devel] [PATCH 1/3] Add muxing/demuxing of RMHD

2018-01-18 Thread Thilo Borgmann
This time including the patch... From aaeaf7721ba5e9afd79ee13dbb9cc7c3226027b3 Mon Sep 17 00:00:00 2001 From: Thilo Borgmann Date: Wed, 17 Jan 2018 23:09:03 +0100 Subject: [PATCH 1/3] Add muxing/demuxing of RMHD --- fftools/ffmpeg.c | 28 +++- fftools/ffmpeg_opt.c | 2 +- libavfor

[FFmpeg-devel] [PATCH 1/3] Add muxing/demuxing of RMHD

2018-01-18 Thread Thilo Borgmann
___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel