Re: [dev] [PATCH] Add fast blink support

From: FRIGN <>
Date: Wed, 25 Jun 2014 19:19:19 +0200

On Wed, 25 Jun 2014 19:13:33 +0200
"Roberto E. Vargas Caballero" <> wrote:

> I don't have a clear idea about this patch. I would like listen what think
> another suckless developers before of taking a decision. If we emulate the
> fast blinking with the slow blinking it will not generate any functional
> problem, and for sure we will be simpler.

I know simplification can also be done by splitting up a big function
into smaller ones, but I don't see the simplification here.
I'd propose to first refactor run() in smaller steps and in a way it
doesn't have to respect two blinking speeds.

I of course am open to your ideas!



Received on Wed Jun 25 2014 - 19:19:19 CEST

This archive was generated by hypermail 2.3.0 : Wed Jun 25 2014 - 19:24:10 CEST