[FFmpeg-devel] url_open_protocol

[Ronald S. Bultje]

Hi,

On Tue, Aug 19, 2008 at 7:31 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Tue, Aug 19, 2008 at 06:30:42PM -0400, Ronald S. Bultje wrote:
>> Hi,
>>
>> On Tue, Aug 19, 2008 at 4:41 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
>> > On Tue, Aug 19, 2008 at 04:06:04PM -0400, Ronald S. Bultje wrote:
>> >> Hi,
>> >>
>> >> On Tue, Aug 19, 2008 at 3:24 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
>> >> > On Tue, Aug 19, 2008 at 02:00:28PM -0400, Ronald S. Bultje wrote:
>> >> >> Hi,
>> >> >>
>> >> >> I sent this before [1], but no reply, so I'll send it again. Is this
>> >> >> interesting for svn for those of us that want to open protocols
>> >> >> instead of just files?
>> >> >
>> >> > patch ok
>> >>
>> >> It wasn't against latest SVN. I redid the patch against SVN. The patch
>> >> is the same, except that url_open now sets is_streamed, I moved that
>> >> chunk from url_open to _protocol() also.
>> >>
>> > [...]
>> >
>> >>              uc->is_streamed= 1;
>> >>      *puc = uc;
>> >>      return 0;
>> >> +
>> >>   fail:
>> >>      *puc = NULL;
>> >>      return err;
>> >>  }
>> >
>> > unrelated cosmetic
>>
>> Hm, I keep doing that. This one should be OK. I'll apply the proto_str
>> -> up->name conversion separately if you wish.
>
> patch ok, and no need to seperate this its just one line

OK, applied all as one then.

Thanks,
Ronald