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.

Bug 121270 - Incompatible API changes in debuggercore
Summary: Incompatible API changes in debuggercore
Status: NEW
Alias: None
Product: debugger
Classification: Unclassified
Component: Code (show other bugs)
Version: 6.x
Hardware: All All
: P2 blocker (vote)
Assignee: Martin Entlicher
URL:
Keywords: API
Depends on:
Blocks:
 
Reported: 2007-11-06 10:39 UTC by Martin Entlicher
Modified: 2007-11-06 10:59 UTC (History)
0 users

See Also:
Issue Type: TASK
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Entlicher 2007-11-06 10:39:54 UTC
There are several known problems in debuggercore APIs, which will caused incompatible API changed when fixed.

We will list/discuss the desired API changes here and then decide which release we will put this into.
Comment 1 Martin Entlicher 2007-11-06 10:59:05 UTC
One problem is in DebuggerManager. 

code like
                dl.breakpointAdded (breakpoint);
                dl.propertyChange (ev);
and
            ((DebuggerManagerListener) l.elementAt (i)).watchRemoved (watch);
            // TODO: fix nonsense double firing
            ((DebuggerManagerListener) l.elementAt (i)).propertyChange (ev);
does not look right, but can not be changed in a compatible way.

Watch initialization is implemented via DebuggerManagerListener and (unlike breakpoint initialization) the listener
implementation is supposed to add the created watches explicitely into DebuggerManager. This makes it impossible to
distinguish between creation and addition.

createWatch() is public, but removeWatch() is package-private and called from Watch.remove() - inconsistent.

Watch is a final class - encapsulation of String, but Breakpoint is an abstract class. Perhaps this is O.K., but the
reasons are not clear.