Item-repository allocation/locking refactor

Milian Wolff mail at milianw.de
Wed Dec 19 16:43:29 UTC 2012


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.

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/20121219/c1022104/attachment.sig>


More information about the KDevelop-devel mailing list