Odp: Power Manager cleanup

cosiekvfj cosiekvfj at o2.pl
Sat Mar 30 14:40:59 CET 2019


Hi, Alex.  thank you for your sugestions.   Here is my cloned repo:   github.com github.com   For now only cleanup and style changes. If you are ok with them then I will think about next steps.   Cheers,  Kacper  Dnia 29 marca 2019 23:22 Alex <acs82 at gmx.de> napisał(a):  Hi,   thanks for your effort.   It would be good to have many small
      commits, each doing a single cleanup step, instead of one giant
      commit.   Specially exchanging methods (
      xfconf_channel_new / xfconf_channel_get ) IMO should happen in a
      dedicated commit.  Reading the doc, I would agree that the channel
      does not need to be released, since it is not mentioned, like in
      xfconf_channel_new.  "The reference
        count of the returned channel is owned by libxfconf." ... so I
        guess  libxfconf will free the object when
          the refcount reaches zero.   Since we dont have gitlab yet, it would
      be nice if you could upload your power-manager branch on your
      personal e.g. gitlab/github/others account, so that we can put
      comments on specific lines.   Cheers,  Alex   ______________________________  Xfce4-dev mailing list   Xfce4-dev at xfce.org  mail.xfce.org mail.xfce.org
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.xfce.org/pipermail/xfce4-dev/attachments/20190330/6f53f2e2/attachment.html>


More information about the Xfce4-dev mailing list