Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 374603 - Allow user to set a breakpoint filter
Summary: Allow user to set a breakpoint filter
Status: RESOLVED FIXED
Alias: None
Product: TCF
Classification: Tools
Component: Core (show other bugs)
Version: 1.0   Edit
Hardware: PC Linux
: P3 enhancement (vote)
Target Milestone: 1.0.0   Edit
Assignee: Project Inbox CLA
QA Contact: Eugene Tarassov CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-03-18 13:14 EDT by Scott Tepavich CLA
Modified: 2013-06-05 05:42 EDT (History)
1 user (show)

See Also:


Attachments
proposed patch (31.02 KB, patch)
2012-03-18 13:20 EDT, Scott Tepavich CLA
no flags Details | Diff
Updated patch (33.55 KB, patch)
2012-03-20 02:25 EDT, Scott Tepavich CLA
no flags Details | Diff
proposed patch (33.02 KB, patch)
2012-03-20 13:37 EDT, Scott Tepavich CLA
no flags Details | Diff
Patch to coincide with bug 373278. (39.44 KB, patch)
2012-03-21 02:28 EDT, Scott Tepavich CLA
eugene: iplog-
Details | Diff
proposed patch (41.23 KB, patch)
2012-04-03 18:39 EDT, Scott Tepavich CLA
no flags Details | Diff
proposed patch (41.41 KB, patch)
2012-04-05 19:03 EDT, Scott Tepavich CLA
mober.at+eclipse: iplog-
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Scott Tepavich CLA 2012-03-18 13:14:50 EDT
A user should be able to specify a ContextQuery filter for a given BP via the UI.
Comment 1 Scott Tepavich CLA 2012-03-18 13:20:48 EDT
Created attachment 212833 [details]
proposed patch

This is the first in a series of user stories which define a workflow for a user to set filters for breakpoints.

This patch covers US3233 and US2644.  These are basic functional stories which will be enhanced with user stories to follow.
Comment 2 Scott Tepavich CLA 2012-03-19 11:52:04 EDT
Incase the previously sited user stories cannot be retrieved, the sumary for these are:

Persist a context scoped breakpoint across debug sessions.

  As an AMP systems developer, I want to scope a breakpoint to a specific core name, and have it persist accross multiple debug sessions.  As I re-launch my debug session multiple times (including disconnect/reconnect ; recompile source and reload etc), I expect the breakpoint properties to persist such that I continue getting the same debug experience.
Comment 3 Scott Tepavich CLA 2012-03-20 02:25:21 EDT
Created attachment 212896 [details]
Updated patch

This updated patch adds basic expression format checking.
Comment 4 Scott Tepavich CLA 2012-03-20 13:37:06 EDT
Created attachment 212934 [details]
proposed patch

One last update, to alter the workflow of the expression parameter selection dialog.  This allows a user to build an expression.
Comment 5 Scott Tepavich CLA 2012-03-21 02:28:35 EDT
Created attachment 212968 [details]
Patch to coincide with bug 373278.

Patch which is merged with the changes to bug 373278, and so should only be referenced if 373278 is incorporated first.  If this is not the case, the last proposed patch should be referenced.
Comment 6 Scott Tepavich CLA 2012-04-03 18:39:41 EDT
Created attachment 213537 [details]
proposed patch
Comment 7 Eugene Tarassov CLA 2012-04-04 14:07:36 EDT
(In reply to comment #6)
> Created attachment 213537 [details]
> proposed patch

I'm getting assertion failures when I try to test the patch:

java.lang.AssertionError
	at org.eclipse.tcf.core.AbstractChannel.getRemoteService(AbstractChannel.java:740)
	at org.eclipse.tcf.internal.cdt.ui.breakpoints.TCFThreadFilterEditor.getAvailableAttributes(TCFThreadFilterEditor.java:369)
	at org.eclipse.tcf.internal.cdt.ui.breakpoints.TCFThreadFilterEditor.access$11(TCFThreadFilterEditor.java:363)
	at org.eclipse.tcf.internal.cdt.ui.breakpoints.TCFThreadFilterEditor$ExpressionSelectButton.handleEvent(TCFThreadFilterEditor.java:452)
	at org.eclipse.swt.widgets.EventTable.sendEvent(EventTable.java:84)
        ...

Could you, please, re-test it with asserts enabled (-ea), fix failures and re-submit the patch?
Comment 8 Scott Tepavich CLA 2012-04-05 19:03:45 EDT
Created attachment 213687 [details]
proposed patch

Sorry about that!  I've corrected the assert, did a quick review for other potential issues like this, and retested.  The attached no longer throws any assert when testing the workflows.
Comment 9 Eugene Tarassov CLA 2012-04-06 11:12:16 EDT
I have committed the patch.
Thanks!
Comment 10 Martin Oberhuber CLA 2013-05-24 18:23:04 EDT
Comment on attachment 213687 [details]
proposed patch

iplog- since git has the author:

http://git.eclipse.org/c/tcf/org.eclipse.tcf.git/commit?id=07fd3cb8c028dba668ee4905f76b3a024d950e75