[Mplayer-cvslog] CVS: main m_option.c,1.12,1.13

[Diego Biurrun]

Alban Bedel writes:
 > on Wed, 02 Jul 2003 18:51:45 +0200 you wrote:
 > > On Tue, Jul 01, 2003 at 11:50:33AM +0200, Alban Bedel wrote:
 > > > on Tue, 1 Jul 2003 11:27:15 +0200 you wrote:
 > > > 
 > > > > Update of /cvsroot/mplayer/main
 > > > > In directory mail:/var/tmp.root/cvs-serv2001
 > > > > 
 > > > > Modified Files:
 > > > > 	m_option.c 
 > > > > Log Message:
 > > > > Fix the segfault in case of missing arg for flag options
 > > > > in config files.
 > > > This must be backported to main.
 > > > 	Albeu
 > > 
 > > ??  Wasn't this fixed some time ago on both main and 0_90?
 > I can't really tell as i missed more than a month of cvs log.

Just hit the archives ;-)

 > But this was at least broken in main, so i doubt it's fixed in
 > main.

OK, I suppose you are confusing main and 0_90 again..

What is the problem?  Could you give some examples?  I get an error
message instead of a crash if I have something like

dr=
dr

in my config file now.

 > > And what do you mean by backported to main?  You committed to main,
 > > didn't you?
 > Well ... i meant 0_90. It's just that i'm short on dsik space.So if
 > someone who have a 0_90 tree could commit this 1 line fix that
 > would be nice.

Here are the patches and commits:

main:
http://mplayerhq.hu/pipermail/mplayer-dev-eng/2003-June/019036.html
http://mplayerhq.hu/pipermail/mplayer-cvslog/2003-June/015350.html

0_90:
http://mplayerhq.hu/pipermail/mplayer-dev-eng/2003-June/019079.html
http://mplayerhq.hu/pipermail/mplayer-cvslog/2003-June/015363.html

Diego