[FFmpeg-devel] [PATCH] Implement av_get_token()
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.
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
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-devel