| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
`jni_libs` is more specific than `required`, and allows Soong to be more
restrictive when creating the dependency edge from the java_binary to
the cc_library.
Bug: 370110572
Test: m installclean && m dicttool_aosp
Test: verified that libjni_latinime gets installed in
out/host/linux-x86/lib64
Change-Id: I49bde551c65e344c566decffc7f447fc4813f70b
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This commit is part of a large scale change to fix errorprone
errors that have been downgraded to warnings in the android
source tree, so that they can be promoted to errors again.
The full list of changes include the following, but not all
will be present in any one individual commit:
BadAnnotationImplementation
BadShiftAmount
BanJNDI
BoxedPrimitiveEquality
ComparableType
ComplexBooleanConstant
CollectionToArraySafeParameter
ConditionalExpressionNumericPromotion
DangerousLiteralNull
DoubleBraceInitialization
DurationFrom
DurationTemporalUnit
EmptyTopLevelDeclaration
EqualsNull
EqualsReference
FormatString
FromTemporalAccessor
GetClassOnAnnotation
GetClassOnClass
HashtableContains
IdentityBinaryExpression
IdentityHashMapBoxing
InstantTemporalUnit
InvalidTimeZoneID
InvalidZoneId
IsInstanceIncompatibleType
JUnitParameterMethodNotFound
LockOnBoxedPrimitive
MathRoundIntLong
MislabeledAndroidString
MisusedDayOfYear
MissingSuperCall
MisusedWeekYear
ModifyingCollectionWithItself
NoCanIgnoreReturnValueOnClasses
NonRuntimeAnnotation
NullableOnContainingClass
NullTernary
OverridesJavaxInjectableMethod
ParcelableCreator
PeriodFrom
PreconditionsInvalidPlaceholder
ProtoBuilderReturnValueIgnored
ProtoFieldNullComparison
RandomModInteger
RectIntersectReturnValueIgnored
ReturnValueIgnored
SelfAssignment
SelfComparison
SelfEquals
SizeGreaterThanOrEqualsZero
StringBuilderInitWithChar
TreeToString
TryFailThrowable
UnnecessaryCheckNotNull
UnusedCollectionModifiedInPlace
XorPower
See https://errorprone.info/bugpatterns for more
information on the checks.
Bug: 253827323
Test: m RUN_ERROR_PRONE=true javac-check
Change-Id: I454a105ae82484a2d19aff1808e8d9dd55ba64f4
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Added SPDX-license-identifier-Apache-2.0 to:
Android.bp
common/Android.bp
java/Android.bp
native/dicttoolkit/Android.bp
native/jni/Android.bp
tests/Android.bp
tools/EditTextVariations/Android.bp
tools/dicttool/Android.bp
tools/make-keyboard-text/Android.bp
Bug: 68860345
Bug: 151177513
Bug: 151953481
Test: m all
Exempt-From-Owner-Approval: janitorial work
Change-Id: I440008bffac5c97a2497970af377a9d03262b6d8
|
|
|
|
|
|
|
|
| |
See https://source.android.com/setup/contribute/respectful-code for reference
Bug: 161896447
Test: tapas LatinIME && m
Change-Id: Ic06f948c023a091e1532db826a78dd01031d7205
|
|
|
|
|
|
|
| |
See build/soong/README.md for more information.
Test: cd packages/inputmethods/LatinIME; mma
Change-Id: Ib8867d3b74f09fc1d9f95adc9a49a81ac0f7f054
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL partially reverts
- Id88b02b74bdfe4ca05b08181ceb6b34d5652fc0c
- I05c7d8429e8d9a26139456763c77997340fea8c2
And followup (remove shortcut support)
- I73b7dc008a5acaf75a31a36a2d332b5afabd82d0
Bug: 28255684
Test: make -j10 dicttool_aosp
Change-Id: I2e01ed86b9517a1141aee35ea6d8ef39258981d1
|
|
|
|
|
|
|
|
|
|
| |
This builds on Darwin fine, so remove the non-standard darwin checks.
Remove HOST_JDK_IS_64BIT_VERSION, it's always true now, and this is the
only user.
Test: mmm packages/inputmethods/LatinIME/tools/dicttool:liblatinime-aosp-dicttool-host
Change-Id: I750c6baa29cf1d27a885ff064f0daf9d986db0b9
|
|
|
|
|
|
|
|
|
|
|
|
| |
Use framework/junit-host.jar instead of framework/junit.jar
Ran the script before and after making the change and it failed
in the same way with a build error but it was not due to a
problem with JUnit.
Bug: 30188076
Test: make checkbuild and ran the test-dicttool.sh script
Change-Id: I4cdd518f38184ff3ade5a06d968d6902770ece6b
|
|
|
|
|
|
| |
Bug: http://b/32019064
Test: builds
Change-Id: I17611b6d9598992bd52cf3dc93509f0f5e0bdcde
|
|
|
|
| |
Change-Id: I256d1c6bf96c07b10d2d063d7935e20e0ab8ea17
|
|
|
|
| |
Change-Id: I50a286c115f5bd6e93763bd2f79031676d6fffd8
|
|
|
|
| |
Change-Id: I738735186213b3a40eff997ae2fd83069c6445f1
|
|\ |
|
| |
| |
| |
| | |
Change-Id: I4198f6b463711feb4ab78020934cca4d23870fbb
|
| |
| |
| |
| |
| |
| |
| | |
Also why did we have two copies of LocaleUtils >.>
Bug: 18108776
Change-Id: I03b4403dfd51934e66b567f2f8b87da419cfb3ab
|
|\ \
| |/
|/| |
|
| |
| |
| |
| |
| | |
Bug: 18108776
Change-Id: I0f57594617f8d5080aea9002b2356d18bfe97043
|
| |
| |
| |
| |
| |
| | |
Also fix a failing test
Change-Id: I280aa6b5836c7332da08a88f1258546edeb25a37
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This will allow to greatly improve the performance of the
metadata-generating files, as they won't have to wait for
the info command to read the entire dictionary when the
header is all we need.
Also add tests, and while we're at it, use the seed as
intended to enable reproducible tests.
Change-Id: I0ba79ef62f0292b23e63aed57ff565bb102281a2
|
|/
|
|
|
|
|
|
|
|
|
| |
It's still unused as of this change but the next change will use it
As a reference point, generating the metadata for Bayo takes
3'02" on my machine with the info command; it's down to 16" if
made to use this instead. The gains increases with the number
of dictionaries obviously.
Change-Id: I0eeea2d8f81bb74b0d1570af658e91b56f7c2b79
|
|
|
|
|
|
|
| |
This will allow for not copying the whole dictionary when only
the header is needed.
Change-Id: Ie4a649b507ccd4a430201824ed87b8b8bbf55e9f
|
|
|
|
|
|
|
|
|
|
|
| |
That is where the last refactorings were leading. This code is
simpler, but it's far more flexible. Importantly, it only makes
a single copy instead of making a full disk copy for every
intermediate step.
Next we're going to make the "copy" part modular for processes
that don't need to copy the whole file.
Change-Id: Ief32ac665d804b9b20c44f443a9c87452ceb367a
|
|\ |
|
| |
| |
| |
| |
| |
| | |
Bug: 10059681
Change-Id: Ib730af8ebc944e08aaada869c0626724a499747c
|
| |
| |
| |
| |
| | |
Bug: 18108776
Change-Id: Ic81c5df2a685160c249021ff4cfe41553dbdeebe
|
|\ \ |
|
| | |
| | |
| | |
| | |
| | | |
Bug: 18108776
Change-Id: I9b399a44241e05a7add9bb8094263aa76de37880
|
|/ /
| |
| |
| |
| | |
Bug: 11031090
Change-Id: I5cc0d006ab003656498eb82b0875eb9c051d331e
|
| |
| |
| |
| |
| | |
Bug: 18108776
Change-Id: Ie5c0430aa41a8d6c58bf510a10173223d7dfe9fe
|
|/
|
|
|
| |
Bug: 18108776
Change-Id: Ia46a4102a0e86e71118ca5e641f9f531998e166b
|
|
|
|
| |
Change-Id: I538288054a58eb2c81ce3cbe5c9bef900fb653a5
|
|\ |
|
| |
| |
| |
| | |
Change-Id: I23fa4e4ed96228406e70aa94d84fd7b8d3f69347
|
|\| |
|
| |
| |
| |
| |
| |
| |
| | |
This is a preliminary refactoring change to improve performance
in dicttool diagnostic tools.
Change-Id: I9a59328af62e336809246be5bebbbf2e154366b3
|
|\ \ |
|
| | |
| | |
| | |
| | | |
Change-Id: Idc4533e57b7f4ddccd66b1f1a4730153064b1e27
|
|/ /
| |
| |
| |
| | |
Bug: 18003991
Change-Id: Id6b67bf66b397301e5186826dba2b60df9cb4c65
|
|/
|
|
|
|
| |
Move Constants.java too.
Change-Id: I55e8cf06f61c0a37746a7328834a52a5306cf0f2
|
|
|
|
|
|
|
| |
This CL also adds @SuppressWarning("unused" to java-overridable package.
Bug: 18003991
Change-Id: If70527e30654384705d7a814f5efd181d9f539e1
|
|
|
|
|
|
|
| |
This is just a placeholder in this CL. It will be shared by LatinIME,
dicttool, and tests.
Change-Id: I6fb3516a5061555f6f24b29141c2871d2319b023
|
|
|
|
|
|
| |
This hasn't been used for a while. It's deprecated. Let's kill it.
Change-Id: Ib1c491fa14b6406f6f77f2b0869f4db1810eb078
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL fixes the following compiler warnings.
- Indirect access to static member
- Access to a non-accessible member of an enclosing type
- Parameter assignment
- Method can be static
- Local variable declaration hides another field or variable
- Value of local variable is not used
- Unused import
- Unused private member
- Unnecessary 'else' statement
- Unnecessary declaration of throw exception
- Redundant type arguments
- Missing '@Override' annotation
- Unused '@SuppressWarning' annotations
Bug: 18003991
Change-Id: Icfebe753e53a2cc621848f769d6a3d7ce501ebc7
|
|
|
|
|
|
|
|
|
| |
No behaviour changes.
Unified the overloaded FusionDictionary::add method to always take an
isPossiblyOffensive argument.
Bug: 11031090
Change-Id: I5741a023ca1ce842d2cf10d4f6c926b0efabaa78
|
|
|
|
|
|
|
| |
I've never seen the NPE happen and only happened to notice
this by chance. Let's fix the code.
Change-Id: If458646229f9cadcd6c15779348133f382fde783
|
|
|
|
| |
Change-Id: I89eb530c1a44e5ea9b5fcb37c792cb8c81d6dc1c
|
|\ |
|
| |
| |
| |
| |
| | |
Bug:17097992
Change-Id: I677a5eb3a704e4386f6573360e44ca335d81d2df
|