News from SharedUrls land

Andreas Pakulat apaku at gmx.de
Sat Jan 5 10:57:53 UTC 2013


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.

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 :)

Andreas


More information about the KDevelop-devel mailing list