[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [tyndur-devel] [PATCH 2/2] libc: key_mask fuer Baeume



On Sat, Apr 18 13:05, Kevin Wolf wrote:
> + libc: Nicht alle 64 Bit des Schluessels muessen als Schluessel benutzt
>   werden, sondern koennen z.B. Flags enthalten. Das neue Feld key_mask
>   filtert dann raus, was nicht zum Schluessel gehoert
> ---
>  src/include/collections.h  |    1 +
>  src/lib/collections/tree.c |    6 +++++-
>  2 files changed, 6 insertions(+), 1 deletions(-)
> 
> diff --git a/src/include/collections.h b/src/include/collections.h
> index 2593e3a..0ecb321 100644
> --- a/src/include/collections.h
> +++ b/src/include/collections.h
> @@ -81,6 +81,7 @@ typedef struct {
>      struct tree_item*   root;
>      size_t              tree_item_offset;
>      size_t              sort_key_offset;

Wenn das Feld schon zum selber befüllen da ist, sollte da vielleicht
noch ein Kommentar hin, der auch sagt, dass man das nicht ändern will
wenn was drin ist...

> +    uint64_t            key_mask;
>  } tree_t;
>  
>  /**

Sonst kann das rein.

-- 
Antoine Kaufmann
<toni@xxxxxxxxxxxxxxxx>

Attachment: pgpgdSQcm47g_.pgp
Description: PGP signature