[FFmpeg-cvslog] r13061 - trunk/libavformat/avio.c

Corey Hickey bugfood-ml
Sat May 10 00:37:37 CEST 2008


Michael Niedermayer wrote:
>>>> -------------------------------------------------------------------
>>>> if(url_fopen(&priv->oc->pb, mp_filename, URL_WRONLY))
>>>> {
>>>> 	mp_msg(MSGT_MUXER, MSGL_FATAL, "Could not open outfile\n");
>>>>         goto fail;
>>>> }
>>>> ((URLContext*)(priv->oc->pb->opaque))->priv_data= muxer;
>>> mp_open() should set this IMHO.
>>> (for example by using mp_filename "menc://%p" or a static var)
>> Thanks, but I don't understand what you mean. I've looked at the
>> relevant code and I think I can tell what it's doing, but I keep coming
>> back to what looks like a circular dependency.
>>
>> If it's not much trouble for you, feel free to explain your solution by
>> committing a patch and letting me read -cvslog. ;)  Otherwise, the only
> 
> Iam not interrested in working on mplayer currently, maybe i will step back
> as maintainer of all mplayer code for which iam maintainer, maybe not
> ive not decided yet. But i prefer to work on projects where the developers
> can affect decissions about the project and themselfs.

You mean because of the latest Uoti fiasco? That's a shame. Actually,
the whole situation is regrettable, but I particularly hate to see it
driving developers away. I don't do much work for the project myself,
though, so I have no call to pass judgment....

Anyway, I guess I'll send a hack patch to mplayer-dev-eng for now. Feel
free to reject it and commit/explain something better. :)

-Corey




More information about the ffmpeg-cvslog mailing list