News from SharedUrls land

[Milian Wolff]

On Saturday 05 January 2013 11:57:53 Andreas Pakulat wrote:
> Hi,
> 
> finally got around to actually look at the branch.
> 
> On Wed, Dec 5, 2012 at 3:02 AM, Milian Wolff <mail at milianw.de> wrote:
> > Hey all,
> > 
> > I want to invite you all to review my sharedurls branch and maybe even
> > start helping out in the refactoring/proting effort. It's mostly about
> > finding deprecation warnings and using KDevelop::Path instead of KUrl
> > where appropriate.
> > 
> > I decided to follow Andreas' suggestion of naming the new type "Path"
> > instead of URL or such, since it really is that - a path. It might be
> > remote or local, yes. But it still points to a file or folder and cannot
> > have any fancy URL features like fragments or queries.
> 
> For the API questions I've opened a review since that makes it easier
> to talk about the code.

Yep, I also replied already - many thanks for that!

> I'm also thinking that for making the porting easier it would be good
> to introduce a commit which removes all the deprecated API, so that
> the compile actually fails where porting is needed. You can later on
> rebase the branch removing this commit again to keep third-party
> plugins working (or just do a git revert) but I was surprised when I
> noticed that all things still compile, I was looking for the errors to
> see where I have to start porting :)

I did this deliberately to be able to actually use a KDevplatform+KDevelop 
build against that branch to see how it performs.

My idea is that everyone who would like to help the porting effort picks a 
deprecated method of his choice, removes it and tries to compile. Then fix all 
the compile errors and be done with it.

This way the branch stays usable (and we can continue to investigate the 
performance etc. pp.).

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/20130105/1dde4270/attachment.sig>