Skip to content

Commit

Permalink
Move double metaphone implementation to C rather than C++ (#454)
Browse files Browse the repository at this point in the history
This eliminates the need to have different binaries for different Linux
platforms (at least for now), and also simplifies the compilation
process.
  • Loading branch information
mnunberg authored and MeirShpilraien committed Aug 23, 2018
1 parent 4a8dde6 commit 6967f07
Show file tree
Hide file tree
Showing 9 changed files with 389 additions and 320 deletions.
2 changes: 1 addition & 1 deletion src/debug_commads.c
Expand Up @@ -140,7 +140,7 @@ static void DumpPhoneticHash(RedisModuleCtx *ctx, RedisModuleString *term) {
char *primary = NULL;
char *secondary = NULL;

PhoneticManager_ExpandPhonerics(NULL, term_c, len, &primary, &secondary);
PhoneticManager_ExpandPhonetics(NULL, term_c, len, &primary, &secondary);

RedisModule_ReplyWithArray(ctx, 2);
RedisModule_ReplyWithStringBuffer(ctx, primary, strlen(primary));
Expand Down
2 changes: 1 addition & 1 deletion src/dep/phonetics/CMakeLists.txt
@@ -1 +1 @@
ADD_LIBRARY(metaphone OBJECT double_metaphone.cpp)
ADD_LIBRARY(metaphone OBJECT double_metaphone.c)

0 comments on commit 6967f07

Please sign in to comment.