Fix errorprone warnings that should be errors

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: I0584059c6195f18e8a602ae5924ae4d76373535d
2 files changed
tree: b87071c3bbe29ded20fa7539c557bee41dccd8ce
  1. build/
  2. c2hal/
  3. hashing/
  4. hidl2aidl/
  5. host_utils/
  6. lint/
  7. metadata/
  8. scripts/
  9. test/
  10. utils/
  11. Android.bp
  12. Annotation.cpp
  13. Annotation.h
  14. ArrayType.cpp
  15. ArrayType.h
  16. AST.cpp
  17. AST.h
  18. CleanSpec.mk
  19. CompoundType.cpp
  20. CompoundType.h
  21. ConstantExpression.cpp
  22. ConstantExpression.h
  23. Coordinator.cpp
  24. Coordinator.h
  25. DeathRecipientType.cpp
  26. DeathRecipientType.h
  27. DocComment.cpp
  28. DocComment.h
  29. EnumType.cpp
  30. EnumType.h
  31. FmqType.cpp
  32. FmqType.h
  33. FormattingConstants.h
  34. generateCpp.cpp
  35. generateCppImpl.cpp
  36. generateDependencies.cpp
  37. generateFormattedHidl.cpp
  38. generateInheritanceHierarchy.cpp
  39. generateJava.cpp
  40. generateJavaImpl.cpp
  41. generateVts.cpp
  42. HandleType.cpp
  43. HandleType.h
  44. hidl-gen_l.h
  45. hidl-gen_l.ll
  46. hidl-gen_y-helpers.h
  47. hidl-gen_y.yy
  48. HidlTypeAssertion.cpp
  49. HidlTypeAssertion.h
  50. Interface.cpp
  51. Interface.h
  52. Location.cpp
  53. Location.h
  54. main.cpp
  55. MemoryType.cpp
  56. MemoryType.h
  57. Method.cpp
  58. Method.h
  59. NamedType.cpp
  60. NamedType.h
  61. NOTICE
  62. OWNERS
  63. PointerType.cpp
  64. PointerType.h
  65. PREUPLOAD.cfg
  66. README.md
  67. Reference.h
  68. ScalarType.cpp
  69. ScalarType.h
  70. Scope.cpp
  71. Scope.h
  72. StringType.cpp
  73. StringType.h
  74. TEST_MAPPING
  75. Type.cpp
  76. Type.h
  77. TypeDef.cpp
  78. TypeDef.h
  79. update-all-google-makefiles.sh
  80. update-makefiles-helper.sh
  81. VectorType.cpp
  82. VectorType.h
README.md

hidl-gen

Full documentation can be found here: https://source.android.com/devices/architecture/hidl/

hidl-gen is a compiler for the HIDL (HAL Interface Design Language) which generates C++ and Java endpoints for RPC mechanisms. The main userspace libraries which this compiler uses can be found at system/libhidl.

1. Build

m hidl-gen

2. Run

Note that options for hidl-gen expected to be invoked by the build system are marked with ‘internal’ in the help menu.

hidl-gen -h

hidl-gen -o output -L c++-impl -r android.hardware:hardware/interfaces -r android.hidl:system/libhidl/transport android.hardware.nfc@1.0

Some defaults for package roots are also provided

hidl-gen -o output -L c++-impl android.hardware.nfc@1.0
hidl-gen -o output -L vts android.hardware.nfc@1.0
hidl-gen -L hash android.hardware.nfc@1.0

Example command for vendor project

hidl-gen -L c++-impl -r vendor.foo:vendor/foo/interfaces vendor.foo.nfc@1.0

See update-makefiles-helper.sh and update-all-google-makefiles.sh for examples of how to generate HIDL makefiles (using the -Landroidbp option).

NOTE: When using the -Landroidbp option, you can force generated modules to be installed in /system_ext rather than other partition by putting a marker file .hidl_for_system_ext alongside *.hal files.

NOTE: You can also install the vendor variant of the generated modules to be installed in /odm rather than /vendor by putting a marker file .hidl_for_odm alongside *.hal files.

c2hal

This is a helper tool to convert C headers to valid .hal files.

m c2hal && c2hal -h