Re: [dev] [PATCH] Reorder-and-extend-glyph-attributes

From: FRIGN <dev_AT_frign.de>
Date: Thu, 31 Jul 2014 12:22:23 +0200

On Thu, 31 Jul 2014 12:10:43 +0200
"Roberto E. Vargas Caballero" <k0ga_AT_shike2.com> wrote:

> Yeah, it is true. I will apply it only if it fits well in the new
> main-loop. I have modified the original patch and now it may be applied
> to HEAD. The patch itself is not bad, and the complexity it pays for two
> blinks is very small. I have not tested it, but I attach it only to show
> what modifications need to be done.

It's rather short, but as I said, even too complex for two blinking-speeds.
The added complexitiy is sufficient for any desired speed, which is way more
than we need.
Not dishonoring the author of course, who did a fine job. However, I still
keep my opinion.

> Refactoring the main loop is the first priority now, if you have a patch
> serie about it, be sure no any other patch is going to be applied that
> could create a conflict.

Don't worry about that, Roberto. Just keep on doing the great work and I'll
let you know when I have a patchset ready and need the mainline to be frozen.

Cheers

FRIGN

-- 
FRIGN <dev_AT_frign.de>
Received on Thu Jul 31 2014 - 12:22:23 CEST

This archive was generated by hypermail 2.3.0 : Thu Jul 31 2014 - 12:24:06 CEST