| Commit message (Expand) | Author | Age | Files | Lines |
|---|
| ... | |
| * | Simplification (A31)•••Change-Id: I9e4d030974ccec1a6256de6ca87610870e404149
|   Jean Chalard | 2012-06-29 | 1 | -6/+2 |
| * | Small refactoring (A30)•••Change-Id: I41b413986a3c5bb6697bb7bbced17ed1e361913c
| Jean Chalard | 2012-06-29 | 1 | -4/+4 |
| * | Remove a useless parameter (A29)•••Change-Id: I52625e707abf61da9b95e542f0814c66b532f483
| Jean Chalard | 2012-06-29 | 1 | -1/+0 |
| * | 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 | 1 | -1/+1 |
| * | 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 | 1 | -2/+2 |
| * | 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 | 1 | -1/+2 |
| * | Inline a simple method (A15)•••Change-Id: Ib74fcbf33c680bf02c39a9896e30e3afbcfd149a
| Jean Chalard | 2012-06-29 | 1 | -1/+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 | 1 | -4/+11 |
| * | Small readability improvement (A10)•••Change-Id: Iba326921711e1c4eab38264e66b6b44a64fec398
| Jean Chalard | 2012-06-29 | 1 | -1/+3 |
| * | 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 | 1 | -17/+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 | 1 | -1/+3 |
| * | Use a TreeSet to sort suggestions instead of doing it by hand (A1)•••Change-Id: I16ba39321107e87ad48a99c2410a15995a66f23c
| Jean Chalard | 2012-06-28 | 1 | -28/+8 |
| * | Convert local members to local variables•••Change-Id: I852787986c79502a4de7effe9c8281a4cf2d519d
| Jean Chalard | 2012-06-27 | 1 | -17/+13 |
| * | Optimization.•••It's useless to do this for all suggestions including those
that will be ousted from the list.
Change-Id: I7bd63d783e21984a269e4b1ae17e9b9d8c0859e3
| Jean Chalard | 2012-06-27 | 1 | -7/+11 |
| * | Read the suggestion type from the suggestion•••...instead of dirtily passing it from the dictionary loop
Change-Id: I92ddbd7451609b69f6d4dc31bf89d614423b2016
| Jean Chalard | 2012-06-27 | 1 | -4/+5 |
| * | Remember the source dictionary for each suggestion.•••Change-Id: I3c63372bd5572a479a67eaecfe8c8ea1cabc70d9
| Jean Chalard | 2012-06-27 | 1 | -5/+7 |
| * | Move constants to a better place.•••Change-Id: I5c27a3ed99b17f850e26a8503de16f001c7111c1
| Jean Chalard | 2012-06-27 | 1 | -17/+9 |
| * | Cleanup•••Change-Id: I575f22330bf6881af1d9a35f22b51b7bf2198113
| Jean Chalard | 2012-06-27 | 1 | -17/+8 |
| * | Remove unused stuff.•••Change-Id: Ifd0d5b6978f76b841574a6e10157c32d0cd656b6
| Jean Chalard | 2012-06-27 | 1 | -24/+0 |
| * | Update the prototype of onAddSuggestedWord - calling side.•••We want to get rid of all those IDs in Latin IME.
Change-Id: Ibe08100a5e2976c80abb049889233b4047a686f3
| Jean Chalard | 2012-06-27 | 1 | -10/+6 |
| * | Use binarySearch instead of a hand-written linear search•••Yay.
Change-Id: Icf686eb2d675da959f19edc856b521e6288dcea3
| Jean Chalard | 2012-06-27 | 1 | -9/+5 |
| * | Implement a comparator for SuggestedWordInfos.•••Change-Id: Ic4b0ae05cd98a1aec5806c5e4b744158e73aea7a
| Jean Chalard | 2012-06-27 | 1 | -4/+18 |
| * | Merge "Remove some special casing" | Jean Chalard | 2012-06-26 | 1 | -27/+10 |
| |\ |
|
| | * | Remove some special casing•••This special casing is useless. If the word is the same as what
user typed, the scoring algorithm already ensures that it comes
out at the top. Actually, as is written in a comment here, code
executed later is actively relying on this suggestion having
the top score ! There is no need to test it for equalness and
inserting it at the top then.
Change-Id: I263a6de59b77ec72a2dcbb933361b8e16fca0681
| Jean Chalard | 2012-06-26 | 1 | -27/+10 |
| * | | Fix a bug where a log would mysteriously not work•••Change-Id: I5ca77ed65e95a6d357215c0e730238fb463986bb
| Jean Chalard | 2012-06-26 | 1 | -3/+2 |
| |/ |
|
| * | Optimization.•••We already had that computed
Change-Id: I2a9742eaee2e9db1f73aeddda7bae7e237b4d15c
| Jean Chalard | 2012-06-26 | 1 | -2/+2 |
| * | Cut out a method for clarity•••Change-Id: I1dc668698a5cd840297e713a26a525f5e32d35d6
| Jean Chalard | 2012-06-26 | 1 | -13/+25 |
| * | Avoid string conversions when possible.•••Change-Id: Ibf5f3e40da55998364d0d835ecf283f32c80fcf5
| Jean Chalard | 2012-06-26 | 1 | -4/+4 |
| * | Simplify a method call.•••Change-Id: Ieede787a13cef79353f13af52488ef3732ac9850
| Jean Chalard | 2012-06-26 | 1 | -24/+21 |
| * | Refactor and squash long-standing bugs (A2)•••This is awesome.
- Improve performance by stopping useless string/array conversions.
- Simplify method interface.
- Fix localization bugs - we now know our locale
- Fix bugs with surrogate code points
Bug: 4769095
Change-Id: I8a8ee224b713429ee0a4ce5bd52f1b63378cf56f
| Jean Chalard | 2012-06-26 | 1 | -13/+9 |
| * | Remember the locale in the Suggest object (A1)•••This will be necessary for uppercasing and titlecasing words
correctly.
Change-Id: I0f446a4c82bc482240ff730a2ad6586a61571d6b
| Jean Chalard | 2012-06-26 | 1 | -0/+5 |
| * | Remove a useless member (A2)•••Change-Id: Ia727ffc2946352bbf79e0c97684f8fa98f428cef
| Jean Chalard | 2012-06-26 | 1 | -7/+6 |
| * | Convert a member to a local variable (A1)•••Change-Id: I9eab910989443704f83711cf4fb0070c5987a13a
| Jean Chalard | 2012-06-26 | 1 | -17/+17 |
| * | Remove an unused method (A22)•••Change-Id: I2195f365df1799c8b3b2a53670d1df535ab1a742
| Jean Chalard | 2012-06-26 | 1 | -8/+0 |
| * | Simplification and optimization (A20)•••Change-Id: Ia837e553ccfe67cd4303096290194a5186df059b
| Jean Chalard | 2012-06-26 | 1 | -25/+14 |
| * | Preparation for a simplification (A19)•••Change-Id: I4605a571faf5fbc68b18ce525d6019600bfa8cf7
| Jean Chalard | 2012-06-26 | 1 | -0/+4 |
| * | Remove a meaningless separation (A18)•••Change-Id: I267177044c7d7b0d9119839a11057b2bbf41f75f
| Jean Chalard | 2012-06-26 | 1 | -21/+14 |
| * | Remove the Callback interface for simpler code. (A17)•••Bug: 6252660
Bug: 6166228
Bug: 2704000
Bug: 6225530
Change-Id: I009b54fc2e73cadca6734bdca074d033e81f6a2b
| Jean Chalard | 2012-06-26 | 1 | -8/+1 |
