| Commit message (Expand) | Author | Age | Files | Lines |
|---|
| ... | |
| | * | | Fix a bug with the user dictionary locale.•••We internally use the string "zz" to mean no locale, but
the user dictionary service does not understand that and
words added with this locale pitifully end being added to
a "zz" user dictionary, which makes no sense to the user.
This change fixes things so that words added when the
keyboard is in a "no locale" layout, the words are added
to the global, all-locales user dictionary.
Bug: 6645306
Change-Id: Iec3cdd1fe3d5bc43427a43ef9ae9bf89e12be1f2
|   Jean Chalard | 2012-07-02 | 1 | -2/+10 |
| * | | | Small refactoring (A39)•••Change-Id: I3c45b5a6571f986acc7adb3a8fe17b210303f7ea
| Jean Chalard | 2012-07-02 | 1 | -14/+14 |
| * | | | Match calls for future consolidation (A38)•••Change-Id: I2b2ab99e801fa8b4b30eab16a8aefb8cfe104ba9
| Jean Chalard | 2012-07-02 | 1 | -3/+3 |
| * | | | Make the autocorrection scheme more straightforward (A37)•••Change-Id: I4833ca7e057b8c1f1f22390673cbca0d2433b1bd
| Jean Chalard | 2012-07-02 | 2 | -6/+7 |
| |/ / |
|
| * | | Remove a duplicate method (A36)•••Bug: 6705473
Change-Id: Ib6260034b09739b60071e90a018b2dc6be3815d7
| Jean Chalard | 2012-07-02 | 2 | -5/+1 |
| * | | Update log and make a variable private (A35).•••Change-Id: Ib4a8f41dc5941bce46402e1a41a6c21dccb0601f
| Jean Chalard | 2012-07-02 | 2 | -3/+2 |
| * | | Remove old debug code that has been long broken (A34)•••Change-Id: I9b5a0e34529724984aad1e2bf99b3e00f8ed0dd7
| Jean Chalard | 2012-06-29 | 1 | -3/+2 |
| * | | Merge "Add input pointers" |   Satoshi Kataoka | 2012-06-29 | 5 | -33/+155 |
| |\ \ |
|
| | * | | Add input pointers•••Change-Id: I95300bf0a847fb86d026e846ff4ad723bb45284f
| Satoshi Kataoka | 2012-06-29 | 5 | -33/+155 |
| * | | | Merge "Add a more synthetic variable (A33)" | Jean Chalard | 2012-06-29 | 1 | -1/+3 |
| |\ \ \ |
|
| | * | | | Add a more synthetic variable (A33)•••The goal here is to disambiguate TypedWordValid and
HasAutoCorrectionCandidate. Ultimately, HasAutoCorrectionCandidate
will disappear.
Bug: 6705473
Change-Id: I1acbd0a5efc7f4bf7b0bdb12839fe3d84e9d1dda
| Jean Chalard | 2012-06-29 | 1 | -1/+3 |
| * | | | | Merge "Remove cruft (A32)" | Jean Chalard | 2012-06-29 | 1 | -7/+1 |
| |\| | | |
|
| | * | | | Remove cruft (A32)•••Change-Id: I35c2ba4caf6be4ba0449e11c0597ae8e468c0e6e
| Jean Chalard | 2012-06-29 | 1 | -7/+1 |
| * | | | | Merge "Simplification (A31)" | Jean Chalard | 2012-06-29 | 1 | -6/+2 |
| |\| | | |
|
| | * | | | Simplification (A31)•••Change-Id: I9e4d030974ccec1a6256de6ca87610870e404149
| Jean Chalard | 2012-06-29 | 1 | -6/+2 |
| * | | | | Merge "Small refactoring (A30)" | Jean Chalard | 2012-06-29 | 1 | -4/+4 |
| |\| | | |
|
| | * | | | Small refactoring (A30)•••Change-Id: I41b413986a3c5bb6697bb7bbced17ed1e361913c
| Jean Chalard | 2012-06-29 | 1 | -4/+4 |
| * | | | | Merge "Use SparseArray<E> instead of HashMap<Integer,E>" |   Tadashi G. Takaoka | 2012-06-29 | 7 | -51/+55 |
| |\ \ \ \
| |/ / /
|/| | | |
|
| | * | | | Use SparseArray<E> instead of HashMap<Integer,E>•••Change-Id: Id962e670ee1a5164e6c69deb84625139bf5e7974
| Tadashi G. Takaoka | 2012-06-29 | 7 | -51/+55 |
| * | | | | Remove a useless parameter (A29)•••Change-Id: I52625e707abf61da9b95e542f0814c66b532f483
| Jean Chalard | 2012-06-29 | 4 | -6/+1 |
| * | | | | Remove a useless variable (A28)•••Change-Id: I1058123d233540fb72a94028987a43740224f635
| Jean Chalard | 2012-06-29 | 2 | -5/+0 |
| * | | | | Fix a small bug (A27)•••TypedWordValid and AllowsToBeAutoCorrected are almost the
same thing; the difference is whether the word is also a
whitelist entry or not. In this case, TypedWordValid was
the right one to test.
Change-Id: I695fe5b9f6bbe9d3ba314c646da3a25742c9f4ed
| Jean Chalard | 2012-06-29 | 1 | -1/+1 |
| * | | | | Optimization (A26)•••Change-Id: I1d7790289635729645995fb8c38868259ab1b3be
| Jean Chalard | 2012-06-29 | 1 | -9/+2 |
| * | | | | Remove useless code (A25)•••isWhitelistedOrNotAWord takes an 'ignoreCase' argument. By looking
at the contents of the wordcomposer here, there is only one case
where its output will be different : when the word is typed with a
capital, but the lower case version exists in the dictionary.
E.g. the user typed "This".
In this case, isWhitelistedOrNotAWord in line 235 will return false
instead of true, so the test will score a true instead of a false,
so hasAutoCorrection may be true instead of false in this specific
case and that's the only case where it's different.
But in this case, allowsToBeAutoCorrected is certain to be false,
which means the result will not have changed if hasAutoCorrection
was true in the first place. So in the end this change is sure not
to change the behavior.
Change-Id: Ic41cf959c20c19165f84d9b8ff006731fa595d84
| Jean Chalard | 2012-06-29 | 1 | -4/+1 |
| * | | | | Add a few comments for future reference (A24)•••Change-Id: Ia356130b8a95ec9cf676bc6be05d801c13f14b64
| Jean Chalard | 2012-06-29 | 1 | -0/+6 |
| * | | | | Rename a method for readability (A23)•••Change-Id: I6f569322e645d36bc82fddf4ec2fd621972e3b47
| Jean Chalard | 2012-06-29 | 2 | -2/+2 |
| * | | | | Revert a test for readability (A22)•••Change-Id: I1df4cef5ae2736a22273843dce67f0012181bfd3
| Jean Chalard | 2012-06-29 | 1 | -14/+12 |
| * | | | | Remove a useless variable (A21)•••Change-Id: I901e6b0e2b994128da19d68233b9f68bb308cb95
| Jean Chalard | 2012-06-29 | 1 | -4/+4 |
| * | | | | Remove a useless variable (A20)•••Change-Id: Ifc37462f5243563693c8cd7cbeb401ff054daa52
| Jean Chalard | 2012-06-29 | 1 | -6/+4 |
| * | | | | Rename a method for readability (A19)•••Change-Id: I0b8c06d1fd0bfb3caf363c048acc7048abed1752
| Jean Chalard | 2012-06-29 | 2 | -3/+3 |
| * | | | | Remove a useless test (A18)•••allowsToBeAutoCorrected always returns false if the word is empty.
This is because the whitelist never contains an empty string,
and isValidWord returns false if the word is empty.
Change-Id: I34ecc2a1563aea6db5b2f12796f251f6598576a2
| Jean Chalard | 2012-06-29 | 1 | -2/+2 |
| * | | | | Inline a method (A17)•••This will allow us to remove unnecessary processing
Change-Id: I251dfdaffb077906186686ebe65fd82e1e360bd2
| Jean Chalard | 2012-06-29 | 2 | -8/+2 |
| * | | | | Make a function a one-liner (A16)•••...to better inline it
Change-Id: I88496b8f18b81f0c05e4bbf2d1039de1f93e0c25
| Jean Chalard | 2012-06-29 | 1 | -2/+2 |
| * | | | | Inline a simple method (A15)•••Change-Id: Ib74fcbf33c680bf02c39a9896e30e3afbcfd149a
| Jean Chalard | 2012-06-29 | 2 | -5/+1 |
| * | | | | Inline a method (A14)•••The new code is worse than the old one, but this is a necessary
step to make things prettier.
Change-Id: If6e8a139bb85e6920c749743c78792a22a8acb45
| Jean Chalard | 2012-06-29 | 2 | -23/+14 |
| |/ / / |
|
| * | | | Don't special-case 1-letter words (A13)•••There is no reason to prevent 1-letter words to auto-correct
to themselves, or to dictionary words. Don't do it.
Change-Id: Iceada847ae632336026ada29afed0353cd9c51b5
| Jean Chalard | 2012-06-29 | 1 | -8/+6 |
| * | | | Fix a theoretical bug (A12)•••If a word is a dictionary word but still has no suggestion,
this method should return true. In the practice, it makes
no difference since a word without suggestion won't be changed
anyway.
Change-Id: Ib1f5ef254b7da7e5cedb2f973529ad431beb93f9
| Jean Chalard | 2012-06-29 | 1 | -5/+3 |
| * | | | Rename a terribly-named method for clarity (A11)•••Change-Id: Id180000404dc6929d80f98c0e9cd2026230ad2d1
| Jean Chalard | 2012-06-29 | 1 | -2/+3 |
| * | | | Merge "Small readability improvement (A10)" | Jean Chalard | 2012-06-28 | 1 | -1/+3 |
| |\ \ \ |
|
| | * | | | Small readability improvement (A10)•••Change-Id: Iba326921711e1c4eab38264e66b6b44a64fec398
| Jean Chalard | 2012-06-29 | 1 | -1/+3 |
| | |/ / |
|
| * / / | Refactoring and add debug prints (A9)•••Change-Id: I46497bff107f0def0dfa2b043b20d9a0b0cde040
| Jean Chalard | 2012-06-29 | 1 | -4/+13 |
| |/ / |
|
| * | | Optimization (A8)•••Change-Id: I2b9e8cc6a13fd9af84a5189e4d3fbb2b05c68d46
| Jean Chalard | 2012-06-29 | 1 | -9/+13 |
| * | | Cleanup (A7)•••Change-Id: Iba97e0062b702c005ab3ff9ce59f181d19ab7c98
| Jean Chalard | 2012-06-28 | 1 | -4/+2 |
| * | | Stop special-casing a single suggestion (A6)•••Treat it like the others
Change-Id: Ic24e40ada0e75773c3ece9470ab61c350697be5d
| Jean Chalard | 2012-06-28 | 1 | -13/+12 |
| * | | Cleanup (A5)•••Change-Id: I06134145b47c46bbc6c340ca539f34a01e2ca163
| Jean Chalard | 2012-06-28 | 2 | -18/+0 |
| * | | Transform the whitelist suggestion like the others (A4)•••...instead of having a different code path
Change-Id: Icd177e2151e45d25300000d4b63d4fba1f52b6a8
| Jean Chalard | 2012-06-28 | 1 | -16/+7 |
| * | | Small refactoring (A3)•••This will help make upcoming changes clearer
Change-Id: Ida8d4935f7315e5da6007ad364124694da3112d0
| Jean Chalard | 2012-06-28 | 1 | -6/+8 |
| * | | Don't pass everything to a function that needs only the head (A2)•••Change-Id: Ic367836202ab8071c1a9a02eaf0651b0da947d51
| Jean Chalard | 2012-06-28 | 2 | -13/+14 |
| * | | Merge "Use a TreeSet to sort suggestions instead of doing it by hand (A1)" | Jean Chalard | 2012-06-28 | 2 | -28/+57 |
| |\ \ |
|
| | * | | Use a TreeSet to sort suggestions instead of doing it by hand (A1)•••Change-Id: I16ba39321107e87ad48a99c2410a15995a66f23c
| Jean Chalard | 2012-06-28 | 2 | -28/+57 |
