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

Martin Storsjö martin
Fri Jul 16 16:16:49 CEST 2010


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

> On Fri, Jul 16, 2010 at 9:59 AM, Martin Storsj? <martin at martin.st> wrote:
> > On Fri, 16 Jul 2010, Ronald S. Bultje wrote:
> >> 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.
> [..]
> > +            while (isspace(*end))
> > +                end++;
> 
> end += strspn(end, SPACE_CHARS);, OK otherwise.

Applied with this change

// Martin



More information about the ffmpeg-devel mailing list