[FFmpeg-devel] [PATCH] Remove static ByteIOContexts

[Michael Niedermayer]

On Fri, Nov 16, 2007 at 10:39:40AM +0100, Bj?rn Axelsson wrote:
> On Fri, 2007-11-16 at 03:33 +0100, Michael Niedermayer wrote:
> > [...] 
> > >  2. bioc_formats_1.diff
> > >     Update libavformats for dynamic ByteIOContexts
> > 
> > ok (though doesnt apply anymore, not suprissing considering its size)
> > this one could get tricky timing wise as it likely will break again quickly
> > if reposted 
> 
> I better try to be fast then. 
> Resynced with current svn. Regression tests pass.
> 
> > also when reposting please indicate which patches have changed so i dont
> > need to review all of them again and my patch application monkeys just
> > need to run md5sum over them to verify
> > 
> > 
> > [...]
> > 
> > >  6. bioc_ffplay_reindent.diff
> > >     Reindent from previous patch
> > 
> > unneeded (see below)
> 
> Merged with bioc_ffplay-2.diff
> 
> bioc_formats_1.diff and bioc_ffplay-2.diff are changed since last mail.
> bioc_ffplay_reindent.diff is removed. The rest are unchanged as they
> still apply:
> 
>  1. bioc_api_1.diff
>     Dynamic allocation of the ByteIOContext in AVFormatContexts

(no update so not checked)


>  2. bioc_formats_1.diff
>     Update libavformats for dynamic ByteIOContexts

ok

btw, if someone wants to actually look at this then try
1. sed 's/^\+\(.*\)\([ (]\)\([a-z1]*\)->pb\([^.]\)/+\1\2\&\3->pb\4/' ~/bioc_formats_1.diff
2. and apply the output locally
3. look at svn di

this reduces the size to less than a 10th


>  3. bioc_utils.diff
>     Update utils.c for dynamic ByteIOContexts

ok

>  4. bioc_aviobuf_nullcheck.diff
>     Handle NULL parameters in url_fseek(), url_fsize(), url_ferror() and
> url_feof()

(no update so not checked)


>  5. bioc_ffplay-2.diff
>     Update ffplay for dynamic ByteIOContexts

ok


> 6. bioc_ffmpeg-2.diff
>     Update ffmpeg for dynamic ByteIOContexts

ok


> 7. bioc_ffserver.diff
>     Update ffserver for dynamic ByteIOContexts

ok


> 8. bioc_dynbuf_api.diff
>     Let url_open_buf() and url_open_dyn_*() allocate ByteIOContexts

(no update so not checked)


> 9. bioc_dynbuf_formats.diff
>     Update libavformats for url_open_buf() and url_open_dyn_*() changes

ok

> 10. bioc_dynbuf_ffserver.diff
>     Update ffserver for url_open_buf() and url_open_dyn_*() changes

ok


> 11. versionbump.diff
>     Increase libavformat major version

ok


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Good people do not need laws to tell them to act responsibly, while bad
people will find a way around the laws. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071116/0e67b134/attachment.pgp>