On Wed, 20 Jun 2018 21:04:48 +0300 Yordan Karadzhov <y.karadz@xxxxxxxxx> wrote: > My only reason for having the hash functions in the header is to make sure > that the user will be aware that those are cheap and fast functions, but > using > those functions makes sense only in the case when we have relatively small > number of items in the hash table. Nevertheless, If you prefer the > implementation > to be in the .c file this is OK. Either that, or in a local header file. I don't think we need to export this as part of libtracecmd.a I'll make the changes. Thanks! -- Steve