I had MREUser hardcoded as the default app in the uninstaller. Not only should this not be hardcoded, but it has since changed to GREUser.
Comment on attachment 102396 [details] [diff] [review] Patch 1 r=jbetak (seems fairly straightforward :-)
Attachment #102396 - Flags: review+
Comment on attachment 102396 [details] [diff] [review] Patch 1 >Index: wizard/windows/uninstall/extra.c >+ char szDefaultApp[MAX_BUF]; 4K is an insanely wasteful size for this string. (Beware complaining it's the same as the other variables nearby or I'll ask you to fix those too.) See setup.h and the various smaller sized buffers it defines. MAX_BUF_TINY is still huge for this string, but wastes 20x less space. sr=dveditz with that change
Attachment #102396 - Flags: superreview+
Comment on attachment 102396 [details] [diff] [review] Patch 1 a=asa for checkin to 1.2beta (on behalf of drivers)
Attachment #102396 - Flags: approval+
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
20021021 Trunk Build - Verified
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.