[FFmpeg-devel] [PATCH] Implement av_get_token()

Michael Niedermayer michaelni
Fri May 1 16:39:40 CEST 2009


On Fri, May 01, 2009 at 03:55:57PM +0200, Michael Niedermayer wrote:
> On Fri, May 01, 2009 at 01:33:09PM +0200, Stefano Sabatini wrote:
> > On date Thursday 2009-04-30 03:45:13 +0200, Michael Niedermayer encoded:
> > > On Sat, Apr 25, 2009 at 01:21:49PM +0200, Stefano Sabatini wrote:
> > [...]
> > > > > add \ at the end escaing the traiing null
> > > > > and unterminated '
> > > > > and escaped leading and trailing whitespace
> > > > 
> > > > OK.
> > > > 
> > > > [..]
> > > > > > +    /* strip trailing whitespaces */
> > > > > > +    out--;
> > > > > > +    while(--out >= ret && strspn(out, WHITESPACES))
> > > > > > +        *out = 0;
> > > > > 
> > > > > this will remove escaped trailing whitespaces
> > > > 
> > > > Reimplemeneted as a finite state machine, as the previous apporach was
> > > > resulting messy and unreadable.
> > > 
> > > i do not plan to approve this patch
> > > i think work should continue based on the previous version
> > > which was cleaner, simpler and smaller
> > > it only had one trivial bug
> > 
> > The previous patch had more problems that it looked, especially for
> > which regarded the terminating condition which I'm explicitely setting
> > now (for this I'm using the is_end var).
> > 
> > Other than this, the attached patch is absolutely equivalent to the
> > previous one in term of functionality (and now that I look at it I may
> > agree it's nicer).
> 
> patch rejected, i will try to fix the previous version myself.

done

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I wish the Xiph folks would stop pretending they've got something they
do not.  Somehow I fear this will remain a wish. -- M?ns Rullg?rd
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090501/8197c768/attachment.pgp>



More information about the ffmpeg-devel mailing list