[MPlayer-dev-eng] [PATCH] suboption escaping

[Joey Parrish]

On Wed, Aug 31, 2005 at 02:02:44AM +0200, Reimar Döffinger wrote:
> On Sat, Aug 20, 2005 at 04:07:57PM +0200, Reimar D?ffinger wrote:
> > On Fri, Aug 19, 2005 at 10:09:21AM -0500, Joey Parrish wrote:
> > > I would accept "" parsing if you have a patch to offer.  All I care
> > > about is mencoder -info "title=Star Trek: The Next Generation".  If you
> > > can make this work (other than the %len% method, which is a pain for
> > > scripting) then I'll be happy.
> > 
> > My suggestion... it will bring the two suboption parsers a bit more in sync...
> > A few strings aren't freed in the error case, but IMHO that does not
> > matter (an it was like that also before).
> 
> Any comments? Tested?

I'm still in favor of backslash escaping.  I'm going to post a new patch
with documentation in a new thread.  I've also modified the code to
address most of the issues you brought up with the first patch.

I don't want to start a holy war over backslashes versus quotes.  Either
one could be implemented properly and do the job.  But I still like mine
better.  :)  I had assumed that this would be settled by consensus from
other devs, but there has been very little interest expressed.  Have any
ideas?

--Joey

-- 
"Living in the complex world of the future is somewhat
like having bees live in your head.  But, there they are."