[MPlayer-dev-eng] [PATCH] Fix parse comment bug for input.conf

[Ulion]

2007/10/30, Ulion <ulion2002 at gmail.com>:
> 2007/10/23, Ulion <ulion2002 at gmail.com>:
> > 2007/10/16, Rich Felker <dalias at aerifal.cx>:
> > > On Tue, Oct 16, 2007 at 09:43:42PM +0800, Ulion wrote:
> > > > 2007/10/16, Rich Felker <dalias at aerifal.cx>:
> > > > > On Tue, Oct 16, 2007 at 03:02:04PM +0800, Ulion wrote:
> > > > > > Hello,
> > > > > >
> > > > > > For a reported bug:
> > > > > > http://lists.mplayerhq.hu/pipermail/mplayer-users/2007-October/069603.html
> > > > > > I tested, it is a bug when parsing input comand followed by comment in
> > > > > > the same line.
> > > > > > I make a patch for this, so when parse cmd, comment will be ignored.
> > > > >
> > > > > Are you sure this does not break bindings/commands with # in them?
> > > > >
> > > > > Rich
> > > >
> > > > This only break bindings/commands with # as the first char of a
> > > > parameter, that only has meaning for string parameter. But for this
> > > > situation, the parameter can be quoted by " or ' to has a leading '#'.
> > > >
> > > > Check current truck/etc/input.conf, not only one line has comment
> > > > after commands, it already broke the commands. Any other way to fix it
> > > > better?
> > >
> > > OK, I just wanted to make sure. I don't object to your patch as long
> > > as it's not breaking things.
> > >
> > > Rich
> >
> > If there's no further suggestion/objection, I will apply this comment
> > parse patch after a week.
> >
>
> I will apply the comment parsing patch tomorrow.
>

Applied.

-- 
Ulion