Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 335202 - [Extract Local Variable] Suggested name is wrong for nested function calls
Summary: [Extract Local Variable] Suggested name is wrong for nested function calls
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-refactoring (show other bugs)
Version: 7.0.2   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 8.0   Edit
Assignee: Marc-André Laperle CLA
QA Contact: Emanuel Graf CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-01-24 10:25 EST by Marc-André Laperle CLA
Modified: 2011-04-07 09:23 EDT (History)
2 users (show)

See Also:

Attachments
Extract Local Variable, nested function calls patch (4.02 KB, patch)
2011-01-24 10:29 EST, Marc-André Laperle CLA 		no flags Details | Diff
Extract Local Variable, nested function calls patch (4.49 KB, patch)
2011-02-19 01:04 EST, Marc-André Laperle CLA 		malaperle: iplog-
Details | Diff
mylyn/context/zip (1.31 KB, application/octet-stream)
2011-02-19 01:04 EST, Marc-André Laperle CLA 		no flags Details
Show Obsolete (1) 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 Marc-André Laperle CLA 2011-01-24 10:25:46 EST 
Example:

getFoo(getBar())

If I select the whole thing, 'bar' is suggested. Similarly, 

getFoo(getBar(getFooBar()))

fooBar is suggested.
Comment 1 Marc-André Laperle CLA 2011-01-24 10:29:26 EST 
Created attachment 187429 [details]
Extract Local Variable, nested function calls patch

Patch + tests.
Comment 2 Marc-André Laperle CLA 2011-02-19 01:04:39 EST 
Created attachment 189338 [details]
Extract Local Variable, nested function calls patch

Updated copyrights
Comment 3 Marc-André Laperle CLA 2011-02-19 01:04:41 EST 
Created attachment 189339 [details]
mylyn/context/zip
Comment 4 Marc-André Laperle CLA 2011-04-07 08:41:44 EDT 
Fixed in HEAD.