On Wed, 05 Apr 2006 18:04:24 +0000, Stefan Huelswitt wrote:
> On 05 Apr 2006 Soeren Sonnenburg <vdr-ml@xxxxxx> wrote:
>
>> I've tested your version and found that the mplayer volume fix does not work.
>> The attached patch makes it *really* work (well at least it is working here :)
>
> I'm a bit confused now.
> I have a possible response that the current code IS working (with
> a very little change though, the fix is in pre5).
Hmmhh I can assure you that it works here and was completely messed up
(*here*) up before...
> I checked MPlayer 1.0pre2 (which I'm using), pre6 and pre7try2 and none
> of them seem to support a float argument with "volume"
> (MP_CMD_ARG_INT in input/input.c).
>
> What version are you using?
Indeed I am using todays mplayer cvs and did not check how it that was
dealt with in older versions... however the set absolute value seems to be
in older versions...
input/input.c:69:
{ MP_CMD_VOLUME, "volume", 1, { { MP_CMD_ARG_FLOAT,{0} }, {MP_CMD_ARG_INT,{0}}, {-1,{0}} } },
mplayer:1967:
// Audio
{ "volume", mp_property_volume, CONF_TYPE_FLOAT,
M_OPT_RANGE, 0, 100, NULL },
This suggests that you should use the 'set absolute value feature' and %d
and round() the value to be backwards compatible...
> Is there any documentation about a float argument?
basically lines 1967 and 2048 ff. in mplayer.c in mplayer cvs...
and line 69 in input/input.c
Sorry for the confusion I've caused...
All the best,
Soeren.
