Closed Bug 11152 Opened 21 years ago Closed 20 years ago
Browser App Core .cpp code cleanup
Stumbled across this gem at the above URL. Probably don't need to convert to a C string only to convert back to unicode two lines later. Looks this is a product of too many chefs in the kitchen :-)
Component: Browser-General → XPApps
Priority: P3 → P2
Target Milestone: M10
The code in question has changed so the URL doesn't seem to point to the bogus code any longer. I'm not quite sure what to look for in the nearby code. Do you happen to remember more about it (what function, what the current line number is, what the code looked like)? I'll go back and fetch this file by date and see what was at that line number, but it's not a top priority and it might save time if you could provide a more up-to-date pointer. Thanks.
Moving this off to a milestone beyond the foreseeable future...
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
nsBrowserAppCore is history so I'm marking this fixed. The code in question might well be in the replacement nsBrowserInstance.cpp but I have no way of knowing since I still don't know what the code was.
don...ok to mark Verified?
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.