Kill SdbHt or Ht

[radare]

i would go for ht. this means that r2 cant use this api

[condret]

is this the dict api?

[radare]

No

…

On 9 May 2018, at 23:55, condret ***@***.***> wrote: is this the dict api? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

[ret2libc]

SdbHt defined in sdb/src/sdbht.h seems to be just an extension on top of sdb/src/ht.h.

Anyway, I would keep only ht.h as well. I've used it successfully in xrefs, we just need to reintroduce growing.

[radare]

The sdb prefix is to keep the namespace prefix in place and avoid conflicts with other hashtable implementations. If its a public api, aka used outside sdb it must keep that

…

On 31 Jul 2018, at 16:33, Riccardo Schirone ***@***.***> wrote: SdbHt defined in sdb/src/sdbht.h seems to be just an extension on top of sdb/src/ht.h. Anyway, I would keep only ht.h as well. I've used it successfully in xrefs, we just need to reintroduce growing. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.