aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* am b5880034: Refactoring.Jean Chalard2011-12-151-17/+15
|\ | | | | | | | | * commit 'b5880034268111a094c9e684cec9cca784386140': Refactoring.
| * Refactoring.Jean Chalard2011-12-141-17/+15
| | | | | | | | | | | | Small simplification of the code. Change-Id: I1f67ae529bbb2c9316bf5e083d367c57040ea22c
* | am e35e13fc: Merge "Record onStartInputView"Tadashi G. Takaoka2011-12-152-4/+9
|\| | | | | | | | | * commit 'e35e13fc7dc046be7f605cbfc706678f57e40491': Record onStartInputView
| * Merge "Record onStartInputView"Tadashi G. Takaoka2011-12-132-4/+9
| |\
| | * Record onStartInputViewTadashi G. Takaoka2011-12-142-4/+9
| | | | | | | | | | | | | | | Bug: 5722703 Change-Id: I56bcf6ccde82b4176093de4f197a8736b0cc8c10
* | | am 689b9011: Merge "Utilize KeyboardSet XML definitions"Tadashi G. Takaoka2011-12-1549-382/+1087
|\| | | | | | | | | | | | | | * commit '689b90115633146bcdea81bf89cdfe2f9a88c0a7': Utilize KeyboardSet XML definitions
| * | Merge "Utilize KeyboardSet XML definitions"Tadashi G. Takaoka2011-12-1349-382/+1087
| |\ \
| | * | Utilize KeyboardSet XML definitionsTadashi G. Takaoka2011-12-1449-382/+1087
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change introduces KeyboardSet and SubKeyboard XML definitions to represent a set of keyboard layouts. Bug: 5002108 Bug: 5679585 Change-Id: Ib6c8d5936187381bb6725c9fe574e93871c01a86
* | | | am dbdb385b: Merge "Fix a long-standing race condition."Jean Chalard2011-12-151-15/+15
|\| | | | | | | | | | | | | | | | | | | * commit 'dbdb385bb06ccd235b3b6445ceabf07ca893454b': Fix a long-standing race condition.
| * | | Merge "Fix a long-standing race condition."Jean Chalard2011-12-131-15/+15
| |\ \ \
| | * | | Fix a long-standing race condition.Jean Chalard2011-12-141-15/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Upon autocorrection, there is visual feedback in the text view that quickly flashes the background of the text that just changed. This fixes a race condition that happens upon autocorrection when typing fast, and that results in flashing one character too far left (typically, the flashing area includes the whitespace before the corrected word and not the last character of the corrected word). This happens because the call to commitCorrection may happen before or after the IPC sent by sendKeyChar, but the arguments are fit only for the case where it arrives first. Change-Id: I9b5442a665aad5a9bc66cd49228075b9056b37fa
* | | | | am 4fc97c2c: Add a note of documentation to the sample word listJean Chalard2011-12-151-2/+3
|\| | | | | | | | | | | | | | | | | | | | | | | | * commit '4fc97c2c01646d877505295713abdf16d775d3d4': Add a note of documentation to the sample word list
| * | | | Add a note of documentation to the sample word listJean Chalard2011-12-141-2/+3
| | |_|/ | |/| | | | | | | | | | Change-Id: I95f09da03457933a14b549e04575d566de97dd49
* | | | am 8e3faff2: Merge "Decouple member logic from control flow"Jean Chalard2011-12-131-4/+8
|\| | | | | | | | | | | | | | | | | | | * commit '8e3faff244a03aa49dfff03f2a6d982590ff605c': Decouple member logic from control flow
| * | | Merge "Decouple member logic from control flow"Jean Chalard2011-12-131-4/+8
| |\ \ \
| | * | | Decouple member logic from control flowJean Chalard2011-12-131-4/+8
| | |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This place is very confusing because a member variable with a consistent meaning is hijacked for the duration of a function to mean something else. This is in the way of easy-to-understand refactoring. Change-Id: If79bc771950d6bfc0ad5f0e9c51c7ef1dbb45b66
* | | | am f7d6517d: Various mini-cleanupsJean Chalard2011-12-134-10/+11
|\| | | | | | | | | | | | | | | | | | | * commit 'f7d6517d6b1a1dd88e2142e1a15703bb839be01b': Various mini-cleanups
| * | | Various mini-cleanupsJean Chalard2011-12-134-10/+11
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | - Stop the word composer from escaping - take a page from the law of Demeter and only report what is actually needed. - Fix typos in comments. - Add a comment for a fishy processing. - Remove a useless local variable. Change-Id: I5fa78901cbb5483fc9683bfb7094f47244b85df6
* | | am edf4995a: Merge "Remove the input connection sidework from handleCharacter"Jean Chalard2011-12-131-6/+10
|\| | | | | | | | | | | | | | * commit 'edf4995a3bad3b95a2b4184d47ea4ff890d0de73': Remove the input connection sidework from handleCharacter
| * | Merge "Remove the input connection sidework from handleCharacter"Jean Chalard2011-12-131-6/+10
| |\ \
| | * | Remove the input connection sidework from handleCharacterJean Chalard2011-12-131-6/+10
| | | | | | | | | | | | | | | | Change-Id: I3d376dfda3acd8864274390de3ff5752b18456e0
* | | | am 5a603f3f: Merge "Reorganize LatinIME native build directory structure. ↵Ken Wakasa2011-12-137-71/+119
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | Now it got NDK friendly." * commit '5a603f3fa2221a68452939dc1e6bb8f5444f04eb': Reorganize LatinIME native build directory structure. Now it got NDK friendly.
| * | | Merge "Reorganize LatinIME native build directory structure. Now it got NDK ↵Ken Wakasa2011-12-137-71/+119
| |\ \ \ | | | | | | | | | | | | | | | friendly."
| | * | | Reorganize LatinIME native build directory structure. Now it got NDK friendly.Ken Wakasa2011-12-137-71/+119
| | | |/ | | |/| | | | | | | | | Change-Id: I0f62ce8a6a4d2b6134db698f8b8d0576616e524d
* | | | am 117fc93f: Move mBestWord to the word composer.Jean Chalard2011-12-132-19/+55
|\| | | | | | | | | | | | | | | | | | | * commit '117fc93f373cb86d4120c1261f9d0562c6529fec': Move mBestWord to the word composer.
| * | | Move mBestWord to the word composer.Jean Chalard2011-12-132-19/+55
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | mBestWord has a confusing name - it's actually an auto-correction. It's cleaner if it lives in the word composer because an auto-correction should be tied to a specific user input, and should be reset each time the user input changes to avoid race conditions. Change-Id: I718d29395bc747372067e6440e090c6a181994ae
* | | am 7e6f4daa: Merge "Tighten conditions for space-related cancelling"Jean Chalard2011-12-131-4/+10
|\| | | | | | | | | | | | | | * commit '7e6f4daa196f0fd88873b5b360e3fc11b97e1ef7': Tighten conditions for space-related cancelling
| * | Merge "Tighten conditions for space-related cancelling"Jean Chalard2011-12-131-4/+10
| |\ \
| | * | Tighten conditions for space-related cancellingJean Chalard2011-12-131-4/+10
| | | | | | | | | | | | | | | | | | | | | | | | Instead of ignoring silently a found bug, throw an exception. Change-Id: I0aa96d0cab022b6a5e7cffe8a592dcc443c3c7a8
* | | | am c5257508: Merge "Consolidate underlining code under one function."Jean Chalard2011-12-131-18/+17
|\| | | | | | | | | | | | | | | | | | | * commit 'c525750835cbb1e8c2b2bdf0248babaeb8225cba': Consolidate underlining code under one function.
| * | | Merge "Consolidate underlining code under one function."Jean Chalard2011-12-131-18/+17
| |\ \ \ | | |_|/ | |/| |
| | * | Consolidate underlining code under one function.Jean Chalard2011-12-131-18/+17
| | |/ | | | | | | | | | Change-Id: I4393ff1ec28a6b3118d1344dd6d82d04899dea42
* | | am cfe264bf: Merge "Introduce KeyboardSet to hold related KeyboardIds"Tadashi G. Takaoka2011-12-132-80/+144
|\| | | | | | | | | | | | | | * commit 'cfe264bfee15344282f4a2f42cfc7ee905f0eba8': Introduce KeyboardSet to hold related KeyboardIds
| * | Merge "Introduce KeyboardSet to hold related KeyboardIds"Tadashi G. Takaoka2011-12-132-80/+144
| |\ \
| | * | Introduce KeyboardSet to hold related KeyboardIdsTadashi G. Takaoka2011-12-132-80/+144
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change moves * KeyboardSwitcher.getKeyboardId to KeyboardSet.<init> and KeyboardSet.getKeyboardId. * KeyboardSwitcher.getF2KeyMode to KeyboardSet. This is the first step to get rid of shift state from Keyboard. Bug: 5002108 Bug: 5679585 Change-Id: I2a5a97e5e89277bc606916d45c1afbc8d105d773
* | | | am 3dcba427: Merge "Extract the jni library name"satok2011-12-132-2/+28
|\| | | | | | | | | | | | | | | | | | | * commit '3dcba4279a9e663d040330fe13b6ce3ecd0ca879': Extract the jni library name
| * | | Merge "Extract the jni library name"satok2011-12-132-2/+28
| |\ \ \ | | |/ / | |/| |
| | * | Extract the jni library namesatok2011-12-132-2/+28
| | |/ | | | | | | | | | Change-Id: Icc58ccea1178d9f35f5be0dceebab67bbb965b7f
* | | am 77ecf52a: Merge "Remove KeyboardSwitcher.updateShiftLockState method"Tadashi G. Takaoka2011-12-131-15/+7
|\| | | | | | | | | | | | | | * commit '77ecf52a082143ef65fa2de1ca4425ec32ad8cd0': Remove KeyboardSwitcher.updateShiftLockState method
| * | Merge "Remove KeyboardSwitcher.updateShiftLockState method"Tadashi G. Takaoka2011-12-131-15/+7
| |\ \ | | |/ | |/|
| | * Remove KeyboardSwitcher.updateShiftLockState methodTadashi G. Takaoka2011-12-131-15/+7
| | | | | | | | | | | | Change-Id: I032fc50a62a573ba4258530e695c8c88467783c6
* | | am 0a2494fa: Merge "Remove a useless return"Jean Chalard2011-12-131-3/+0
|\| | | | | | | | | | | | | | * commit '0a2494fa5881152a9ed316409ae650353d8969fb': Remove a useless return
| * | Merge "Remove a useless return"Jean Chalard2011-12-131-3/+0
| |\ \
| | * | Remove a useless returnJean Chalard2011-12-131-3/+0
| | | | | | | | | | | | | | | | Change-Id: Iefe10e94e2d88171387351a5b6cab52e281d0f76
* | | | am b2632553: Merge "Use priority queue for native string buffer"satok2011-12-135-85/+178
|\| | | | | | | | | | | | | | | | | | | * commit 'b2632553bfcae5beb92f7366a71f15de30f44a9c': Use priority queue for native string buffer
| * | | Merge "Use priority queue for native string buffer"satok2011-12-125-85/+178
| |\ \ \ | | |/ / | |/| |
| | * | Use priority queue for native string buffersatok2011-12-135-85/+178
| | |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | +1 2 -6 2 Performance before ==== test finished, terminate logcat ===== (0) 100.34 (0.26%) (1) 37149.26 (95.30%) (2) 8.43 (0.02%) (3) 11.18 (0.03%) (4) 9.92 (0.03%) (5) 1330.60 (3.41%) (6) 250.46 (0.64%) (20) 12.41 (0.03%) Total 38982.50 (sum of others 38872.59) after ==== test finished, terminate logcat ===== (0) 97.65 (0.26%) (1) 35427.43 (95.32%) (2) 10.30 (0.03%) (3) 8.95 (0.02%) (4) 11.01 (0.03%) (5) 1224.67 (3.30%) (6) 243.76 (0.66%) (20) 40.91 (0.11%) Total 37167.04 (sum of others 37064.68) Change-Id: Id4d3b88a9cdef765affc52973aeac951ecc6a8ca
* | | am 8e404536: Merge "Simplification"Jean Chalard2011-12-131-4/+2
|\| | | | | | | | | | | | | | * commit '8e404536cd20a5624acf5cea0c3aae19324b0e3f': Simplification
| * | Merge "Simplification"Jean Chalard2011-12-121-4/+2
| |\ \
| | * | SimplificationJean Chalard2011-12-131-4/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Both tests have the same meaning. They should be merged. Also, if "deleteLast()" deletes more than one char (which never happens in the current implementation, but it's not guaranteed), the new code is more correct. Change-Id: I216df7cd45f4a7d76d1d5d6aa7f8d7f9a6e35ea3