[PATCH] devicenotifier and dolphin interactive problem

sonald sycao at redflag-linux.com
Thu Jan 7 07:59:14 GMT 2010


> First a general comment on your patches: Be careful with their 
> formatting,
> there's a couple of places where you mix tabs and spaces for their indenting
> while you probably wanted to use only spaces.
>    
en, sorry for that. I just changed my emacs setting and convert all tabs 
to spaces.
> That was rather on the nitpick level, now let's move on a more fundamental
> level.
>
> I can't really comment on the Dolphin's patch, that'd be more Peter's role.
>
> Now on the KFilePlacesModel patch, I think I need some more information on
> your approach. I somehow doubt that a new accessibilityChanged() signal is
> needed there, as the intent was to have dataChanged() cover that (it's
> supposed to be emitted when the accessibility of the device indeed changes).
> In turn that'd mean that on the Dolphin side connecting to dataChanged() would
> be enough as well.
>
> Regards.
>    
I don't know if I understood correctly. dataChanged only affects 
KFilePlacesView and there is no way
for an client like dolphin to know it. I see that only connection 
between dolphin and PlacesPanel is the
urlChanged signal. I believe that's not enough to solve the problems I 
mentioned in the patch, cause I
need to know the url and state(removed/mounted) to do the right thing. 
maybe it's not so good as to
the final solution.

Regards.

Sian




More information about the kde-core-devel mailing list