Bugzilla – Full Text Bug Listing
| Summary: | [refactoring] Extract method refactoring failed | ||
|---|---|---|---|
| Product: | [WebTools] JSDT | Reporter: | Eugene Ostroukhov <eostroukhov> |
| Component: | General | Assignee: | Project Inbox <jsdt.javascript-inbox> |
| Status: | RESOLVED WORKSFORME | QA Contact: | Chris Jaun <cmjaun> |
| Severity: | normal | ||
| Priority: | P3 | CC: | cmjaun, narayan |
| Version: | 3.2 | Keywords: | needinfo |
| Target Milestone: | Future | ||
| Hardware: | PC | ||
| OS: | Mac OS X - Carbon (unsup.) | ||
| Whiteboard: | |||
|
Description
Eugene Ostroukhov
Do you have any example code? Steps to reproduce the above error.
Create a simple js file with the content below:
function a(){};
function b(){};
function c(){
a();
b();
a();
b();
}
In function c, try to extract the block
a();
b();
into a new function so the end result is
function d(){
a();
b();
}
function c(){
d();
d();
}
Thanks,
Narayan
In the above example, the extract to function d fails. (In reply to comment #2) > Steps to reproduce the above error. > > Create a simple js file with the content below: > > function a(){}; > function b(){}; > function c(){ > a(); > b(); > a(); > b(); > } > > In function c, try to extract the block > > a(); > b(); > > into a new function so the end result is > > function d(){ > a(); > b(); > } > > function c(){ > d(); > d(); > } > > Thanks, > Narayan This is working correctly in 3.5.1. Checked against the 7/18 build. |