On 27/03/15 15:35, Mathieu Malaterre wrote:
> Hi Luca !
> 
> On Fri, Mar 27, 2015 at 1:10 PM, Luca Barbato <[email protected]> wrote:
>> On 27/03/15 09:06, Mathieu Malaterre wrote:
>>
>>> In this case, I'll end up duplicating some code in `mpegvideo.c` and
>>> `mov.c` then, right ?
>>
>> No.
>>
>>> DICOM containers can have either an mpegvideo format with mpeg2video
>>> stream, or MPEG (mov) with h264 video stream.
>>
>> Do you have a specification handy? Dicom can contain both mpegps and isom ?
> 
> Simplified spec can be found here:
> 
> http://medical.nema.org/medical/dicom/current/output/chtml/part05/sect_8.2.7.html
> 
> DICOM is pretty much at application level: it can contains MPEG-2 or
> MP4 (it can even contains JPEG+JFIF header or JPEG 2000 stream...).
> 
>> If that's the case then is _much_ saner to have a dicom.c and inside it
>> you chain the demuxers like it is done for hls or segment.
> 
> I have started something here:
> 
> https://github.com/malaterre/libav/blob/dicom-video/libavformat/dicmdec.c
> 
> So you're saying basically that I need to call `av_probe_input_buffer`
> & `avformat_open_input` from my `dicm_read_header` initial
> implementation ?

avformat_open_input might be enough, but yes, you got the gist of it =)

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

Reply via email to