[Portland-bugs] [Bug 98614] New: Common up xdg-screensaver code that deals with Freedesktop desktop idleness inhibition API

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=98614

            Bug ID: 98614
           Summary: Common up xdg-screensaver code that deals with
                    Freedesktop desktop idleness inhibition API
           Product: Portland
           Version: unspecified
          Hardware: Other
                OS: All
            Status: NEW
          Severity: enhancement
          Priority: medium
         Component: xdg-utils
          Assignee: portland-bugs at lists.freedesktop.org
          Reporter: rrt at sc3d.org

At present xdg-screensaver contains several almost-identical functions to deal
with the various desktops that implement the Freedesktop desktop idleness
inhibition DBus API, or something like it.

One example is the code for MATE, which bears the following comment:

# DBUS interface for mate-screensaver
# This is same as gnome's for now but may change in the future as MATE
# does not follow gnome's developement necessarily.

I think this is an unfortunate attitude to take with respect to Freedesktop
standards in Freedesktop project code. We should be optimistic about the
adoption of standards, and encourage their use.

Hence, I suggest that the following functions have their common code factored
out:

screensaver_freedesktop (ironically only used for KDE)
screensaver_mate_screensaver
screensaver_gnome_screensaver (also used for GNOME 3's gnome-shell)
screensaver_cinnamon_screensaver

Even some DBus end-points not covered by the Freedesktop spec, such as
"SimulateUserActivity" are common between most or all of the above, with the
exception of a different prefix.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/portland-bugs/attachments/20161106/c33bfe19/attachment.html>