It doesn't make sense to get a table passed in then ignore that one
completely so it doesn't look like the intentional behavior, and
regardless of this, the edited code is still more logical.
Change-Id: I98bc841e982d328e268309c06fefd3d3ca9f6cd7
def manualWordsTable(path, conv_table, reconv_table):
fp = open(path, 'r', encoding='U8')
def manualWordsTable(path, conv_table, reconv_table):
fp = open(path, 'r', encoding='U8')
+ reconv_table = reconv_table.copy()
+ out_table = {}
wordlist = [line.split('#')[0].strip() for line in fp]
wordlist = list(set(wordlist))
wordlist.sort(key=lambda w: (len(w), w), reverse=True)
wordlist = [line.split('#')[0].strip() for line in fp]
wordlist = list(set(wordlist))
wordlist.sort(key=lambda w: (len(w), w), reverse=True)
new_word = translate(word, conv_table)
rcv_word = translate(word, reconv_table)
if word != rcv_word:
new_word = translate(word, conv_table)
rcv_word = translate(word, reconv_table)
if word != rcv_word:
- reconv_table[word] = word
- reconv_table[new_word] = word
- return reconv_table
+ reconv_table[word] = out_table[word] = word
+ reconv_table[new_word] = out_table[new_word] = word
+ return out_table
def defaultWordsTable(src_wordlist, src_tomany, char_conv_table,
def defaultWordsTable(src_wordlist, src_tomany, char_conv_table,