Dan Petitt:
Platform_Finalise()
Does not free the library "Msimg32", need to add:
if( hDLLImage )
{
::FreeLibrary( hDLLImage );
hDLLImage = NULL;
}
Platform_Finalise is called from DllMain so is not supposed to call
FreeLibrary. See Microsoft's documentation for DllMain. Really,
Platform_Init shouldn't call LoadLibrary either and MSIMG32 should
load when needed but that would require more code and should probably
be made thread-safe so I haven't bothered. The consequences of failing
during finalisation can be a little scary including executing code
that has been freed.
RESearch does not initialise some memory, need to add to RESearch::Init:
for (int k=0; k<MAXNFA; k++)
nfa[k] = 0;
OK, committed.
Theres also quite a few places where memory is not being initialised and
also a lot of memory leaks that I am still trying to track down.
I run under BoundsChecker occasionally and most existing issues are
in Lua (which seem to be fixed with the newer version of Lua) or are
from places where BoundsChecker can't check the ways in which
addresses are copied, notably where document pointers are converted to
integers to give to the container.
Neil
_______________________________________________
Scintilla-interest mailing list
[email protected]
http://mailman.lyra.org/mailman/listinfo/scintilla-interest
