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 [email protected] 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 [email protected] 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.