[FFmpeg-devel] [Patch] Topfield 58XXPVR and 68XXPVR file support

[James Marsh]

On Wed, Feb 4, 2009 at 4:13 PM, James Marsh <doctorjames at gmail.com> wrote:
> On Tue, Feb 3, 2009 at 11:00 AM, Michael Niedermayer <michaelni at gmx.at> wrote:
>>
>> On Tue, Feb 03, 2009 at 02:40:25AM +0000, M?ns Rullg?rd wrote:
>> > Michael Niedermayer <michaelni at gmx.at> writes:
>> >
>> > > On Sun, Nov 30, 2008 at 10:13:52AM +0000, M?ns Rullg?rd wrote:
>> > >> "James Marsh" <doctorjames at gmail.com> writes:
>> > >>
>> > >> > Topfield make a variety of personal video recorders that allow for the
>> > >> > transfer of recorded programmes via an onboard USB port. The ".rec"
>> > >> > files they produce are MPEG2 transport streams with a 3760 byte header
>> > >> > containing archive information for the PVR. The attached patch checks
>> > >> > for the presence of "TFrc" in the first four bytes of a file and
>> > >> > identifies it as an mpeg transport stream if present. It subsequently
>> > >> > skips this header when the services are probed. With the patch in
>> > >> > place these files are then fully supported by ffmpeg.
>> > >>
>> > >> Please upload a sample.
>> > >
>> > > heres a better patch that fixes this.
>> > >
>> > > can i apply it?
>> > >
>> > > Index: libavformat/mpegts.c
>> > > ===================================================================

[ ... ]

>> >
>> > OK
>>
>> applied and thanks
>
> Splendid. I'll check this out and try it against a variety of files.
>
> Many thanks,
> James
>

Hi Michael,

I can confirm it works without any problems on a wide selection of
files from multiple firmware revisions on the PVR.

Many thanks!
James.