From: Andre B. <and...@gm...> - 2003-05-15 15:55:46
|
Baptiste Lepilleur wrote: > There is a lot of stuff we don't need there. The only need for > refactoring are: > >// retrieve all the text > const std::string getAllText() const; >// retrieve a specified text range > > std::string getTextRange( const Refactoring::SourceRange &range ); > >// replace a specified text range > > void replaceTextRange( const Refactoring::SourceRange &range, > const std::string &text ); > > > This make the interface simpler and less ambiguous. With this interface, >there is no 'state'. It is simpler to implement and less ambigous. > > Does anybody have suggestion to further improve the interface ? > well don't forget the method // get an area/position selected by the user: virtual SourceRange getSelectionRange() const =0; since we need to know what text area is currently selected within a IDE. -- André |