diff options
author | Steven Moreland | 2018-09-06 19:58:16 -0500 |
---|---|---|
committer | Steven Moreland | 2018-09-09 14:31:05 -0500 |
commit | 146405b58a876d1bb50e0520409cff71f376cd84 (patch) | |
tree | 634115c3ac27a87779e54901cc8b8f3e9d6acf9c /vibrator | |
parent | 0a0fa7209e8079a20bbac432d2073be672ea5b31 (diff) | |
download | platform-hardware-interfaces-146405b58a876d1bb50e0520409cff71f376cd84.tar.gz platform-hardware-interfaces-146405b58a876d1bb50e0520409cff71f376cd84.tar.xz platform-hardware-interfaces-146405b58a876d1bb50e0520409cff71f376cd84.zip |
Note confusion-causing breaks in HIDL convention.
These were either made at a time when the convention was not fully
formed or missed during review.
It is somewhat misleading since method overloading isn't supported
and method names need this kind of prefix, but nothing else does.
The reason for this is that everything is namespaced, but methods are
all in the same namespace. The reason method overloading is not supported
in HIDL is because the HIDL types may map to types that collide in the
target languages, and this would cause any sort of overloading rules
to require complicated machinery.
Bug: N/A
Test: hidl-gen -Lcheck android.hardware.usb@1.1 android.hardware.vibrator@1.1 && echo ":)"
:)
Change-Id: Iac23c9311925ed140ff1e15d1366829b078c8866
Diffstat (limited to 'vibrator')
-rw-r--r-- | vibrator/1.1/types.hal | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/vibrator/1.1/types.hal b/vibrator/1.1/types.hal index f7a619a0..72deb4a7 100644 --- a/vibrator/1.1/types.hal +++ b/vibrator/1.1/types.hal | |||
@@ -18,6 +18,8 @@ package android.hardware.vibrator@1.1; | |||
18 | 18 | ||
19 | import @1.0::Effect; | 19 | import @1.0::Effect; |
20 | 20 | ||
21 | // NOTE: suffix '_1_1' is for legacy ABI compatibility. It cannot be | ||
22 | // changed to 'Effect' which the convention dictates. | ||
21 | @export | 23 | @export |
22 | enum Effect_1_1 : @1.0::Effect { | 24 | enum Effect_1_1 : @1.0::Effect { |
23 | /** | 25 | /** |