[PATCH 0/4] xfce-panel: clock: binary cleanups

[Simon Steinbeiss]

Hi Felipe,

I've given you fork permissions on gitlab.xfce.org so you can create a PR
with your changes, which will make it easier to review.

Cheers
Simon

On Thu, Aug 4, 2022 at 12:30 AM Felipe Contreras <felipe.contreras at gmail.com>
wrote:

> Hi,
>
> While implementing a true binary time clock [1] I came up with many
> areas of opportunity in the binary clock code.
>
> I sent these patches more than three years ago (issue 235), and a month
> ago the issue was closed with zero consideration.
>
> The cleanups are obvious so I'm sending the first four patches to be
> properly reviewed.
>
> I have to say that spending many hours to write perfectly good patches
> that are gather dust for years only to be immediately dismissed with zero
> consideration is not very encouraging.
>
> In my opinion this is not a productive way of doing development, which
> is why email is 10000 times a more superior development tool than
> any bugzilla-like issue tracker.
>
> Hopefully this time it will take less than 3 years to get a response.
>
> I've also opened an issue in gitlab:
>
> https://gitlab.xfce.org/xfce/xfce4-panel/-/issues/622
>
> Cheers.
>
> [1] https://en.wikipedia.org/wiki/Binary_clock#Binary_time
>
> Felipe Contreras (4):
>   clock: binary: don't get style of the parent
>   clock: binary: fix state flag check
>   clock: binary: get the color of the current state
>   clock: binary: remove redundant code
>
>  plugins/clock/clock-binary.c | 50 ++++++++++--------------------------
>  1 file changed, 13 insertions(+), 37 deletions(-)
>
> --
> 2.37.1.313.g01a229ef5e
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.xfce.org/pipermail/xfce4-dev/attachments/20220804/732654a5/attachment.html>