[FFmpeg-devel] [PATCH] Log a warning when receiving HTTP error codes

Martin Storsjö martin
Fri Jul 16 15:59:07 CEST 2010


On Fri, 16 Jul 2010, Ronald S. Bultje wrote:

> Hi,
> 
> On Fri, Jul 16, 2010 at 7:52 AM, Martin Storsj? <martin at martin.st> wrote:
> > $subj - this simplifies debugging quite a bit, and improves usability in
> > general, too.
> 
> Great idea, I've always debugged using such code and then never
> committed. Bad me.
> 
> As for that patch, I wonder if you could insert a new variable for the
> endptr argument to strtol() just above this code, then skip_spaces()
> or strspn() and that way get a pointer to the actual text string, and
> debug that also. That would assist even more in "usability" (well, for
> developers I guess).

Good idea - attached version does this, too.

// Martin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Log-a-warning-when-receiving-http-errors.patch
Type: text/x-diff
Size: 1636 bytes
Desc: 
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100716/1c4b92a6/attachment.patch>



More information about the ffmpeg-devel mailing list