Opened 18 years ago

Closed 18 years ago

#26 closed defect (fixed)

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

Reported by: fbakan@… Owned by: anonymous
Priority: normal Milestone:
Component: WVGUI-Core Version: 1.0
Severity: normal Keywords: APM
Cc:

Description

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

Attachments (3)

apmoff.zip (3.9 KB) - added by fbakan@… 18 years ago.
sorry no patch but at at least example-functions that work for me
apmdiffs.zip (2.2 KB) - added by fbakan@… 18 years ago.
ok, here are diffs that work here, I hope it doesn't break something else, but it's a start
apmdiffs-v2.zip (3.1 KB) - added by fbakan@… 18 years ago.
Updated diffs for APM. Should be save now. At least I hope so ;-)

Download all attachments as: .zip

Change History (14)

comment:1 Changed 18 years ago by fbakan@…

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

comment:2 Changed 18 years ago by 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.

comment:3 Changed 18 years ago by 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.

comment:4 Changed 18 years ago by vladest

Owner: changed from vladest to anonymous
Status: newassigned

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

Changed 18 years ago by fbakan@…

Attachment: apmoff.zip added

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

comment:5 Changed 18 years ago by 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.

Changed 18 years ago by fbakan@…

Attachment: apmdiffs.zip added

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

comment:6 Changed 18 years ago by 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

Changed 18 years ago by fbakan@…

Attachment: apmdiffs-v2.zip added

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

comment:7 Changed 18 years ago by fbakan@…

Hi Vlad,

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

Franz

comment:8 Changed 18 years ago by anonymous

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

comment:9 Changed 18 years ago by 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

comment:10 Changed 18 years ago by vladest

applied

comment:11 Changed 18 years ago by vladest

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.