Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 318993 - [debug] private role method bridge is interpreted as callin wrapper
Summary: [debug] private role method bridge is interpreted as callin wrapper
Status: VERIFIED FIXED
Alias: None
Product: Objectteams
Classification: Tools
Component: OTDT (show other bugs)
Version: 0.7   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 0.7.1   Edit
Assignee: Stephan Herrmann CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-07-06 07:48 EDT by Stephan Herrmann CLA
Modified: 2010-09-23 13:42 EDT (History)
0 users

See Also:

Attachments
implementation (2.81 KB, patch)
2010-09-04 14:43 EDT, Stephan Herrmann CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Stephan Herrmann CLA 2010-07-06 07:48:10 EDT 
When stepping into a bridge for a private role method, this bridge
is beautified in the Debug view like a callin wrapper, so using
Team1122lt8_2 (from TeamNesting tests), and using a BP on R2#k->k
and then hitting F5 we see:
  Team1122lt8_1$R1.[R1.private<-k](int, Team1122lt8_1, Team1122lt8_1$R1) line: 7
  Team1122lt8_1.[R1.private<-k]() line: 7	
  Team1122lt8_2$R2.k() line: 11	
The first 2 frames are colored green.

Instead they should be colored pale-purple and say something like
"accessing private method k()"

Also: how about using STEP-INTO linenumber for these?
Comment 1 Stephan Herrmann CLA 2010-09-04 14:43:11 EDT 
Created attachment 178213 [details]
implementation

This patch beatifies a number of different access methods:
 - _OT$create$R : accessor for role constructor
 - _OT$InitFields: field initializations extracted from constructors
 - _OT$decaps$xy : decapsulation access
 - _OT$R$private$m: callout access to private role method
Comment 2 Stephan Herrmann CLA 2010-09-04 14:44:13 EDT 
Patch has been committed as r759
Comment 3 Stephan Herrmann CLA 2010-09-23 13:42:54 EDT 
Verified using I201009211735