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 146018 - invalid profiling point still visible
Summary: invalid profiling point still visible
Status: NEW
Alias: None
Product: profiler
Classification: Unclassified
Component: Base (show other bugs)
Version: 6.x
Hardware: Macintosh All
: P3 blocker (vote)
Assignee: Jiri Sedlacek
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-09-03 10:52 UTC by Kenneth Ganfield
Modified: 2009-05-25 21:03 UTC (History)
0 users

See Also:
Issue Type: ENHANCEMENT
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Kenneth Ganfield 2008-09-03 10:52:20 UTC
If I use the popup menu in source code to insert profiling point, in some cases i see the warning that the location of
the profiling point is invalid and warning that it will not be hit. Though I am told the profiling point is in an
invalid location, the pp appears to be created, the icon is visible in gutter, is listed in profiling points window and
in list of active profiling points. I can also create the profiling point in an invalid location in the profiling points
window.

The behavior seems a bit unclear to me.
If the profiling point will not be hit is there a reason to create it there?
Also, should this be indicated somehow to distinguish the pp from pps in valid locations?
Right now there is no way to visibly see which profiling points will be hit/are valid.
Comment 1 Jiri Sedlacek 2008-09-03 12:52:11 UTC
This is caused by fixing Issue 118529. As mentioned there, the algorithm is just a best guess if the location is
invalid, it's possible that it's actually correct and that's why the profiling point isn't disabled/removed. Currently
it works as intended, in future we should improve the location validity check and for example disable the invalid
profiling points.

I think that marking invalid profiling points is an enhancement, in the current version the dialog correctly suggests to
check the location but using an invalid profiling point doesn't break any functionality. Changing to ENHANCEMENT.
Comment 2 Tomas Hurka 2009-04-08 12:50:50 UTC
Milestone cleanup: future->next