Item-repository allocation/locking refactor

Milian Wolff mail at milianw.de
Wed Jan 9 21:13:53 UTC 2013


On Wednesday 19 December 2012 17:43:29 Milian Wolff wrote:
> On Wednesday 19 December 2012 17:33:56 Ivan Shapovalov wrote:
> > On 06 December 2012 23:44:10 Milian Wolff wrote:
> > > On Wednesday 05 December 2012 10:11:13 Ivan Shapovalov wrote:
> > > > Hi!
> > > > 
> > > > Following Milian's suggestion on
> > > > https://git.reviewboard.kde.org/r/106945/,
> > > > I'm putting the work on refactoring the "disk storage allocation" part
> > > > of
> > > > the item-repositories into a public repo instead of a review-request.
> > > > 
> > > > The refactoring is just about
> > > > - getting rid of locking in the item-repository directories (because
> > > > we
> > > > have rather good per-session locks);
> > > > - getting rid of numbered subdirectories inside of per-session cache
> > > > directories (because nobody shall use and nobody actually uses
> > > > multiple
> > > > ItemRepositoryRegistries);
> > > > - changing the cache path once again
> > > > 
> > > >   from "~/.cache/kdevduchain/$session"
> > > >   to "~/.cache/kdevduchain/$appname-$session";
> > > > 
> > > > - making ItemRepositoryRegistry a true singleton;
> > > > - finally making the cache directories deleted when a session is
> > > > deleted
> > > > from disk.
> > > > 
> > > > The real reason behind all this was the last point.
> > > > 
> > > > The branch is called "itemrepository" and is on
> > > > "git://github.com/intelfx/kdevplatform.git".
> > > 
> > > Cool!
> > > 
> > > Some points I spotted so far - please don't be overwhelmed by that...
> > > But
> > > I
> > > rather want it to merge a nicely designed refactoring which should make
> > > this stuff future proof.
> > 
> > Please, review the next iteration. It's on the same branch, rebased
> > against
> > current master (non-fast-forward, of course).
> 
> <snip>
> 
> Wow, you seem to be able to read minds. I was about to ping you about the
> status here. Awesome work, I'll try to review it as soon as possible.

Reviewing it now and fixing it up and making it ready for merging. You btw 
introduced a cyclic dependency (language required shell and vice versa). I'm 
breaking that up now and cleaning up the session controller/lock code while at 
it.

Cheers
-- 
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/20130109/9b5565e8/attachment.sig>


More information about the KDevelop-devel mailing list