| Commit message (Expand) | Author | Age | Files | Lines |
|---|
| * | Merge "Don't autocorrect after suggestion resuming" into jb-dev |   Jean Chalard | 2012-05-01 | 2 | -0/+15 |
| |\ |
|
| | * | Don't autocorrect after suggestion resuming•••Bug: 6105732
Change-Id: I92e7a9c6d6eb648f747c3b396d7993479fd8478a
| Jean Chalard | 2012-05-01 | 2 | -0/+15 |
| * | | Can override key-style•••Bug: 6412932
Change-Id: I9576ae5b42bcc54cfd8ede0bae5148f3b04c1978
|   Tadashi G. Takaoka | 2012-05-01 | 2 | -56/+47 |
| |/ |
|
| * | Avoid Resources$NotFoundException•••bug: 6410653
Change-Id: I4263632e76b44abf8ecd9114a9eb185ad39201fd
|   Ken Wakasa | 2012-04-28 | 2 | -2/+6 |
| * | Add a flag for bigram presence in the header•••This is a cherry-pick of Icb602762 onto jb-dev.
Bug: 6355745
Change-Id: Icb602762bb0d81472f024fa491571062ec1fc4e9
| Jean Chalard | 2012-04-26 | 2 | -4/+34 |
| * | Merge "Ignore bigrams that are not also listed as unigrams" into jb-dev | Jean Chalard | 2012-04-26 | 3 | -26/+29 |
| |\ |
|
| | * | Ignore bigrams that are not also listed as unigrams•••This is a cherry pick of I14b67e51 on jb-dev
Bug: 6340915
Change-Id: Iaa512abe1b19ca640ea201f9761fd7f1416270ed
| Jean Chalard | 2012-04-26 | 3 | -26/+29 |
| * | | Merge "Fix potential "divided by zero" exception" into jb-dev | Tadashi G. Takaoka | 2012-04-26 | 1 | -44/+35 |
| |\ \ |
|
| | * | | Fix potential "divided by zero" exception•••This change also refactors the language name selection to use text
x-scale.
Bug: 6396854
Change-Id: I31249a85bd042a93d627f40413161aef13617c87
| Tadashi G. Takaoka | 2012-04-26 | 1 | -44/+35 |
| * | | | Merge "Fix SubtypeSwitcher incomplete initialization" into jb-dev | Tadashi G. Takaoka | 2012-04-25 | 1 | -1/+2 |
| |\ \ \
| |/ /
|/| | |
|
| | * | | Fix SubtypeSwitcher incomplete initialization•••Bug: 6400718
Change-Id: Id5dce156ad397b911bfeaedeeddd4aeb21a10b3a
| Tadashi G. Takaoka | 2012-04-26 | 1 | -1/+2 |
| | |/ |
|
| * / | Fix "no language" subtype name (DO NOT MERGE)•••* Move SubtypeLocale.get{Full,Middle,Short}DisplayName() to
LatinLeyboardView and add unit tests (SpacebarTextTests).
* Add SubtypeLocale.getSubtypeDisplayName()
This is a cherry-pick of I57420c6a from Master.
Bug: 6393865
Change-Id: I68748189c17c73984ac4ae05a5a40fb54bf46453
| Tadashi G. Takaoka | 2012-04-26 | 6 | -69/+98 |
| |/ |
|
| * | Support PC QWERTY layout•••Bug: 4460018
Change-Id: I9a29f6a322af66f6a152fa1883b3686c5f7a3328
| Tadashi G. Takaoka | 2012-04-25 | 1 | -0/+3 |
| * | Cancel SubtypePreference dialog properly•••Bug: 6384416
Bug: 6387408
Change-Id: Ia7b0653a09410b42a55915815c887bf0ad56c856
| Tadashi G. Takaoka | 2012-04-25 | 1 | -1/+10 |
| * | Save & restore additional subtype dialog state when orientation is changed•••Bug: 6384416
Change-Id: Id5fa4742ad900fafb82cc37a0bbb9f380b99254d
| Tadashi G. Takaoka | 2012-04-25 | 1 | -67/+100 |
| * | Merge "Ignore unknown keyboard layout safely" | Tadashi G. Takaoka | 2012-04-24 | 3 | -11/+19 |
| |\ |
|
| | * | Ignore unknown keyboard layout safely•••Bug: 6393755
Change-Id: I3a74b23f934cd52925026ad04647cc76de92ee07
| Tadashi G. Takaoka | 2012-04-25 | 3 | -11/+19 |
| * | | Fix percent sign in more keys•••Bug: 6029665
Change-Id: I4bf0f5703d0c11d76bab478da2729b0e20fc64ea
| Tadashi G. Takaoka | 2012-04-25 | 1 | -6/+2 |
| |/ |
|
| * | Merge "Change expandable binary dict write to a temp file first." |   Tom Ouyang | 2012-04-24 | 1 | -1/+4 |
| |\ |
|
| | * | Change expandable binary dict write to a temp file first.•••Bug: 6380724
Change-Id: Ic1d0d902dc45ecb41a1792f33a60ab85e606fcef
| Tom Ouyang | 2012-04-24 | 1 | -1/+4 |
| * | | Merge "Add header commnets of KeyboardTextsSet.tmpl" | Tadashi G. Takaoka | 2012-04-24 | 1 | -1/+17 |
| |\ \ |
|
| | * | | Add header commnets of KeyboardTextsSet.tmpl•••Bug: 6371604
Change-Id: I1054685b9985b6923e3176679b3d95424d8114e0
| Tadashi G. Takaoka | 2012-04-24 | 1 | -1/+17 |
| * | | | Merge "Revert keyHintLetterPadding only to apply horizontally" | Tadashi G. Takaoka | 2012-04-24 | 1 | -2/+2 |
| |\ \ \ |
|
| | * | | | Revert keyHintLetterPadding only to apply horizontally•••Partially reverts the Ia3680d84.
Bug: 6356125
Change-Id: I8faa810b833cb98ce6f0f760aa1722e63475cd45
| Tadashi G. Takaoka | 2012-04-24 | 1 | -2/+2 |
| | |/ / |
|
| * | | | Merge "Fix binary reading code performance." | Jean Chalard | 2012-04-23 | 1 | -1/+10 |
| |\ \ \ |
|
| | * | | | Fix binary reading code performance.•••This is not the Right fix ; the Right fix would be to read
the file in a buffered way. However this delivers tolerable
performance for a minimal amount of code changes.
We may want to skip submitting this patch, but keep it around
in case we need to use the functionality until we have a good
patch.
Change-Id: I1ba938f82acfd9436c3701d1078ff981afdbea60
| Jean Chalard | 2012-04-24 | 1 | -1/+10 |
| * | | | | Merge "Make the "Touch again to save" hint text touchable." | Tadashi G. Takaoka | 2012-04-23 | 1 | -7/+18 |
| |\ \ \ \ |
|
| | * | | | | Make the "Touch again to save" hint text touchable.•••Bug: 6355346
Change-Id: I96933719cc94f25e6c7ecb1d6345c11b6acc4134
| Tadashi G. Takaoka | 2012-04-24 | 1 | -7/+18 |
| | | |/ /
| |/| | |
|
| * | | | | Merge "Fix a bug where a node size would be seen as increasing." | Jean Chalard | 2012-04-23 | 1 | -5/+25 |
| |\ \ \ \ |
|
| | * | | | | Fix a bug where a node size would be seen as increasing.•••The core reason for this is quite shrewd. When a word is a bigram
of itself, the corresponding chargroup will have a bigram referring
to itself. When computing bigram offsets, we use cached addresses of
chargroups, but we compute the size of the node as we go. Hence, a
discrepancy may happen between the base offset as seen by the bigram
(which uses the recomputed value) and the target offset (which uses
the cached value).
When this happens, the cached node address is too large. The relative
offset is negative, which is expected, since it points to this very
charnode whose start is a few bytes earlier. But since the cached
address is too large, the offset is computed as smaller than it should
be.
On the next pass, the cache has been refreshed with the newly computed
size and the seen offset is now correct (or at least, much closer to
correct). The correct value is larger than the previously computed
offset, which was too small. If it happens that it crosses the -255 or
-65335 boundary, the address will be seen as needing 1 more byte than
previously computed. If this is the only change in size of this node,
the node will be seen as having a larger size than previously, which
is unexpected. Debug code was catching this and crashing the program.
So this case is very rare, but in an even rarer occurence, it may
happen that in the same node, another chargroup happens to decrease
it size by the same amount. In this case, the node may be seen as
having not been modified. This is probably extremely rare. If on
top of this, it happens that no other node has been modified, then
the file may be seen as complete, and the discrepancy left as is
in the file, leading to a broken file. The probability that this
happens is abyssally low, but the bug exists, and the current debug
code would not have caught this.
To further catch similar bugs, this change also modifies the test
that decides if the node has changed. On grounds that all components
of a node may only decrease in size with each successive pass, it's
theoritically safe to assume that the same size means the node
contents have not changed, but in case of a bug like the bug above
where a component wrongly grows while another shrinks and both cancel
each other out, the new code will catch this. Also, this change adds
a check against the number of passses, to avoid infinite loops in
case of a bug in the computation code.
This change fixes this bug by updating the cached address of each
chargroup as we go. This eliminates the discrepancy and fixes the
bug.
Bug: 6383103
Change-Id: Ia3f450e22c87c4c193cea8ddb157aebd5f224f01
| Jean Chalard | 2012-04-24 | 1 | -5/+25 |
| | | |/ /
| |/| | |
|
| * | | | | Fix NPE in AdditionalSubtypeSettings•••Bug: 6384416
Change-Id: I878071b8b78acce51649f9637b59830ab681c696
| Tadashi G. Takaoka | 2012-04-24 | 1 | -0/+1 |
| | |/ /
|/| | |
|
| * | | | Merge "Fix a bug where two spaces would be added instead of one" | Jean Chalard | 2012-04-23 | 1 | -15/+16 |
| |\ \ \
| |/ /
|/| | |
|
| | * | | Fix a bug where two spaces would be added instead of one•••Bug: 6378291
Change-Id: Id2453333ae16146f816faf0eb68baa8ea1dd106a
| Jean Chalard | 2012-04-23 | 1 | -15/+16 |
| * | | | Merge "Fix a bug where a batch edit would not be started." | Jean Chalard | 2012-04-23 | 1 | -1/+3 |
| |\| | |
|
| | * | | Fix a bug where a batch edit would not be started.•••This would end up in TextView sometimes calling onUpdateSelection
multiple times (this is the correct behavior for TextView). We now
commit the space and the word in a batch edit, and we only get
onUpdateSelection once.
Bug: 6300527
Change-Id: I9579f3d8f5320c1cc24a7a42f19db8e105eb090d
| Jean Chalard | 2012-04-23 | 1 | -1/+3 |
| * | | | Merge "Pass words as int[] to the native code." | Jean Chalard | 2012-04-23 | 1 | -6/+6 |
| |\ \ \
| |_|/
|/| | |
|
| | * | | Pass words as int[] to the native code.•••We need to get the bigrams during the call to getSuggestions for
bug#6313806. We already give an int[] to getSuggestions and we
wanted to get rid of char[]'s anyway because it doesn't work with
surrogate pairs, so here we go.
Bug: 6313806
Change-Id: I56ce99f1db6b3302cdf42f0527343bded837091e
| Jean Chalard | 2012-04-23 | 1 | -6/+6 |
| * | | | Merge "Binary bigram lookup checks both uppercase and lowercase when previous... | Tom Ouyang | 2012-04-23 | 2 | -14/+37 |
| |\ \ \ |
|
| | * | | | Binary bigram lookup checks both uppercase and lowercase when previous word i...•••Bug: 6363029
Change-Id: I71a1ed5b88f54fcc15bfaf89d3645b8f17ebb821
| Tom Ouyang | 2012-04-22 | 2 | -14/+37 |
| * | | | | Fix key preview when shifted hint letter is activated•••Change-Id: I70d2627318d21eb612c3c3fee2ce8e80555dba12
| Tadashi G. Takaoka | 2012-04-23 | 1 | -3/+4 |
| | |_|/
|/| | |
|
| * | | | Merge "Fix using keyboard layout display name in custom input style spinner" | Tadashi G. Takaoka | 2012-04-22 | 1 | -9/+8 |
| |\ \ \
| |_|/
|/| | |
|
| | * | | Fix using keyboard layout display name in custom input style spinner•••Change-Id: I80626591574aa9995011202cf868f9c2c5168303
| Tadashi G. Takaoka | 2012-04-23 | 1 | -9/+8 |
| | |/ |
|
| * / | Use the best matching cached dictionary for each category•••Bug: 6327270
Change-Id: I5a0e732c8a3fd55fd8ac3c8fe1c58e7f91555d97
| Jean Chalard | 2012-04-23 | 1 | -9/+62 |
| |/ |
|
| * | Remove unused methods and classes•••Change-Id: I1a82bf2202d436efa4c5135d705a048fe45a0257
| Ken Wakasa | 2012-04-22 | 1 | -75/+0 |
| * | Remove an unused method•••Change-Id: I3abbf6462c31b11167e356fe77b8afca2514a869
| Ken Wakasa | 2012-04-22 | 1 | -24/+0 |
| * | Fix the build•••Change-Id: I5c4e230500f058f7db86c05d554be45411004ffc
|   Svetoslav Ganov | 2012-04-20 | 1 | -2/+4 |
| * | Merge "Add language switch key to symbols and symbols shifted layout" | Tadashi G. Takaoka | 2012-04-20 | 1 | -2/+5 |
| |\ |
|
| | * | Add language switch key to symbols and symbols shifted layout•••Bug: 6370179
Change-Id: I061559ea5d0cb11a059de4b6e741f9c17ee69a27
| Tadashi G. Takaoka | 2012-04-20 | 1 | -2/+5 |
| * | | Merge "Predefined keyboard layouts are configurable via XML resource" | Tadashi G. Takaoka | 2012-04-20 | 4 | -35/+46 |
| |\ \
| |/
|/| |
|
| | * | Predefined keyboard layouts are configurable via XML resource•••Bug: 4460018
Change-Id: Ia03ec826fb3853ea814d9764686903dbe31a6507
| Tadashi G. Takaoka | 2012-04-20 | 4 | -35/+46 |
