video2 works as perfect shutdown-tool if 'Use APM' is enabled in settings

[fbakan@…]

If 'Use APM' is checked video2 shuts down my laptop (T23).

Just starting video2 is enough. version: 10rc2

STDOUT log (complete):

set APM state 0 returns 4

cycles: 2

t: 0.062538, t1: 0.031254

CPU Freq: 23262518 Hz

Current codepage: 850 (IBM-850)

pipe [\PIPE\WV] open error: 3

pipe 3 opened

CD popup thread started

Built: 14 Feb 2006

OS/2 minor revision: 45

SSaver support inited

[fbakan@…]

Looking into the sources: seems that calling processAPMOffRequest() is the wrong function for _enabeling_ Power-Management.

[anonymous]

"Use APM" should be selected if you want to turn _off_ APM while using Warpvision (so as to have better performance) so processAPMOffRequest() is correct.

[fbakan@…]

Ok, if processAPMOffRequest() is the right funktion, then the used parameter SUBID_DISABLE_POWER_MANAGEMENT or SUBID_ENABLE_POWER_MANAGEMENT are not valid parameters for this funktion.

[vladest]

well, patches are welcome Since I dont have any APM pc/notebook

[fbakan@…]

sorry no patch but at at least example-functions that work for me

[fbakan@…]

I hope the example is all you need for implementing.

But if you insist I could try to patch the sources myself, hoping not to break something else ;-) but I'm soon on holidays for 3 weeks and offline, so it might take some time.

[fbakan@…]

ok, here are diffs that work here, I hope it doesn't break something else, but it's a start

[fbakan@…]

ok, I tried to write a patch, see the attachement from the last entry.

Hope it works. (At least it compiles and works for me)

Franz

[fbakan@…]

Updated diffs for APM. Should be save now. At least I hope so ;-)

[fbakan@…]

Hi Vlad,

I hope you like the diffs and they make it into the trunk.

Franz

[anonymous]

Is it necessary to write 3rd reincarnation od DosOpen/DosIoctl/DosClose? :) Regards, Vlad

[fbakan@…]

Not shure, but your version of open returned int where it should return HFILE which is not int.

I´m not shure, if this is a problem or not, so I thought it is save to rewrite.

Franz

[vladest]

applied