Hi vladimir,<br>If you agree, I will commit it today and i'll wait to move the proper things to the platform to 4.1, I don't have much time atm and I'd prefer to spend it getting kdevelop stable.<br><br>Thanks,<br>
Aleix<br><br><div class="gmail_quote">On Wed, Dec 3, 2008 at 10:53 PM, Vladimir Prus <span dir="ltr"><<a href="mailto:ghost@cs.msu.su">ghost@cs.msu.su</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="Wj3C7c">On Thursday 20 November 2008 01:19:00 Aleix wrote:<br>
> hi list (and in special vladimir),<br>
><br>
> I spent some time trying to split this breakpoints code from the gdb support<br>
> to the platform so that we can take advantage from it for new future<br>
> debuggers. I don't really know if it is the best approach (i'm not really<br>
> familiar with this code) but i think it is not breaking the code logic and<br>
> could be useful enough.<br>
><br>
> some more code would be able to be ported to the platform like specially the<br>
> widget but i prefer to go step by step.<br>
><br>
> obviously here the files that would go to the platform would be the i* ones.<br>
<br>
</div></div>Hi Aleix,<br>
<br>
sorry for delayed response. I've tried your patch, and found that it does<br>
not break anything, so please commit it. As I've said on IRC, I'd prefer that<br>
classes that are not pure interfaces be not called IWhatever, but otherwise<br>
seems good. Please ping me on IRC if you plan further refactorings -- I've noticed<br>
a few bugs/regression that I want to fix, so we better coordinate to avoid<br>
merge messup later.<br>
<br>
Thanks,<br>
Volodya<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>
</blockquote></div><br>