Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 333443

Summary: The ImportDialog for the transport services has a protected method which is required to be public
Product: [Tools] CDT Reporter: Randy Rohrbach <Randy.Rohrbach>
Component: cdt-memoryAssignee: Randy Rohrbach <Randy.Rohrbach>
Status: RESOLVED FIXED QA Contact: Ted Williams <ted>
Severity: normal    
Priority: P3 CC: cdtdoug, Randy.Rohrbach
Version: 7.0.1   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
Changes a protected method to be public cdtdoug: iplog-

Description Randy Rohrbach CLA 2011-01-03 20:18:20 EST
The ImportDialog for the transport services has a protected method which is required to be public.

The method scrollRenderings(BigInteger address) is protected. This has
not been a problem since all of the transports to date have been done
within the same package. 

I am creating a proprietary transport and when I do this method comes up
as unavailable.

It needs to be public. This is technically a change of an API, but I unless
I use reflection to get to this method, it needs to be changed.

I am creating a patch to show this and will discuss, it with the community before committing it.

Randy
781-364-2226
Wind River
Comment 1 Randy Rohrbach CLA 2011-01-03 20:21:54 EST
Created attachment 185979 [details]
Changes a protected method to be public

It is the case that this method is standardly used in all if the various importers. I am creating a custom importer and do not want to loose the
functionality. It should have been public in the first place.
Comment 2 Randy Rohrbach CLA 2011-01-07 22:31:01 EST
Well I got only one comment in response and it was a vote for.

So I am applying this change to 7.0.x and 8.0

Randy
Comment 3 Randy Rohrbach CLA 2011-01-07 22:38:02 EST
I checked the change in to 7.0.1 and 8.0