USE_BINARY_SEARCH not really used

[GregSlazinski]

The new binary name search functionality, says:

#define USE_BINARY_SEARCH to enable it).  This gives a very large
 * speedup for libraries that exports lots of functions.

However USE_BINARY_SEARCH is not really tested anywhere and from what I see binary search is always enabled.

[fancycode]

The sorted table is created and sorted only once, the first time a lookup by name is requested and it is cached in the module for further lookups:

MemoryModule/MemoryModule.c

Line 814 in 5f83e41

if (!module->nameExportsTable) {

Additional lookups will only perform the binary search but not regenerate the table. That is unless I'm missing something - in which case please provide additional information or even better a PR.

However USE_BINARY_SEARCH is not really tested anywhere and from what I see binary search is always enabled.

This comment is probably a leftover of some old code where the binary search was not always used.

[fancycode]

Well, feel free to modify the code locally as you need for debugging ;-)