// XXX for now we use c:\heapdump.txt until we figure out how to pass in message parameters. please don't hardcode drive letters. My prefered wNT installation doesn't have a c: you could use an env var, or a message, or ...
How about %TEMP%\heapdump.txt? Win9x doesn't have the greatest selection of environment variables that could be used for this (TEMP, TMP, and WINDIR are all I can think of) and it's probably not worth the bother of separating 9x and NT at runtime.
i was thinking more like %MOZ_HEAPDUMP% since anyone who's going to use this feature could certainly set it. there's actually a win32 function to get the os temp dir, and there is also the possibility to dump the file anywhere the directory service offers if we wanted to go one of those routes...
Assignee: dp → general
QA Contact: asa → general
Assignee: general → nobody
Component: General → Widget: Win32
Product: SeaMonkey → Core
QA Contact: general → win32
Summary: don't hardcode c:\ → don't hardcode c:\ on heapdump
This debug code was removed long ago.
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.