C / C++ / MFC

Re: Windows Shutdown [modified]

14-Mar-09 10:10

hxhl95 wrote:
The only param I don't get is flags, which I put NULL for

Flags is of type ULONG. Instead of putting it NULL you may try substituting it with some unsigned value like 0x123.... Smile | :)

And since it's hit and trial approach after all, this might not work. Instead you may try looking for actual kernel calls and then finding right parameter values using tools like process explorer etc. Best of luck!!! Smile | :)

hxhl9514-Mar-09 12:08

hxhl95

14-Mar-09 12:08

I just found out that NtSetSystemPowerState returns STATUS_NOT_IMPLEMENTED in NT 4.0. Now I'm wondering if it's implemented in 5.1 and above. I know for a fact that it's in Vista though. Laugh | :laugh:

All my calls to NtSetSystemPowerState have been returning 0xC00000EF. STATUS_NOT_IMPLEMENTED is 0xC0000002. If my research is correct, 0xC00000EF is NT_STATUS_INVALID_PARAMETER_1

Frown | :(

Anyone have clues regarding the parameters? I can't find any calls to it inside anything.

modified on Saturday, March 14, 2009 6:53 PM

[Message Deleted]

mini_25068914-Mar-09 7:56

mini_250689

14-Mar-09 7:56

[Message Deleted]

Re: help me!!!!!!!!

Stuart Dootson14-Mar-09 8:00

14-Mar-09 8:00

Plenty of help for your data structures homework task can be found[^] on the internet.

Java, Basic, who cares - it's all a bunch of tree-hugging hippy cr*p

Re: help me!!!!!!!!

NeoAks00714-Mar-09 8:04

14-Mar-09 8:04

That was a nice one!!! Big Grin | :-D

Re: help me!!!!!!!!

NeoAks00714-Mar-09 8:02

How to delete handles of a thread created with AfxBeginThread

14-Mar-09 8:02

This article[^] could help!! Smile | :)

keret14-Mar-09 7:29

Re: How to delete handles of a thread created with AfxBeginThread

14-Mar-09 7:29

I have an MFC dialog application. It monitors certain types of windows and collects information on them. I start a new thread for every window based on it's title. I use a timer to find windows that meet the criteria. I created a struct named ThreadInfo and I pass it to the thread. At the end I either close the monitored window(s) or close my dialogs.
This causes several memory leaks.

Here is the code in the timer if it finds a window that meets the criteria:

//Create ThreadInfo
    ThreadInfo *pInfo = new ThreadInfo;
   
    //put the information into pInfo
    pInfo->hMainWnd = dlg->GetSafeHwnd();
    pInfo->hMonitoredWnd = hwnd;
    pInfo->x = y; //And so on...

    //Start thread
    CWinThread *pThread = AfxBeginThread(ThreadFunction, (PVOID) pInfo, THREAD_PRIORITY_NORMAL, 0, 0);

Here is the code for ThreadFunction:

UINT ThreadFunction(LPVOID pParam)
{
    ThreadInfo *pInfo = reinterpret_cast<threadinfo> (pParam);
 
    while (IsWindow(pInfo->hMonitoredWindow))
    {
      
       do.Something;
       Sleep(100);
       
    }
    //delete pInfo;
    return 0;
}</threadinfo>

I tried the following scenarions to eliminate or at least identify the memory leaks. The resulting leaks are below them:

A:
- I create pInfo and close the dialog first:
thrcore.cpp 68B client block
mydlg.cpp 748B normal block
strcore.cpp 46B normal block

B:
- I create pInfo and close the monitored window first:
mydlg.cpp 748B nromal block
strcore.cpp 46B normal block

C:
- I create pInfo, but don't start the thread:
mydlg.cpp 748B normal block
strcore.cpp 46B normal block

D:
- I don't create pInfo and pass NULL to the thread:
no leaks!

E:
- I delete pInfo in ThreadFunction and close monitored window first:
no cpp file, 160B normal block

F:
- I delete pInfo in ThreadFunction and close dialog first:
no cpp file, 160B normal block
thrcore.cpp 68B client block
mydlg.cpp 748B normal block
strcore.cpp 46B normal block

So I think the 748B and 46B blocks are for pInfo, while the 68B and 160B block are for pThread. But how and where should I delete them?
I think I should delete them in the dialog, but how do I send a message from the thread when it ends? And how I delete the handles if I close the dialog first?

Stuart Dootson14-Mar-09 7:58

Re: How to delete handles of a thread created with AfxBeginThread

14-Mar-09 7:58

1. Why not take a local copy of *pInfo early in the thread function, so you can delete pInfo early:

UINT ThreadFunction(LPVOID pParam)
{
    ThreadInfo info(reinterpret_cast<ThreadInfo*>(pParam));
    delete reinterpret_cast<ThreadInfo*>(pParam);
 
    while (IsWindow(info.hMonitoredWindow))
    {
      
       do.Something;
       Sleep(100);
       
    }
    return 0;
}

2. Do you keep track of all the threads you've created? Doesn't look like it. Store them in a vector or something and delete them in your dialog's WM_CLOSE handler.

3. If the monitored windows aren't closed, does the thread terminate? You probably ought to have some way of telling the threads to exit even if their monitored window still exists.

Basically, you need to think about the likely lifetimes of objects and work out where you need to be deleting objects.

Java, Basic, who cares - it's all a bunch of tree-hugging hippy cr*p

keret14-Mar-09 9:51

Re: How to delete handles of a thread created with AfxBeginThread

14-Mar-09 9:51

Thanks!

1. This didn't compile, error was:
error C2664: 'ThreadInfo::ThreadInfo(const ThreadInfo &)' : cannot convert parameter 1 from 'ThreadInfo *' to 'const ThreadInfo &'

but I used the following:

ThreadInfo *pInfo = reinterpret_cast<threadinfo> (pParam);
ThreadInfo info = *pInfo;
delete pInfo;</threadinfo>

And it seems to work. The 748B and 160B blocks are eliminated.

2. I had a threads vector earlier, but I didn't use it. Now I use it as you recommended, but I use ON_WM_DESTROY (I switched off the close button):

void CFredoDlg::OnDestroy()
{
	//delete threads;
	for(vector<handle>::size_type i = 0; i < threads.size(); i++)
	{
		threads.erase(threads.begin() + i);
	}

}</handle>

But the 68B and 46B blocks are still there. Now I will try to find a way to terminate the threads. If you have a solution for that, don't keep it!

Stuart Dootson14-Mar-09 10:36

Re: How to delete handles of a thread created with AfxBeginThread

14-Mar-09 10:36

keret wrote:
But the 68B and 46B blocks are still there

Implies you have a thread object and a string that you haven't deleted.

keret wrote:
Now I will try to find a way to terminate the threads. If you have a solution for that, don't keep it!

You could use a global boolean-ish variable. You would set it using InterlockedExchange[^] and read as you would any other variable.

Set it to zero before any threads are started and to one when you want to exit. Use this code to set it (let's presume it's called letsExitNow)

InterlockedExchange(&letsExitNow, 1);

Now, in each thread, you would change your with loop to somethig like this:

while (!letsExitNow && IsWindow(pInfo->hMonitoredWindow))

In the ON_WM_DESTROY handler, wait for each thread handle to be signalled (signalled means the thread has exited) - you can use WaitForSingleObject[^] or WaitForMultipleObjects[^] to do that.

Java, Basic, who cares - it's all a bunch of tree-hugging hippy cr*p

keret14-Mar-09 11:17

Re: How to delete handles of a thread created with AfxBeginThread

14-Mar-09 11:17

My TreadFunction is in a separate file (MyThread.cpp).
How can I reference letsExitNow from there? Or should I put the code in MyDlg.cpp?

Stuart Dootson14-Mar-09 11:52

Re: How to delete handles of a thread created with AfxBeginThread

14-Mar-09 11:52

Add a definition for letsExitNow in one file:

LONG letsExitNow;

and an external declaration in the other:

extern LONG letsExitNow;

Java, Basic, who cares - it's all a bunch of tree-hugging hippy cr*p

keret15-Mar-09 4:03

Re: How to delete handles of a thread created with AfxBeginThread

15-Mar-09 4:03

Sorry, I have never used extern before.

I put this in MyDlg.h:

public:
LONG letsExitNow

I put this in CMyDlg::OnInitDialog:

InterlockedExchange(&letsExitNow, 0);

And I put this in ThreadFunction:

extern LONG letsExitNow;

But I got an "unresolved external symbol "long letsExitNow"" error. Did I something wrong?

Stuart Dootson15-Mar-09 6:07

Re: How to delete handles of a thread created with AfxBeginThread

15-Mar-09 6:07

Minor mis-understanding Smile | :)

Put the LONG letsExitNow in MyDlg.cpp - NOT in a method, just in the file.

Java, Basic, who cares - it's all a bunch of tree-hugging hippy cr*p

keret15-Mar-09 9:55

Template Class Issues in C++.

15-Mar-09 9:55

Thank You, I really appreciate your help. This was a complicated problem (for me now at least). The 160B object was another struct I didn't include in the code above.

NeoAks00714-Mar-09 7:24

14-Mar-09 7:24

Hi all,
I am developing a VC++ console application in which I have encountered a very strange problem. I have a class class A in which there are member objects of other different classes say class B and C. In Class A there is a member function search(int condition). This search() function depending upon argument returns a pointer to the member objects with the class. Code for this is:

class B{
      // Class Definitions
};
class C{
      // Class Definitions
};
class A{
public:
      B obj1;
      C obj2;
      someThing* search(int condition)  //What do I specify return type???
      {
            if(condition==1)
                  return &obj1;
            if(condition==2)
                  return &obj2;
            if(condition==3)
                  return this;
      }
};

Now the problem is that I am not able to specify any return type since it depends upon the condition which is known at runtime. I tried to introduce template as:

template -typename type-
type* search(int condition)
{
      //Function Definition
}

But, doing so I cannot call the function from main() since, while calling I have to specify template parameter which again is dependent on condition at run-time. I am empty headed and clueless rite now about solving this weird situation.. Confused | :confused:

Please suggest any problem solving approach. Also to mention that class structure is not to be changed.... HELP!!

N a v a n e e t h14-Mar-09 7:31

N a v a n e e t h

14-Mar-09 7:31

NeoAks007 wrote:
Now the problem is that I am not able to specify any return type since it depends upon the condition which is known at runtime. I tried to introduce template as:

You can't use templates here since your type is determined at run time. Create a abstract class and derive class B and C from it. You can return pointer to this abstract class instantiated with proper derived one.

Smile | :)

Navaneeth

How to use google | Ask smart questions

NeoAks00714-Mar-09 7:38

14-Mar-09 7:38

Hmmmm.. That seems to be a nice and elegant solution... Let me try it before I thank you (since actual class hierarchies are quite complex so implementing this solution might introduce new errors) !!! Smile | :)

N a v a n e e t h14-Mar-09 7:45

N a v a n e e t h

14-Mar-09 7:45

Good luck then. Smile | :)

Navaneeth

How to use google | Ask smart questions

NeoAks00714-Mar-09 7:50

14-Mar-09 7:50

N a v a n e e t h wrote:
You can return pointer to this abstract class instantiated with proper derived one.

Can you demonstrate how to do it? Confused | :confused:

Stuart Dootson14-Mar-09 8:07

14-Mar-09 8:07

class DummyBase {};

class B : public DummyBase{
      // Class Definitions
};
class C : public DummyBase{
      // Class Definitions
};

class A{
public:
      B obj1;
      C obj2;
      DummyBase* search(int condition)  //What do I specify return type???
      {
            if(condition==1)
                  return &obj1;
            if(condition==2)
                  return &obj2;
            if(condition==3)
                  return this;
      }
};

Java, Basic, who cares - it's all a bunch of tree-hugging hippy cr*p

N a v a n e e t h14-Mar-09 8:14

N a v a n e e t h

14-Mar-09 8:14

Here is a trivial example.

class Car{
public:
    virtual string SayName() = 0;
};

class Mercedez : public Car{
public:
    string SayName(){
	return "Mercedez";
    }
};

class Ferrari : public Car{
public:
    string SayName(){
	return "Ferrari";
    }
};

Car* CreateACar(string carType){
    if(carType == "Mercedez")
       return new Mercedez;
    else if(carType == "Ferrari")
       return new Ferrari;
}

Use it like

Car* car = CreateACar("Mercedez");
std::cout << car->SayName();

Read about factory design pattern. Smile | :)

Navaneeth

How to use google | Ask smart questions

NeoAks00714-Mar-09 8:32

14-Mar-09 8:32

I had implemented DummyBase this way only. Smile | :)

But there was problem accessing functions of class B & C through main() . After figuring out why, I have another serious problem now. Actual Implementations of

class B and C are:<br />
<pre><code><br />
class DummyBase {};<br />
<br />
template <'class abc'><br />
class Base{<br />
public:<br />
      abc func1();<br />
};<br />
<br />
class B : public Base`<datatype1`>, DummyBase{<br />
      // Class Definitions<br />
};<br />
class C : public Base`<datatype2`>, DummyBase{<br />
      // Class Definitions<br />
};<br />
<br />
class A: public Base`<datatype3`>, DummyBase{<br />
public:<br />
      B obj1;<br />
      C obj2;<br />
      DummyBase* search(int condition)  //What do I specify return type???<br />
      {<br />
            if(condition==1)<br />
                  return &obj1;<br />
            if(condition==2)<br />
                  return &obj2;<br />
            if(condition==3)<br />
                  return this;<br />
      }<br />
};<br />
<br />
void main()<br />
{<br />
      DummyBase *ptr;<br />
      A objA;<br />
      int n;<br />
      cin>>n;<br />
      ptr = objA.search(n);<br />
      ptr->func1();                   //Error : Since DummyBase does not contain func1()<br />
}<br />

Now the problem is that func1() cannot be included in DummyBase since return type of func1() is a template parameter. How to solve this..... Confused | :confused:

NeoAks00714-Mar-09 9:16