Repo // Tidy-up all usage scenarios of keyHandler.clear().

This commit is contained in:
ShikiSuen 2022-08-06 16:15:37 +08:00
parent 02e312d3f4
commit c406d70584
6 changed files with 4 additions and 23 deletions

View File

@ -51,7 +51,6 @@ extension KeyHandler {
// //
// 使 BackSpace // 使 BackSpace
// compositor.isEmpty // compositor.isEmpty
clear()
stateCallback(InputState.EmptyIgnoringPreviousState()) stateCallback(InputState.EmptyIgnoringPreviousState())
stateCallback(InputState.Empty()) stateCallback(InputState.Empty())
} else { } else {
@ -67,7 +66,6 @@ extension KeyHandler {
if input.isEnter { if input.isEnter {
if state is InputState.AssociatedPhrases, !mgrPrefs.alsoConfirmAssociatedCandidatesByEnter { if state is InputState.AssociatedPhrases, !mgrPrefs.alsoConfirmAssociatedCandidatesByEnter {
clear()
stateCallback(InputState.EmptyIgnoringPreviousState()) stateCallback(InputState.EmptyIgnoringPreviousState())
stateCallback(InputState.Empty()) stateCallback(InputState.Empty())
return true return true
@ -335,7 +333,6 @@ extension KeyHandler {
didSelectCandidateAt: candidateIndex, didSelectCandidateAt: candidateIndex,
ctlCandidate: ctlCandidateCurrent ctlCandidate: ctlCandidateCurrent
) )
clear()
stateCallback(InputState.EmptyIgnoringPreviousState()) stateCallback(InputState.EmptyIgnoringPreviousState())
stateCallback(InputState.Empty()) stateCallback(InputState.Empty())
return handle( return handle(

View File

@ -107,7 +107,6 @@ extension KeyHandler {
isTypingVertical: input.isTypingVertical isTypingVertical: input.isTypingVertical
) )
if choosingCandidates.candidates.count == 1 { if choosingCandidates.candidates.count == 1 {
clear()
let reading: String = choosingCandidates.candidates.first?.0 ?? "" let reading: String = choosingCandidates.candidates.first?.0 ?? ""
let text: String = choosingCandidates.candidates.first?.1 ?? "" let text: String = choosingCandidates.candidates.first?.1 ?? ""
stateCallback(InputState.Committing(textToCommit: text)) stateCallback(InputState.Committing(textToCommit: text))

View File

@ -69,7 +69,6 @@ extension KeyHandler {
// BackSpace // BackSpace
} else if input.isCapsLockOn || input.isASCIIModeInput { } else if input.isCapsLockOn || input.isASCIIModeInput {
// //
clear()
stateCallback(InputState.Empty()) stateCallback(InputState.Empty())
// Shift // Shift
@ -99,7 +98,6 @@ extension KeyHandler {
if !(state is InputState.ChoosingCandidate || state is InputState.AssociatedPhrases if !(state is InputState.ChoosingCandidate || state is InputState.AssociatedPhrases
|| state is InputState.SymbolTable) || state is InputState.SymbolTable)
{ {
clear()
stateCallback(InputState.Empty()) stateCallback(InputState.Empty())
stateCallback(InputState.Committing(textToCommit: inputText.lowercased())) stateCallback(InputState.Committing(textToCommit: inputText.lowercased()))
stateCallback(InputState.Empty()) stateCallback(InputState.Empty())
@ -162,7 +160,6 @@ extension KeyHandler {
if !composingBuffer.isEmpty { if !composingBuffer.isEmpty {
stateCallback(InputState.Committing(textToCommit: composingBuffer)) stateCallback(InputState.Committing(textToCommit: composingBuffer))
} }
clear()
stateCallback(InputState.Committing(textToCommit: " ")) stateCallback(InputState.Committing(textToCommit: " "))
stateCallback(InputState.Empty()) stateCallback(InputState.Empty())
} else if currentLM.hasUnigramsFor(key: " ") { } else if currentLM.hasUnigramsFor(key: " ") {

View File

@ -276,7 +276,7 @@ extension KeyHandler {
isTypingVertical: isTypingVertical isTypingVertical: isTypingVertical
) )
if candidateState.candidates.count == 1 { if candidateState.candidates.count == 1 {
clear() clear() // candidateState
if let candidateToCommit: (String, String) = candidateState.candidates.first, !candidateToCommit.1.isEmpty { if let candidateToCommit: (String, String) = candidateState.candidates.first, !candidateToCommit.1.isEmpty {
stateCallback(InputState.Committing(textToCommit: candidateToCommit.1)) stateCallback(InputState.Committing(textToCommit: candidateToCommit.1))
stateCallback(InputState.Empty()) stateCallback(InputState.Empty())
@ -302,7 +302,6 @@ extension KeyHandler {
) -> Bool { ) -> Bool {
guard let currentState = state as? InputState.Inputting else { return false } guard let currentState = state as? InputState.Inputting else { return false }
clear()
stateCallback(InputState.Committing(textToCommit: currentState.composingBuffer)) stateCallback(InputState.Committing(textToCommit: currentState.composingBuffer))
stateCallback(InputState.Empty()) stateCallback(InputState.Empty())
return true return true
@ -331,8 +330,6 @@ extension KeyHandler {
composingBuffer = composingBuffer.replacingOccurrences(of: "-", with: " ") composingBuffer = composingBuffer.replacingOccurrences(of: "-", with: " ")
} }
clear()
stateCallback(InputState.Committing(textToCommit: composingBuffer)) stateCallback(InputState.Committing(textToCommit: composingBuffer))
stateCallback(InputState.Empty()) stateCallback(InputState.Empty())
return true return true
@ -369,8 +366,6 @@ extension KeyHandler {
composed += key.contains("_") ? value : "<ruby>\(value)<rp>(</rp><rt>\(key)</rt><rp>)</rp></ruby>" composed += key.contains("_") ? value : "<ruby>\(value)<rp>(</rp><rt>\(key)</rt><rp>)</rp></ruby>"
} }
clear()
stateCallback(InputState.Committing(textToCommit: composed)) stateCallback(InputState.Committing(textToCommit: composed))
stateCallback(InputState.Empty()) stateCallback(InputState.Empty())
return true return true
@ -394,7 +389,6 @@ extension KeyHandler {
guard state is InputState.Inputting else { return false } guard state is InputState.Inputting else { return false }
if input.isShiftHold { if input.isShiftHold {
clear()
stateCallback(InputState.EmptyIgnoringPreviousState()) stateCallback(InputState.EmptyIgnoringPreviousState())
stateCallback(InputState.Empty()) stateCallback(InputState.Empty())
return true return true
@ -443,7 +437,6 @@ extension KeyHandler {
guard state is InputState.Inputting else { return false } guard state is InputState.Inputting else { return false }
if input.isShiftHold { if input.isShiftHold {
clear()
stateCallback(InputState.EmptyIgnoringPreviousState()) stateCallback(InputState.EmptyIgnoringPreviousState())
stateCallback(InputState.Empty()) stateCallback(InputState.Empty())
return true return true
@ -580,7 +573,6 @@ extension KeyHandler {
if mgrPrefs.escToCleanInputBuffer { if mgrPrefs.escToCleanInputBuffer {
/// ///
/// macOS Windows 使 /// macOS Windows 使
clear()
stateCallback(InputState.EmptyIgnoringPreviousState()) stateCallback(InputState.EmptyIgnoringPreviousState())
stateCallback(InputState.Empty()) stateCallback(InputState.Empty())
} else { } else {

View File

@ -62,7 +62,6 @@ class ctlInputMethod: IMKInputController {
/// 調 /// 調
handle(state: InputState.Committing(textToCommit: state.composingBufferConverted)) handle(state: InputState.Committing(textToCommit: state.composingBufferConverted))
} }
keyHandler.clear()
handle(state: InputState.Empty()) handle(state: InputState.Empty())
} }
@ -96,7 +95,7 @@ class ctlInputMethod: IMKInputController {
// //
if keyHandler.delegate == nil { keyHandler.delegate = self } if keyHandler.delegate == nil { keyHandler.delegate = self }
setValue(IME.currentInputMode.rawValue, forTag: 114_514, client: client()) setValue(IME.currentInputMode.rawValue, forTag: 114_514, client: client())
keyHandler.clear() keyHandler.clear() // handle State.Empty()
keyHandler.ensureParser() keyHandler.ensureParser()
if isASCIIMode { if isASCIIMode {
@ -123,7 +122,6 @@ class ctlInputMethod: IMKInputController {
/// - Parameter sender: 使 /// - Parameter sender: 使
override func deactivateServer(_ sender: Any!) { override func deactivateServer(_ sender: Any!) {
_ = sender // _ = sender //
keyHandler.clear()
handle(state: InputState.Empty()) handle(state: InputState.Empty())
handle(state: InputState.Deactivated()) handle(state: InputState.Deactivated())
} }
@ -149,7 +147,7 @@ class ctlInputMethod: IMKInputController {
if keyHandler.inputMode != newInputMode { if keyHandler.inputMode != newInputMode {
UserDefaults.standard.synchronize() UserDefaults.standard.synchronize()
keyHandler.clear() keyHandler.clear() // handle State.Empty()
keyHandler.inputMode = newInputMode keyHandler.inputMode = newInputMode
/// ///
/// macOS /// macOS

View File

@ -116,9 +116,7 @@ extension ctlInputMethod: ctlCandidateDelegate {
let inputting = keyHandler.buildInputtingState let inputting = keyHandler.buildInputtingState
if mgrPrefs.useSCPCTypingMode { if mgrPrefs.useSCPCTypingMode {
keyHandler.clear() handle(state: InputState.Committing(textToCommit: inputting.composingBuffer))
let composingBuffer = inputting.composingBuffer
handle(state: InputState.Committing(textToCommit: composingBuffer))
// selectedValue.1 // selectedValue.1
if mgrPrefs.associatedPhrasesEnabled, if mgrPrefs.associatedPhrasesEnabled,
let associatePhrases = keyHandler.buildAssociatePhraseState( let associatePhrases = keyHandler.buildAssociatePhraseState(