On Sat, Sep 06, 2014 at 08:24:10AM +0200, Reimar Döffinger wrote:
> On 06.09.2014, at 00:23, wm4 <nfx...@googlemail.com> wrote:
> > On Sat, 6 Sep 2014 00:00:10 +0200
> > Michael Niedermayer <michae...@gmx.at> wrote:
> > 
> >> On Thu, Sep 04, 2014 at 10:40:27PM +0200, wm4 wrote:
> >>> ---
> >>> Follows reimar's suggestion for detecting UTF-16. If the detection goes
> >>> wrong, probing the format won't succeed, so this should not break
> >>> anything.
> >>> 
> >> 
> >>> I didn't use ffio_ensure_seekback), because this apparently reallocates
> >>> the buffer - which we certainly don't want, because the probe API
> >>> doesn't allow this.
> >> 
> >> the format probe API doesnt provide a AVIOContext, and if one is
> >> provided, it could use a allocated and reallocatable buffer
> >> or do i misunderstand ?
> >> 
> >> [...]
> > 
> > I'm ffio_init_context() to initialize an AVIOContext on a buffer (the
> > probe buffer)
> 
> I do not know the actual behaviour for sure, but for a context based on a 
> buffer, seekback should never ever be possible to fail...
> So if we can detect that case, I think the ensure function should behave like 
> a NOP.

ffio_ensure_seekback() could check if AVIOContext.read_packet is set
and return success if its not

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Democracy is the form of government in which you can choose your dictator

Attachment: signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to