[MPlayer-dev-eng] [PATCH] forceable software volume control

[Torinthiel]

On Fri, Nov 05, 2004 at 06:23:52PM +0200, Oded Shimon wrote:
> 
> On Friday 05 November 2004 18:11, Torinthiel wrote:
> > Have you read it? Or have you read the code and here is wrong?
> > I see the range is 50-200. How is 90 close to 200? If you want it neat
> > the middle of scale, which I find reasonable, then 90 is better then 50,
> > and 100 would be even better. Or the code accepts 50-100, and 200 in
> > manpage patch is a mistake.
> You misunderstood the code.

No, I didn't. I just haven't read it. Only the docs part.

> That variable sets the maximum possible volume, 1 divided by that number.
[cut]
> Get it? :)

I think so. But if so, than meaning of the parameter is contrary to the
obvious. Obvious for me, that when I increase parameter I increase
volume. If the opposite is true, than IMHO it HAS to be described in
docs. Wonder how many people would think of it as a bug. I would.
Torinthiel

-- 
 Waclaw "Torinthiel" Schiller       GG#: 542916, 3073512
   torinthiel(at)megapolis(dot)pl
   gpg: B06901F1 fpr: FAA3 559F CAE9 34DE CDC8  7346 2B6E 39F2 B069 01F1
 "No classmates may be used during this examination"
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20041105/9664bddf/attachment.pgp>