Closed Bug 1412 Opened 26 years ago Closed 26 years ago

"Top 100 Sites" Viewer.exe Menu Item doesn't work in optimized builds

Categories

(Core Graveyard :: Viewer App, defect, P2)

x86
Windows 95
defect

Tracking

(Not tracked)

VERIFIED WONTFIX

People

(Reporter: angus, Assigned: rickg)

References

Details

We need to either: 1) Remove the menu item from optimized builds 2) Make it work from optimized builds I think option #2 is preferable. Note this probably affects the xpviewer.exe, too (cc'ing those guys)
Assignee: troy → angus
Component: Viewer%20App → Viewer App
I have nothing to do with the viewer...
Terrific news from Bill <law@netscape.com> -- cutting and pasting from email --: "I fixed this in the xpviewer. I imagine the viewer needs the same tweak. The code that looks for the "Top 100" menu choice selection was #ifdef'd out in non-debug builds. I added a case outside the debug menu routine to look for this one item, but I also had to expose the declaration/definition of the function that implements it. I'll take a look at the viewer code when I get a chance, if no one else does. Bill"
Assignee: angus → rickg
Reassigning to rick for viewer.
Assignee: rickg → jevering
It's your call what to do on this one.
*** Bug 1306 has been marked as a duplicate of this bug. ***
Assignee: jevering → rickg
Im not maintaining viewer now, rick who owns this now?
Status: NEW → RESOLVED
Closed: 26 years ago
Resolution: --- → WONTFIX
Look, the viewer is a test app. It's not a good use of time to have an engineer fix this when it will be going away soon.
Status: RESOLVED → VERIFIED
Old bug...marking Verified/Won't Fix.
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.