← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Patchlets


On Tue, May 15, 2012 at 01:51:17PM +0200, Edwin van den Oetelaar wrote:
> Why is this stored in a bitmask anyway? Now everything depends on
> #define INT_BITS
> would not a <vector> or simple array[] make more sense?

A bit_vector, also. The issue is that there are lots of thing cabled on
the bitmask logic. At least if you look at my code these 'enum' *should*
be reasonably substituted with other 'things' (passed by value) without
too much work.

Part of this is to enable creation of more technical layers (a looong
standing issue).

In other words there enum are standoff for a future 'more suitable' data
structure. The patch is already big as it is, do not ask for too much in
one step XD

Lorenzo Marcantonio
Logos Srl

Follow ups