Description Adam Kiezun 2002-03-15 07:03:34 EST

extract method could replace more than 1 occurence of extracted code

in the simplest case - if extracting 1 expression, it could replace all 
occurences in the current method

int m(int p){
   int i= /*[*/m(m(1))/*]*/;
   int j= m(m(1)) + m(m(1));
   return p+1;
}
could be refactored to:

int m(int p){
   int i= three();
   int j= three() + three();
   return p+1;
}

in a more compilacted case, it could find occurences of code in the same cu
(but then, you must compare bindings - identified 'i' may mean different things 
in different methods)
and, it's not matching 1 node then - you must match a sequence of nodes possibly

i'd be happy with the simple case (although it's possible to achieve the same 
goal by using 'extract temp' and 'inline temp')