[MPlayer-cvslog] CVS: main parser-mecmd.c, 1.12, 1.13 parser-mpcmd.c, 1.25, 1.26

[Reynaldo H. Verdejo Pinochet]

On Sat, Apr 22, 2006 at 05:33:10PM +0200, Reimar Döffinger wrote:
> Hi,
> On Sat, Apr 22, 2006 at 03:09:40AM -0400, Reynaldo H. Verdejo Pinochet wrote:
> > > +	  else
> > >  	  if(tmp < 0){
> > >  //	    mp_msg(MSGT_CFGPARSER, MSGL_ERR, "m_config_set_option() failed (%d)\n",tmp);
> > > +	    mp_msg(MSGT_CFGPARSER, MSGL_ERR, "Error parsing option on the command line: -%s\n",opt);
> > 
> > IMHO this should be a fatal.
> 
> I do not disagree, this is just a copy-and paste (maybe from the config
> file parser I think?
> So anyone who feels like, go through those messages and fix it. I know,
> it's my commit, but unless you insist that I fix it myself I'll use the
> time to get some other patches out...

Not a prob, I will fix it.

  Reynaldo
-------------- 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/mplayer-cvslog/attachments/20060422/4889b0e0/attachment.pgp>