[MPlayer-DOCS] r21656 - trunk/DOCS/man/en/mplayer.1

[Guillaume POIRIER]

Hi,

On 12/17/06, mail at kraymer.de <mail at kraymer.de> wrote:


> Guillaume, I think you forgot to update the option line:
>
> > denoise3d[=luma:chroma:time]

Woops, you're right.


> Btw, is it necessary to change the parameter names? I translated
> luma:chroma:time so I'm a bit unsure if I should either ignore the change
> or use the English descriptions.. (Probably I'll ignore them and only add
> the new parameter since I translated these in many other parts of man page
> and want to keep things consistent)

I think they really had to be changed, though maybe the names I came
up with weren't the best you can imagine.

- 4th parameter had to be named in a sensible way, and I couldn't just
name it "time", right?
- previous names weren't that good because it didn't make sense to
summarize spacial chroma as "chroma", why wouldn't temporal chroma be
summarized as "chroma"?

Off course, if you have a better naming scheme, I'm all ears.

Guillaume
-- 
An association of men who will not quarrel with one another is a thing
which has never yet existed, from the greatest confederacy of nations
down to a town meeting or a vestry.
 -- Thomas Jefferson