<div dir="ltr"><div>First off, thanks for all the reviews!<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">We simulate a "virtual://" protocol[*] which can contain virtual<br>folders containing references to real files and folders.<br></blockquote><div><br></div><div>I did not know that such a feature exists! It seems like this feature is Krusader specific without tapping into KIO as such?<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">It would be great if we can get rid of this "Krusader only" solution and replace it with<br>your *real* KIO protocol. Some questions: Is it persistent (between reboots)? And does it<br>support general file URLs (e.g. <a href="ftp://localhost/path/to/file.txt">ftp://localhost/path/to/file.txt</a>) or only local<br>"file://"-files?</blockquote><div><br></div><div>The list of files is not persistent across reboots. That idea was scrapped fairly early on during the development of this project. It could probably be implemented using xattr if such a feature is needed. I haven't tested much on other protocols, but it does have problems copying from some protocols such as mtp.<br><br></div><div>That said, being a kioslave, it works well with Dolphin, the KDE Folder view Plasmoid, and Konqueror for file operations.<br></div><div><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">* ../src/iodaemon/stashnotifier.<wbr>cpp:183:9: warning: variable 'fileType' is uninitialized when used here <br>
This should probably be fixed.<br>
<br>
* You are linking to KI18n but you are not using i18n() calls in your code. Have a look at [1].<br>
<br>
* The dbus adaptor could probably use build-time generation via cmake, 
rather than being committed to the git repo. If you need an example look
 at the CMakeLists.txt in kio/src/kioexec (qt5_add_dbus_adaptor and 
friends).<br>
</blockquote></div><div><br></div><div> Thanks, I have fixed the first two issues. My response to the last issue is given below.<br><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Files that contain autogenerated code should ideally not be "edited".<br>
<br>
Do you remember why editing was needed?<br></blockquote><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-im">
</span>Not exactly, Arnav will know that. But it's not really too big a<br>
problem for a project this size. In fact I do the same thing in<br>
Spectacle.<br></blockquote>
<br></div><div>I believe we needed to make some specific changes to the autogenerated XML at some point in the project. These changes were later scrapped, so it uses the directly autogenerated code with no modifications. As Boudhayan said, the justification for not changing it to be invoked on compilation was for simplicity and that the D-Bus code was rarely updated throughout the development of the project.<br><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Maintainers will leave and we will inherit the code, so there will be noone to<br>
ask.<br>
</blockquote><div><br></div><div>I will be available to maintain this code when it gets released in the KDE software ecosystem. <br></div></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, May 27, 2017 at 2:53 PM, Albert Astals Cid <span dir="ltr"><<a href="mailto:aacid@kde.org" target="_blank">aacid@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">El dissabte, 27 de maig de 2017, a les 0:29:51 CEST, Boudhayan Gupta va<br>
<div><div class="h5">escriure:<br>
> Hi,<br>
><br>
> On 27 May 2017 at 00:20, Albert Astals Cid <<a href="mailto:aacid@kde.org">aacid@kde.org</a>> wrote:<br>
> > El divendres, 26 de maig de 2017, a les 23:48:18 CEST, Boudhayan Gupta va<br>
> ><br>
> > escriure:<br>
> >> Hi,<br>
> >><br>
> >> On 26 May 2017 at 20:31, Elvis Angelaccio <<a href="mailto:elvis.angelaccio@kde.org">elvis.angelaccio@kde.org</a>><br>
wrote:<br>
> >> > * The dbus adaptor could probably use build-time generation via cmake,<br>
> >> > rather than being committed to the git repo. If you need an example<br>
> >> > look<br>
> >> > at<br>
> >> > the CMakeLists.txt in kio/src/kioexec (qt5_add_dbus_adaptor and<br>
> >> > friends).<br>
> >><br>
> >> IIRC we did that first, but then figured we needed to edit the<br>
> >> generated code (I was the mentor for this project).<br>
> ><br>
> > That seems dangerous, what if sometime in the future you need to<br>
> > regenerate<br>
> > the adaptor because you add new functions or something?<br>
> ><br>
> > Files that contain autogenerated code should ideally not be "edited".<br>
> ><br>
> > Do you remember why editing was needed?<br>
><br>
> Not exactly, Arnav will know that. But it's not really too big a<br>
> problem for a project this size. In fact I do the same thing in<br>
> Spectacle.<br>
><br>
> Granted this isn't best practice, but the effort required to do it<br>
> right (whatever way that might be) was inordinately large for the<br>
> scope of the project. At some point someone who's trying to patch this<br>
> part of the code might get stuck, but figuring out that generated code<br>
> was edited isn't difficult at all, as neither is asking the maintainer<br>
> how the code works.<br>
<br>
</div></div>Maintainers will leave and we will inherit the code, so there will be noone to<br>
ask.<br>
<br>
If you need to do some changes to generated code either document it or just<br>
don't do changes in the generated code.<br>
<br>
Cheers,<br>
  Albert<br>
<div class="HOEnZb"><div class="h5"><br>
><br>
> Thanks,<br>
> Boudhayan<br>
><br>
> > Cheers,<br>
> ><br>
> >   Albert<br>
> ><br>
> >> > Cheers,<br>
> >> > Elvis<br>
> >> ><br>
> >> > [1]: <a href="https://api.kde.org/frameworks/ki18n/html/prg_guide.html" rel="noreferrer" target="_blank">https://api.kde.org/<wbr>frameworks/ki18n/html/prg_<wbr>guide.html</a><br>
> >><br>
> >> Freundliche Grüße<br>
> >> Boudhayan Gupta<br>
> >> KDE e.V. - Sysadmin and Community Working Groups<br>
> >> +49 151 71032970<br>
<br>
<br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><span style="font-family:arial,helvetica,sans-serif"></span><span style="font-family:arial,helvetica,sans-serif">arnav dhamija<br></span></div></div></div></div></div>
</div>