204304 - Do CRT DBG stuff on MingW builds

Status: RESOLVED WONTFIX

(Core Graveyard :: GFX: Win32, defect)

Description

[Jonathan Wilson]

Once MingW gets crtdbg.h, it will be possible to do the crtdbg stuff in
nsViewerApp.cpp

Comment 1

[Dan Mosedale (:dmosedale, :dmose)]

Taking, since I'm the MingW port owner.

jonwil: what "crtdbg stuff" do you think is missing?

Comment 2

[Jonathan Wilson]

its in webshell/tests/viewer/nsViewerApp.cpp
that #includes <crtdbg.h> which (along with some code that uses it) is currenty
disabled on MingW. If MingW gets a crtdbg.h header file, it will become possible
to enable that code again.

Comment 3

[David G King]

Hmmm, a thought, as WINE (www.winehq.com) has a crtdbg.h, couldn't Mozilla use
that one? (Yes, I know, WINE is for running Windows apps on top of X/Unix).

Comment 4

[Jonathan Wilson]

the crtdbg.h from WINE is licenced under LGPL.
If we can use LGPL stuff in mozilla without problems, we could use that.
We could also use other headers (zmouse.h, mapi stuff etc) to potentially fix
other MingW specific bugs.

Comment 5

[Eli Friedman]

Anything that still needs to be done here, considering that viewer is gone?

Comment 6

[cls]

Nope.