Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 9c9f2d06 authored by Keisuke Kuroyanagi's avatar Keisuke Kuroyanagi
Browse files

Fix TrieMap: Cannot create next level map of small value entry. 

Bug: 14425059
Change-Id: I864f81ea7dd1d847ef2b016366ce046b1c43d5e5
parent ffaa9c2a

native/jni/src/suggest/policyimpl/dictionary/utils/trie_map.cpp

+1 −1
Original line number Diff line number Diff line
@@ -85,7 +85,7 @@ int TrieMap::getNextLevelBitmapEntryIndex(const int key, const int bitmapEntryIn 
    if (!writeEntry(EMPTY_BITMAP_ENTRY, valueEntryIndex + 1)) {

        return INVALID_INDEX;

    }

    if (!writeField1(valueEntryIndex | TERMINAL_LINK_FLAG, valueEntryIndex)) {

    if (!writeField1(valueEntryIndex | TERMINAL_LINK_FLAG, terminalEntryIndex)) {

        return INVALID_INDEX;

    }

    return valueEntryIndex + 1;

native/jni/tests/suggest/policyimpl/dictionary/utils/trie_map_test.cpp

+1 −0
Original line number Diff line number Diff line
@@ -40,6 +40,7 @@ TEST(TrieMapTest, TestSetAndGet) { 
    trieMap.putRoot(11, 1000);

    EXPECT_EQ(1000ull, trieMap.getRoot(11).mValue);

    const int next = trieMap.getNextLevelBitmapEntryIndex(10);

    EXPECT_EQ(1000ull, trieMap.getRoot(10).mValue);

    trieMap.put(9, 9, next);

    EXPECT_EQ(9ull, trieMap.get(9, next).mValue);

    EXPECT_FALSE(trieMap.get(11, next).mIsValid);