Go Premium for a chance to win a PS4. Enter to Win

x
?
Solved

Dynamically creating an array of structures

Posted on 2000-05-05
5
Medium Priority
?
141 Views
Last Modified: 2013-11-20
Hi all
Win 95; VC 5.0 SP3

I am trying to create an array dynamically. this sort of works but I am
having trouble with memory leaks when destrying the array. any ideas.

CScaleThread **  ScThread;

in OnInitialUpdate()
{
....
 if (pDoc->iNumScales)
  {
  ScThread = new CScaleThread *[pDoc->iNumScales];
  }

 for (int y = 0; y < pDoc->iNumScales; y++)
  {
  ScThread[y] = 0;
  }
....
 for (int x =0; x < pDoc->iNumScales; x++)
  {
  GetDialogSize(&R);
  ScThread[x] = new CScaleThread;
  ScThread[x]->CreateThread(CREATE_SUSPENDED);
// Other stuff here
  ScThread[x]->ResumeThread();
  }
...
}

In OnDestroy()
{
 CTestscrollDoc* pDoc = GetDocument();
 if (ScThread)
  {
  for (int x =0; x < pDoc->iNumScales; x++)
   {
   if (ScThread[x])
    delete ScThread[x];
   }
  delete [] ScThread;
  }
 ScThread = 0;
}

This leaves me with a whole host of memory leaks. Why?
0
Comment
Question by:tcss
  • 3
  • 2
5 Comments
 
LVL 49

Expert Comment

by:DanRollins
ID: 2784646
You did not provide the definition of a CScaleThread.  It is possible that the ctor of this object allocates but the dtor does not free.

Also make sure that your OnDestroy() is in fact being called and when (put a break point there).  If it is not being called, put the delete stuff in the the dtor of the window object or call it from some other point in the program.

0
 
LVL 1

Author Comment

by:tcss
ID: 2787223
Hi Dan
Thank you for your input. By commenting you "took the blinkers" off my thinking. I found that the memory leak is comming from a "tested and working" portion of the code.

Though you did not answer the question directly (because the answer lay beyound the question asked) your invaluble comment helped be find the problem. Should you want the points, please replay with an answer indicating so. If not comment as such and I can delete the question.

Many thanks
Hylton
0
 
LVL 1

Author Comment

by:tcss
ID: 2797690
This question has a deletion request Pending
0
 
LVL 49

Expert Comment

by:DanRollins
ID: 2797733
This question no longer is pending deletion
0
 
LVL 49

Accepted Solution

by:
DanRollins earned 300 total points
ID: 2797734
Yes, I want points otherwise I would not bother answering questions.
0

Featured Post

Free Tool: ZipGrep

ZipGrep is a utility that can list and search zip (.war, .ear, .jar, etc) archives for text patterns, without the need to extract the archive's contents.

One of a set of tools we're offering as a way to say thank you for being a part of the community.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Introduction: Dialogs (1) modal - maintaining the database. Continuing from the ninth article about sudoku.   You might have heard of modal and modeless dialogs.  Here with this Sudoku application will we use one of each type: a modal dialog …
Have you tried to learn about Unicode, UTF-8, and multibyte text encoding and all the articles are just too "academic" or too technical? This article aims to make the whole topic easy for just about anyone to understand.
This video will show you how to get GIT to work in Eclipse.   It will walk you through how to install the EGit plugin in eclipse and how to checkout an existing repository.
Look below the covers at a subform control , and the form that is inside it. Explore properties and see how easy it is to aggregate, get statistics, and synchronize results for your data. A Microsoft Access subform is used to show relevant calcul…
Suggested Courses

824 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question