kicad-developers team mailing list archive
Mailing list archive
Re: [PATCH] Patchlets
On 05/15/2012 06:51 AM, Edwin van den Oetelaar wrote:
> On Tue, May 15, 2012 at 1:31 PM, Lorenzo Marcantonio
> <l.marcantonio@xxxxxxxxxxxx> wrote:
>> At last I remade the work...
>> 016 The layer number from int becomes LAYER_NUM (original name!)
>> 017 Replaced all the ugly (1 << layer) with GetLayerMask
>> 018 Layer bitmaps (like visibles, enabled and so on) become
>> LAYER_MSK (which aligns to LAYER_NUM!)
> 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?
Since all objects are in a single display list rather than organized by layer, being able
to select which objects are displayed based on layer is a time critical operation.
An integer bitmask is very efficient, and probably the fastest way to make this subset
extraction, since it can be done with a single bit wise AND test.