[MPlayer-dev-eng] [SURVEY] change mencoder's default ofilename extension?

[Reimar Döffinger]

Hi,
On Tue, Nov 29, 2005 at 08:23:34PM +0200, Oded Shimon wrote:
> > better use strrchr and make sure it works with "file.avi.mpg" and "file"
> > and things like that.
> 
> RTFS, it works great.

I said "make sure it works...", not it is broken. I usually mean what I
say.

> file.avi.mpg => "mpg"
> file         => "file"

Sorry, but I really think this is wrong. "file" has no extension, so why
consider "file" the extension??

> file.        => ""
> 
> using strrchr would not work with "file", but i could do it with strrchr if 
> you preffer, i see no point in it though.

Well, it is a standard function, IMHO making the code easier to
understand.

Greetings,
Reimar Döffinger