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

Bug 324888

Summary: Need to subclass CSourceHover
Product: [Tools] CDT Reporter: Violaine Batthish <batthish>
Component: cdt-coreAssignee: Vivian Kong <vivkong>
Status: RESOLVED FIXED QA Contact: Doug Schaefer <cdtdoug>
Severity: normal    
Priority: P3 CC: aleherb+eclipse, ankitp, recoskie, vivkong
Version: 7.0Flags: aleherb+eclipse: review+
Target Milestone: 7.0.1   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
patch for protected methods vivkong: iplog+

Description Violaine Batthish CLA 2010-09-09 14:45:20 EDT
Created attachment 178545 [details]
patch for protected methods

I need to subclass CSourceHover, and I need for the following methods to be made protected so that I can add my code:
SingletonRule
ComputeSourceRunnable
searchInIndex

I have attached a patch
Comment 1 Chris Recoskie CLA 2010-09-09 14:57:29 EDT
Other than the usual caveats about the dangers of subclassing internals, I don't really see a problem with making these things protected instead of private.  Toni, do you have any thoughts?
Comment 2 Anton Leherbauer CLA 2010-09-13 04:44:31 EDT
That's fine with me.
Comment 3 Vivian Kong CLA 2010-09-14 10:33:19 EDT
Applied to HEAD and cdt_7_0.  Thanks.