<div dir="ltr"><div>Hi Felipe,</div><div><br></div><div>I've given you fork permissions on <a href="http://gitlab.xfce.org">gitlab.xfce.org</a> so you can create a PR with your changes, which will make it easier to review.</div><div><br></div><div>Cheers</div><div>Simon<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Aug 4, 2022 at 12:30 AM Felipe Contreras <<a href="mailto:felipe.contreras@gmail.com">felipe.contreras@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<br>
While implementing a true binary time clock [1] I came up with many<br>
areas of opportunity in the binary clock code.<br>
<br>
I sent these patches more than three years ago (issue 235), and a month<br>
ago the issue was closed with zero consideration.<br>
<br>
The cleanups are obvious so I'm sending the first four patches to be<br>
properly reviewed.<br>
<br>
I have to say that spending many hours to write perfectly good patches<br>
that are gather dust for years only to be immediately dismissed with zero<br>
consideration is not very encouraging.<br>
<br>
In my opinion this is not a productive way of doing development, which<br>
is why email is 10000 times a more superior development tool than<br>
any bugzilla-like issue tracker.<br>
<br>
Hopefully this time it will take less than 3 years to get a response.<br>
<br>
I've also opened an issue in gitlab:<br>
<br>
<a href="https://gitlab.xfce.org/xfce/xfce4-panel/-/issues/622" rel="noreferrer" target="_blank">https://gitlab.xfce.org/xfce/xfce4-panel/-/issues/622</a><br>
<br>
Cheers.<br>
<br>
[1] <a href="https://en.wikipedia.org/wiki/Binary_clock#Binary_time" rel="noreferrer" target="_blank">https://en.wikipedia.org/wiki/Binary_clock#Binary_time</a><br>
<br>
Felipe Contreras (4):<br>
  clock: binary: don't get style of the parent<br>
  clock: binary: fix state flag check<br>
  clock: binary: get the color of the current state<br>
  clock: binary: remove redundant code<br>
<br>
 plugins/clock/clock-binary.c | 50 ++++++++++--------------------------<br>
 1 file changed, 13 insertions(+), 37 deletions(-)<br>
<br>
-- <br>
2.37.1.313.g01a229ef5e<br>
<br>
</blockquote></div>