Re: [PATCH spice-gtk 2/3] Add a desktop-integration helper class

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi

----- Mensaje original -----
> > Looks reasonable to me, however, I don't see why it should be in
> > spice-glib, since it's very much desktop related.  Wouldn't it make
>  > more sense in spice-gtk?
> 
> I actually put it in spice-gtk at first, but that does not work very
> well
> because usb-device-manager needs access to the automount disable
> functionality.

Ok
 
>  > Since it's not directly dependent and related
> > to a particular spice session, it could perhaps even be a separate
> > library (although for now, a few extra functions and dependency in
> > spice-gtk is surely fine).
> 
> There is various state inside, which needs to be coupled to some
> object,
> using the session for this seems best.

Except setting/getting its own SpiceDesktopIntegration *desktop_integration; what does it need from SpiceSession? If I understand correctly, the changes made by desktop-integration are session-wide, so why having a 1-1 relation with each spice session?
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel


[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]