[FFmpeg-devel] [PATCH] cmdutils: Restore opt_protocols() as it was before the API "cleanup"

Michael Niedermayer michaelni at gmx.at
Sun Feb 5 15:07:15 CET 2012


On Sun, Feb 05, 2012 at 01:36:38PM +0000, Paul B Mahol wrote:
> On 2/5/12, Michael Niedermayer <michaelni at gmx.at> wrote:
> > This uses internal API as its not possible print a list of
> > protocols as we did before with the new public API.
> >
> > Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> > ---
> >  cmdutils.c |   22 ++++++++++++++--------
> >  1 files changed, 14 insertions(+), 8 deletions(-)
> >
> 
> This obviously breaks shared builds.

yes, it was more intended as a RFC than a ready to be commited patch


> 
> So implementing public api to support this and leaving
> ffurl_protocol_next() private is prefered solution.

the old API worked quite fine, only thing it was missing where av
prefixes
"cleaning" it up and then reinventing it is kinda retarded.
Iam not sure what is best, but i think the protocol stuff should
become public again. An application should be able to implement its
own URLProtocols. As a sideeffect it allows printing more details
about the protocols too than what can be done in the current public
API


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

In a rich man's house there is no place to spit but his face.
-- Diogenes of Sinope
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120205/cc0e1b56/attachment.asc>


More information about the ffmpeg-devel mailing list