don't hardcode c:\ on heapdump

RESOLVED INVALID

Status

()

Core
Widget: Win32
--
enhancement
RESOLVED INVALID
15 years ago
7 years ago

People

(Reporter: timeless, Unassigned)

Tracking

Trunk
x86
Windows 2000
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Reporter)

Description

15 years ago
// 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 ...

Comment 1

15 years ago
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.
(Reporter)

Comment 2

15 years ago
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

Updated

9 years ago
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

Comment 3

7 years ago
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.