[SSSD] [PATCH] DHASH: Don't use backward jumps
Lukas Slebodnik
lslebodn at redhat.com
Fri Apr 5 11:52:23 UTC 2013
On (02/04/13 17:44), Ondrej Kos wrote:
>https://fedorahosted.org/sssd/ticket/1855
>
>I couldn't let it this way when I saw the code :)
>Patch is attached
>
>Ondra
>--
>Ondrej Kos
>Associate Software Engineer
>Identity Management
>Red Hat Czech
>
>phone: +420-532-294-558
>cell: +420-736-417-909
>ext: 82-62558
>loc: 1013 Brno 1 office
>irc: okos @ #sssd #brno
>diff --git a/dhash/dhash.h b/dhash/dhash.h
>index baa0d6aa88b767b63018127c4ac454922fdd77d8..7f44c57f8091c70766c9d7fccb166e81b377721e 100644
>--- a/dhash/dhash.h
>+++ b/dhash/dhash.h
>@@ -133,6 +133,14 @@ typedef enum
> HASH_ENTRY_DESTROY
> } hash_destroy_enum;
>
>+typedef enum {
>+ HI_STATE_0,
>+ HI_STATE_1,
>+ HI_STATE_2,
>+ HI_STATE_3A,
>+ HI_STATE_3B
>+} hash_iter_state;
>+
This enum is only used in static function hash_iter_next().
It is unnecessarily to be available in public API.
I think that this typedef could be safety moved to implementation file dhash.c
> typedef struct hash_key_t {
> hash_key_enum type;
> union {
>--
>1.8.1.4
>
>_______________________________________________
>sssd-devel mailing list
>sssd-devel at lists.fedorahosted.org
>https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
More information about the sssd-devel
mailing list