<div dir="ltr"><div><div>Thanks for all the feedback,<br><br></div>I'll prepare the patches ASAP.<br></div><div>* Remove bookmarks syncing code from 
KFilePlacesModel (KF5 and kdelibs4)<br></div><div>* Use user-places.xbel instead of bookmarks.xml in KFilePlacesModel and Dolphin (KF5 and kdelibs4)<br><br></div><div>Cheers,<br></div><div>Emmanuel<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-04-25 22:43 GMT+02:00 David Faure <span dir="ltr"><<a href="mailto:faure@kde.org" target="_blank">faure@kde.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Thursday 23 April 2015 18:24:26 Marco Martin wrote:<br>
> On Thursday 23 April 2015, Emmanuel Pescosta wrote:<br>
> > So the possible solutions I see are:<br>
> > * Store the bookmarks in one file (user-places.xbel) and remove the<br>
> > syncing<br>
><br>
> I like the idea of storing places bookmarks only in a cross desktop location<br>
<br>
</span>Me too. It would certainly make the code much simpler than the two-way sync<br>
logic.<br>
<span class=""><br>
> I think they were initially synced in two places due to some legacy<br>
> reason?<br>
<br>
</span>Yes. Support for user-places.xbel was added on top of the older bookmarks.xml<br>
code.<br>
<span class=""><br>
> what could be drawbacks of using only user-places.xbel ?<br>
<br>
</span>I wonder too.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
David Faure, <a href="mailto:faure@kde.org">faure@kde.org</a>, <a href="http://www.davidfaure.fr" target="_blank">http://www.davidfaure.fr</a><br>
Working on KDE Frameworks 5<br>
<br>
</font></span></blockquote></div><br></div>