KDevPlatform Itemrepository bucket freeItems

[Milian Wolff]

David Nolden, 04.03.2011:
> 2011/3/4 floris <flo.ruijt at gmail.com>:
> > On Thu, 2011-03-03 at 22:13 +0100, Milian Wolff wrote:
> >> floris, 01.03.2011:
> >> > On Tue, 2011-03-01 at 10:15 +0100, Milian Wolff wrote:
> >> > > you have quite some in-depth knowledge on these
> >> > > tasks, many of us (esp. me) don't have that. I for one am just a
> >> > > self-taught
> >> > > programmer and hence would greatly appreciate it if you could
> >> > > shortly outline
> >> > > what you want to achieve an
> >> > 
> >> > well a short introduction then :)
> >> 
> >> ok thanks, do I see that correctly that this is supposed to be used in
> >> https://git.reviewboard.kde.org/r/100730/diff/#index_header or is this
> >> for usage somewhere else?
> >> 
> >> bye
> > 
> > no this is to be used in
> > kdevplatform/language/duchain/repositories/itemrepository.h
> > 
> > I made major edits there, but the code is still failing(sometimes
> > segfaults sometimes assertion failures). I just wanted to show something
> > before i finished it, as the patch will be large.
> > 
> > btw it's pretty annoying that a major part of kdevelop/kdevplatform
> > includes itemrepository.h(directly or indirectly) which means a
> > recompile of 15-20 minutes after each edit.
> > 
> > Floris
> 
> You should create a unit-test for testing this, see for example
> test_Itemrepository.cpp. Then you also need to recompile only that one
> file, instead of whole kdevplatform.

Considering that this touches a core part of the language integration, heavy 
unit tests are a requirement anyways.

And itemrepository.h is simply used in so many places, hence there is nothing 
we can do about it to prevent long recompile times.

bye
-- 
Milian Wolff
mail at milianw.de
http://milianw.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20110304/eb7756f5/attachment.sig>