Re: [BUGS] BUG #14764: #if HASH_DEBUG source in dynahash.c is fault. - Mailing list pgsql-bugs

From Tom Lane
Subject Re: [BUGS] BUG #14764: #if HASH_DEBUG source in dynahash.c is fault.
Date
Msg-id 9352.1501689875@sss.pgh.pa.us
Whole thread Raw
In response to [BUGS] BUG #14764: #if HASH_DEBUG source in dynahash.c is fault.  (twodragon72@gmail.com)
List pgsql-bugs
twodragon72@gmail.com writes:
> dynahash.c

> #if HASH_DEBUG
>     fprintf(stderr,
> "init_htab:\n%s%p\n%s%ld\n%s%ld\n%s%d\n%s%ld\n%s%u\n%s%x\n%s%x\n%s%ld\n%s%ld\n",
>             "TABLE POINTER   ", hashp,
>             "DIRECTORY SIZE  ", hctl->dsize,
>             "SEGMENT SIZE    ", hctl->ssize,
>             "SEGMENT SHIFT   ", hctl->sshift,
>             "FILL FACTOR     ", hctl->ffactor,
>             "MAX BUCKET      ", hctl->max_bucket,
>             "HIGH MASK       ", hctl->high_mask,
>             "LOW  MASK       ", hctl->low_mask,
>             "NSEGS           ", hctl->nsegs,
>             "NENTRIES        ", hash_get_num_entries(hctl));
> #endif


> ", hash_get_num_entries(hctl));
> ->
> ", hash_get_num_entries(hashp));

Hm, yeah, that's broken, but I'm inclined to remove that field from the
printout rather than fix the computation.  Since we just initialized the
hashtable, it can't possibly have any entries here.  Also
hash_get_num_entries makes assumptions that we could do without in
debug code.
        regards, tom lane


--
Sent via pgsql-bugs mailing list (pgsql-bugs@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-bugs

pgsql-bugs by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: [BUGS] BUG #14758: Segfault with logical replication on afunction index
Next
From: Peter Eisentraut
Date:
Subject: Re: [HACKERS] Re: [BUGS] BUG #14758: Segfault with logicalreplication on a function index