[FFmpeg-devel] [PATCH 1/3] libavformat/avio: Utility function to return URLContext

[Nicolas George]

Le sextidi 26 brumaire, an CCXXVI, Jeyapal, Karthick a écrit :
> I have done the change as suggested. Please find the new patch attached.

Sorry, but I still have doubts about it.

To begin with, -1 is not an acceptable error code.

But what bothers me most is that on the two uses, one has an assert, the
other has an error return, the inconsistency looks suspicious.

I think, at least, it should include a comment at both places to
indicate why the test is done like that.

> I need to send ‘0’ byte http packet to indicate close.
> But ffurl_write doesn’t call http_write for a 0 byte packet.
> Hence, I am calling prot->url_write directly. 

Let me see if I understand correctly: are you saying that it is not
possible to perform multiple requests using only public APIs? I find it
suspicious, since multiple_requests is a public option. If that is true,
then it needs fixing.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20171117/6be4254e/attachment.sig>