Closed Bug 183074 Opened 22 years ago Closed 13 years ago

don't hardcode c:\ on heapdump

Categories

(Core :: Widget: Win32, enhancement)

Product:
Core
Component:
Widget: Win32
Platform:
x86
Windows 2000
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED INVALID

People

(Reporter: timeless, Unassigned)

References

(
URL
)

Details

// 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...
Product: Browser → Seamonkey
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
Closed: 13 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.