<div class="gmail_quote">On Fri, Oct 2, 2009 at 6:04 PM, Hugo Parente Lima <span dir="ltr"><<a href="mailto:hugo.pl@gmail.com">hugo.pl@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div><div></div><div class="h5">On Friday 02 October 2009 20:54:57 Carlos Licea wrote:<br>
> Dear list,<br>
>       I've just found a problem, consider the following classes:<br>
><br>
>       class A<br>
>       {<br>
>               virtual someFunction();<br>
>       }<br>
><br>
>       class B : public A<br>
>       {<br>
>               virtual someFunction();<br>
>       }<br>
><br>
> Now I tried to rename B::someFunction() because the way it was being done<br>
>  was more accurate for an internal function... but the change propagated to<br>
>  the base class, so in the end I had something like this:<br>
><br>
>       class A<br>
>       {<br>
>               virtual someFunctionPrivate();<br>
>       }<br>
><br>
>       class B : public A<br>
>       {<br>
>               virtual someFunctionPrivate();<br>
>       }<br>
><br>
> When I expected<br>
><br>
>       class A<br>
>       {<br>
>               virtual someFunction();<br>
>       }<br>
><br>
>       class B : public A<br>
>       {<br>
>               virtual someFunctionPrivate();<br>
>       }<br>
><br>
> So that I could later reimplement someFunction properly...<br>
><br>
> I think that the current behavior is kind of unexpected, at least for me; I<br>
> don't think that changing the derived class should change the base class...<br>
>  at least not by default.<br>
> I think that asking the user if he wants to propagate the changes from a<br>
> derived/base class to a derived/base class when renaming a virtual function<br>
>  is the proper thing to do. What do you think?<br>
<br>
</div></div>The way it works is right, you are just a ultra corner case :-P, but asking<br>
the user is always the best way.<br>
<div><div></div><div class="h5"><br>
> Carlos<br>
><br>
> _______________________________________________<br>
> KDevelop-devel mailing list<br>
> <a href="mailto:KDevelop-devel@kdevelop.org">KDevelop-devel@kdevelop.org</a><br>
> <a href="https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel" target="_blank">https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel</a><br>
><br>
<br>
</div></div><font color="#888888">--<br>
Hugo Parente Lima<br>
"Precisamos de mais gênios humildes no mundo, hoje somos poucos!"<br>
JID: <a href="mailto:hugo@jabber.org">hugo@jabber.org</a><br>
</font><br>_______________________________________________<br>
KDevelop-devel mailing list<br>
<a href="mailto:KDevelop-devel@kdevelop.org">KDevelop-devel@kdevelop.org</a><br>
<a href="https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel" target="_blank">https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel</a><br>
<br></blockquote></div><br>I don't really think asking the user is the best way, and I don't think this is a corner case.<br><br>If you wanted to differentiate, you first had to rename the class B method to tell KDevelop it's something different.<br>