Rename Assistant

Olivier JG olivier.jg at gmail.com
Fri Oct 29 03:02:37 UTC 2010


On 10/28/2010 07:19 PM, David Nolden wrote:
>
> PersistentMovingRange is simpler in the sense that you don't need to 
> listen to any signals from MovingInterface etc.. Adding a bool flag 
> should be fine.
>
> About deciding when to offer the assistant: You can catch local 
> declarations by checking "decl->context()->kind() == 
> DUContext::Other", and function-arguments with "...->kind() == 
> DUContext::Function", and that should be all places where this should 
> be allowed.
>
> Unless the assistant causes grave problems, you can commit it.
>
> Greetings, David
>
I went ahead and pushed the rename assistant, I will of course modify 
the rename conditions as needed.
-Olivier JG




More information about the KDevelop-devel mailing list