Problem with Pointers in my App

Quote:

Original post by joseka17 My problem is that i need to allocate and WNdCLASSEX and a LPSTR dinamically ( I use realloc because the New operator dont reallocate memory)

Don't get me wrong but i think its time you looked into the c++ standard library, here is part of it, i recommend you stop your usage of C-style dynamic arrays & strings move to C++ style dynamic arrays (std::vector) & strings (std::strings), i would change your code to something like this:

#include <vector>#include <string>//....namespace foo {  //....  class Ventana {       typedef std::vector<WNDCLASSEX> vec_of_wcls;       vec_of_wcls winds;       //....   public:      //.....     UINT Registrar(const std::string& InNameClassRef, UINT InCstyle) {         WNDCLASSEX temp_wndc;          //....          winds.push_back(temp_wndc);          //....     }     //....  };};

Quote:

Original post by joseka17 m_wclase=(WNDCLASSEX*)realloc(m_wclase,sizeof(WNDCLASSEX)*m_numreg) m_nombreclase=(LPSTR*)realloc(m_nombreclase,sizeof(LPSTR)*m_numreg)

What are the values of m_wclase and m_nombreclase that are passed to realloc when you get the error? My guess is that they are uninitialized. In that case, you need to set them to 0 before realloc is called the first time (like in the constructor). And don't forget to free them in the destructor.

To JohnBolton
Yep it work , i fixed putting a m_wclass=NULL in the constructor of my class, but well this is a newbie question but , why i have to initializate my variables??? (i mean if i replace all the code to register and create a windows directly in WinMain as in many tutorials out there, I dont have to init my varibales, Why is that so??)

To snk_kid
wow interesting library , im using it now

Thanks for reply

You have to initialize them because you use realloc. Well you should initialize them anyway, but realloc does depend on the pointer's previous memory. If you don't init the pionter to null realloc assumes the pointer allreadyu points to existing memory, and very bad things ensue.

From MSDN

realloc returns a void pointer to the reallocated (and possibly moved) memory block. The return value is NULL if the size is zero and the buffer argument is not NULL, or if there is not enough available memory to expand the block to the given size. In the first case, the original block is freed. In the second, the original block is unchanged. The return value points to a storage space that is guaranteed to be suitably aligned for storage of any type of object. To get a pointer to a type other than void, use a type cast on the return value.

Cheers
Chris

Thank for the help and well I have 2 other question .1)Do i have to initialize my pointer if i use an stl vector instead of reallocating it (Well i guest i have to initializate anyway)?? and another question is .. what happen if i register a window class variable with lets say... the name "win1" and then i fill the same windows class variable with the name "win2" and register it?? Could it be possible? Could it give me problems unregistering it??(i save the registered names in a stl vector).

Regards Joseka