aboutsummaryrefslogtreecommitdiffstats
path: root/java/src (follow)
Commit message (Collapse)AuthorAgeFilesLines
...
| * Ignore unknown keyboard layout safelyTadashi G. Takaoka2012-04-253-11/+19
| | | | | | | | | | Bug: 6393755 Change-Id: I3a74b23f934cd52925026ad04647cc76de92ee07
* | Fix percent sign in more keysTadashi G. Takaoka2012-04-251-6/+2
|/ | | | | Bug: 6029665 Change-Id: I4bf0f5703d0c11d76bab478da2729b0e20fc64ea
* Merge "Change expandable binary dict write to a temp file first."Tom Ouyang2012-04-241-1/+4
|\
| * Change expandable binary dict write to a temp file first.Tom Ouyang2012-04-241-1/+4
| | | | | | | | | | Bug: 6380724 Change-Id: Ic1d0d902dc45ecb41a1792f33a60ab85e606fcef
* | Merge "Add header commnets of KeyboardTextsSet.tmpl"Tadashi G. Takaoka2012-04-241-1/+17
|\ \
| * | Add header commnets of KeyboardTextsSet.tmplTadashi G. Takaoka2012-04-241-1/+17
| | | | | | | | | | | | | | | Bug: 6371604 Change-Id: I1054685b9985b6923e3176679b3d95424d8114e0
* | | Merge "Revert keyHintLetterPadding only to apply horizontally"Tadashi G. Takaoka2012-04-241-2/+2
|\ \ \
| * | | Revert keyHintLetterPadding only to apply horizontallyTadashi G. Takaoka2012-04-241-2/+2
| |/ / | | | | | | | | | | | | | | | | | | Partially reverts the Ia3680d84. Bug: 6356125 Change-Id: I8faa810b833cb98ce6f0f760aa1722e63475cd45
* | | Merge "Fix binary reading code performance."Jean Chalard2012-04-231-1/+10
|\ \ \
| * | | Fix binary reading code performance.Jean Chalard2012-04-241-1/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | Merge "Make the "Touch again to save" hint text touchable."Tadashi G. Takaoka2012-04-231-7/+18
|\ \ \ \
| * | | | Make the "Touch again to save" hint text touchable.Tadashi G. Takaoka2012-04-241-7/+18
| | |/ / | |/| | | | | | | | | | | | | | Bug: 6355346 Change-Id: I96933719cc94f25e6c7ecb1d6345c11b6acc4134
* | | | Merge "Fix a bug where a node size would be seen as increasing."Jean Chalard2012-04-231-5/+25
|\ \ \ \
| * | | | Fix a bug where a node size would be seen as increasing.Jean Chalard2012-04-241-5/+25
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | Fix NPE in AdditionalSubtypeSettingsTadashi G. Takaoka2012-04-241-0/+1
| |/ / |/| | | | | | | | | | | Bug: 6384416 Change-Id: I878071b8b78acce51649f9637b59830ab681c696
* | | Merge "Fix a bug where two spaces would be added instead of one"Jean Chalard2012-04-231-15/+16
|\ \ \ | |/ / |/| |
| * | Fix a bug where two spaces would be added instead of oneJean Chalard2012-04-231-15/+16
| | | | | | | | | | | | | | | Bug: 6378291 Change-Id: Id2453333ae16146f816faf0eb68baa8ea1dd106a
* | | Merge "Fix a bug where a batch edit would not be started."Jean Chalard2012-04-231-1/+3
|\| |
| * | Fix a bug where a batch edit would not be started.Jean Chalard2012-04-231-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | Merge "Pass words as int[] to the native code."Jean Chalard2012-04-231-6/+6
|\ \ \ | |_|/ |/| |
| * | Pass words as int[] to the native code.Jean Chalard2012-04-231-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | Merge "Binary bigram lookup checks both uppercase and lowercase when ↵Tom Ouyang2012-04-232-14/+37
|\ \ \ | | | | | | | | | | | | previous word is uppercase."
| * | | Binary bigram lookup checks both uppercase and lowercase when previous word ↵Tom Ouyang2012-04-222-14/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | is uppercase. Bug: 6363029 Change-Id: I71a1ed5b88f54fcc15bfaf89d3645b8f17ebb821
* | | | Fix key preview when shifted hint letter is activatedTadashi G. Takaoka2012-04-231-3/+4
| |_|/ |/| | | | | | | | Change-Id: I70d2627318d21eb612c3c3fee2ce8e80555dba12
* | | Merge "Fix using keyboard layout display name in custom input style spinner"Tadashi G. Takaoka2012-04-221-9/+8
|\ \ \ | |_|/ |/| |
| * | Fix using keyboard layout display name in custom input style spinnerTadashi G. Takaoka2012-04-231-9/+8
| |/ | | | | | | Change-Id: I80626591574aa9995011202cf868f9c2c5168303
* / Use the best matching cached dictionary for each categoryJean Chalard2012-04-231-9/+62
|/ | | | | Bug: 6327270 Change-Id: I5a0e732c8a3fd55fd8ac3c8fe1c58e7f91555d97
* Remove unused methods and classesKen Wakasa2012-04-221-75/+0
| | | | Change-Id: I1a82bf2202d436efa4c5135d705a048fe45a0257
* Remove an unused methodKen Wakasa2012-04-221-24/+0
| | | | Change-Id: I3abbf6462c31b11167e356fe77b8afca2514a869
* Fix the buildSvetoslav Ganov2012-04-201-2/+4
| | | | Change-Id: I5c4e230500f058f7db86c05d554be45411004ffc
* Merge "Add language switch key to symbols and symbols shifted layout"Tadashi G. Takaoka2012-04-201-2/+5
|\
| * Add language switch key to symbols and symbols shifted layoutTadashi G. Takaoka2012-04-201-2/+5
| | | | | | | | | | Bug: 6370179 Change-Id: I061559ea5d0cb11a059de4b6e741f9c17ee69a27
* | Merge "Predefined keyboard layouts are configurable via XML resource"Tadashi G. Takaoka2012-04-204-35/+46
|\ \ | |/ |/|
| * Predefined keyboard layouts are configurable via XML resourceTadashi G. Takaoka2012-04-204-35/+46
| | | | | | | | | | Bug: 4460018 Change-Id: Ia03ec826fb3853ea814d9764686903dbe31a6507
* | Allow using a format version 2 word list in LatinIMEJean Chalard2012-04-201-7/+13
|/ | | | Change-Id: I73a4df3a83e49be6e8d3a7d14eb027cfe10f1a23
* Support Dvorak layoutTadashi G. Takaoka2012-04-202-2/+10
| | | | | Bug: 4460018 Change-Id: I08a9b556720f6c4f7089a8838c0869c9f88f479c
* Merge "Don't allow cancelling manual pick."Jean Chalard2012-04-201-0/+2
|\
| * Don't allow cancelling manual pick.Jean Chalard2012-04-201-0/+2
| | | | | | | | | | Bug: 6347202 Change-Id: I30b53256e83c534436b0f0ff594fd4aaabf4558c
* | Fix confusable "label" usage, use "text" insteadTadashi G. Takaoka2012-04-204-60/+60
|/ | | | | | | | * Rename makelabel tool to maketext * Rename KeyboardLablesSet class to KeyboardTextsSet * Rename "!label/name" to "!text/name" Change-Id: Ia9494a89fe56f20ca8e8fedb253adbcf8b771bd3
* Use string attribute for icon nameTadashi G. Takaoka2012-04-205-85/+87
| | | | Change-Id: I336acf33c7e6bc993b9da9b17ec689975fa8127d
* Merge "Make KeySpecParser case insensitive"Tadashi G. Takaoka2012-04-194-24/+47
|\
| * Make KeySpecParser case insensitiveTadashi G. Takaoka2012-04-204-24/+47
| | | | | | | | Change-Id: I76c3e9179dd8777d3cf3138ad76513d83672debf
* | Merge "remove unnecessary KeyboardState logging"Kurt Partridge2012-04-193-153/+0
|\ \
| * | remove unnecessary KeyboardState loggingKurt Partridge2012-04-193-153/+0
| | | | | | | | | | | | | | | Bug: 6188932 Change-Id: Iafbb1825777122a1810fa3339853263aced3077e
* | | Merge "researchLogger detail on IME sessions"Kurt Partridge2012-04-193-55/+107
|\| |
| * | researchLogger detail on IME sessionsKurt Partridge2012-04-193-55/+107
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | log device type, screen size, etc. also simplification inside ResearchLogger to use exceptions better, and fix ResearchLoggerTests to not change preferences to specify the logfile. multi-project change with Idfd193b16260215cca32886e288f03a0c12b2781 Bug: 6188932 Change-Id: Idd3ca6eb8e8e3934f68234b241c7fd7d0cc3bcd5
* | | Merge "Change binary dictionary output buffer size to match dictionary size."Tom Ouyang2012-04-191-32/+65
|\ \ \ | |_|/ |/| |
| * | Change binary dictionary output buffer size to match dictionary size.Tom Ouyang2012-04-191-32/+65
| | | | | | | | | | | | | | | Bug: 6355943 Change-Id: Iaab7bc16ba0dbc7bfde70b06e7bd355519838831
* | | Add shortcut_as_more_key labelTadashi G. Takaoka2012-04-201-329/+323
| |/ |/| | | | | | | | | | | This change also optimezes KeyboardLabelsSet.java by reordering donottranslate-more-keys.xml in makelabel tool. Change-Id: Ief46ee783454bedcdffb0f24d9d03df51b6edcd6
* | Fix comment handling of makelabel toolTadashi G. Takaoka2012-04-191-29/+62
| | | | | | | | Change-Id: Id40336b543987eee1d2777394d9ee5d0a6343a4c
generated by cgit v1.2.3-83-g751a (git 2.47.1) at 2025-05-11 03:28:54 +0000