This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.
1. it should be a question dialog, not warning dialog 2. it uses language which might be correct from the debugger implementation perspective (pop stack/leave/...), but this tells nothing to the average user - suggest to use human terminology like "Call the last method again" 3. the text should be reworded so that the user makes direct mental connection between the buttons and the action described (ie. something like: "Press button A to call this method again, press button B to do whatever else...")
1. After discussion with HI folks, it is agreed that this should be a question alert box. 2. I agree that the wording can probably be improved. Within the current text, there is an inconsistency (is the thing on the stack a method or a call?). (in general, we probably shouldn't be using the word 'method'). I'm reassigning this issue to Gail to find the right phrasing. 3. This, also, sounds good, particularly in this case. There are no HI standards that make this manditory, but I agree that this is probably needed given the complexity of the issue. Again, though, I think this is in Gail's court to make sure the wording is good and consistent with what gets documented. FWIW, the current wording is: *Old Method on Stack* The most recent call on the current stack blongs to one of the classes that you fixed. you can pop the call so that you will then call the new version. [[&Pop Call]] [&Leave Call] [&Help] (I'm moving this to debuggerjpda since this alert is specific to the jpda debugger. other debuggers will have different alerts).
David John, Gail, do you have some idea how to fix it?
Gail should make the final decision about the words. I would recommend: *Old Call on Stack* The most recent call on the stack is associated with one of the classes that you fixed. To remove this call so that the fixed methid will be called, click on "Pop Call", otherwise click on "Use Old". [[&Pop Call]] [&Use Old] [&Help] I know this still uses the word "pop" and "stack", but these are terms used elsewhere in the interface, so it seems unnecessary to try to avoid them (and I am assuming the majority of the people using this product will know what a stack is and about push and pop operations on it. I assume this is taught in most CS programs all around the world).
Nice job, David John. I agree with using the terms stack and pop since they are both used in the interface and in other companies' debugger interfaces. Here is an update of David John's write-up with some minor edits. *Old Call on Stack* The most recent call on the stack is associated with one of the classes that you fixed. To remove this call so that the fixed method is called, click the Pop Call button. Otherwise, click the Use Old button. [[&Pop Call]] [&Use Old] [&Help]
fixed in the main trunk
Verified.