Skip to content

Commit

Permalink
Fix bad test in WASM version of HNTrieContainer
Browse files Browse the repository at this point in the history
The erroneous test does not seem to interfere
with the proper functioning of the trie, due
to the fact that nodes are never split without
a OR node or boundary node being present.

The issue was found when undertaking a rewrite
of the algorithm to avoid having to create
boundary nodes.
  • Loading branch information
gorhill committed Aug 9, 2021
1 parent 385acd7 commit b54bf55
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 1 deletion.
Binary file modified src/js/wasm/hntrie.wasm
Binary file not shown.
2 changes: 1 addition & 1 deletion src/js/wasm/hntrie.wat
Original file line number Diff line number Diff line change
Expand Up @@ -197,7 +197,7 @@
br_if $noSegment
;; if ( this.buf32[icell+2] === 0 ) {
get_local $icell
i32.load
i32.load offset=8
i32.eqz
if
;; if ( ineedle === 0 || this.buf[ineedle-1] === 0x2E ) {
Expand Down

0 comments on commit b54bf55

Please sign in to comment.