951fb131787a56b818a5e1058f38cc4110d582da |
|
07-Nov-2017 |
Steven Moreland <smoreland@google.com> |
Move preload test to libhidl_test. Since hidl_test is completely static now, moving the DISABLED preload test so that we can actually test the API here. Test: libhidl_test Change-Id: I1e34ddc959d761130cf5b2c0a30fa2e1ec675d99
/system/libhidl/test_main.cpp
|
aa661fc957c8d3acbc0abd409f5e4a16a65ce3b1 |
|
06-Oct-2017 |
Steven Moreland <smoreland@google.com> |
Remove flaky test and clarify documentation. Resize doesn't set primitive types to zero. Test: hidl's run_all_device_tests.sh Change-Id: I0b7031bc563e3201815ac238ea5a375602393773
/system/libhidl/test_main.cpp
|
96e9de049f4dd2ee7d02eaf9a662e286964c2039 |
|
29-Sep-2017 |
Steven Moreland <smoreland@google.com> |
Add size constructor to hidl_vec. This pattern is everywhere: hidl_vec<...> x; x.resize(3); It doesn't look very pretty. Test: libhidl_test Change-Id: I2776e81afbe6f12e34cfb31a2f40d539554f0cbc
/system/libhidl/test_main.cpp
|
41649d5bb219a3b316eae8af54673f54e0a75935 |
|
03-Aug-2017 |
Chih-Hung Hsieh <chh@google.com> |
Fix clang-tidy performance-* warnings in system/libhidl. * Use const reference parameter type to avoid unnecessary copy. * Use more efficient overloaded string methods. * Use const reference type for loop index variables to avoid unnecessary copy. Bug: 30407689 Bug: 30411878 Bug: 30413223 Test: build with WITH_TIDY=1 Test: libhidl_test Change-Id: I1deae1f09d56c2fa9cd56728b13ca50305d18f3e
/system/libhidl/test_main.cpp
|
86fd2e14d4f96322df0bb26b5dffafb3a6356c4c |
|
02-Aug-2017 |
Steven Moreland <smoreland@google.com> |
libhidl_test: remove flake We have many other performance tests for passthrough mode. I've changed the TaskRunner test here to only test that the task is run. Bug: 64020862 Test: libhidl_test (100 times) Change-Id: I045718a4f8cfbd08b3d88a4f7c4232e2b54ea869
/system/libhidl/test_main.cpp
|
a9f60730682d198419a310ca19ecb5d4fc9f2fd6 |
|
30-Jun-2017 |
Tomasz Wasilczyk <twasilczyk@google.com> |
Add range constructor to hidl_vec. Bug: b/36864090 Test: Unit test Test: a device boots Test: marlin boots and completes unit test Change-Id: I7d37f75ec6c8ba0252c3c60afd515eff87dfc4dd
/system/libhidl/test_main.cpp
|
4917296debaeb74f48ddac5ce4d8f5e65020ee69 |
|
29-Apr-2017 |
Steven Moreland <smoreland@google.com> |
libhidl: add withDefault Shorthand way to provide default values for HIDL calls. Fixes: 37088970 Test: libhidl_test Change-Id: If91cfb774fa76314aeec334fbc2b18ea790912c4
/system/libhidl/test_main.cpp
|
eb0c337c4fdfe35b12039036555c2760a7c60884 |
|
03-Apr-2017 |
Scott Randolph <randolphs@google.com> |
Implicit const char* cast removed, operator<< added to hidl_string. hidl_string no longer will provide an implicit cast to const char* as it interfers with other expected behaviors of the class. It now emulated std::string in requiring a call to .c_str() to get the same behavior. Bug: 36532780 Test: Build the tree for marlin, bullhead, bat (cherry picked from commit 0c84ab4192f9b0b03b9840b85fc430afa17f55f1) Merged-In: I6ba76522ef65aa211bc156144990ad8b7495b051 Change-Id: I6ba76522ef65aa211bc156144990ad8b7495b051
/system/libhidl/test_main.cpp
|
727e3dd867a760aa7c62b5abec4d8184c47035e3 |
|
23-Mar-2017 |
Steven Moreland <smoreland@google.com> |
std::string from const hidl_string test Bug: 36532780 Test: libhidl_test passes Change-Id: I22cb52fa9d441e92f6c4dfdcc8d0d0657bf0c0f5
/system/libhidl/test_main.cpp
|
6f6675464c82fc796d72458d7268e28d759b997f |
|
21-Mar-2017 |
Yifan Hong <elsk@google.com> |
TaskRunner starts the background thread only when needed Test: libhidl_test Test: hidl_test Test: boots Change-Id: I5b6de5cd4a475b0653102e8004953ba008288ab1
/system/libhidl/test_main.cpp
|
0a35139d24ae0071475adeae3e1370b8f8617f8f |
|
21-Mar-2017 |
Yifan Hong <elsk@google.com> |
Move TaskRunner and SynchronizedQueue to details Bug: 36433220 Test: compiles Test: hidl_test Test: libhidl_test Change-Id: Ia48bf9abba293f177ec2f6cfab49da5422a59b82
/system/libhidl/test_main.cpp
|
72db40f4a612d04187ae2fc1f9a3766933264bdf |
|
10-Mar-2017 |
Steven Moreland <smoreland@google.com> |
Status.h: remove service specific error codes. These are leftover from binder and are never used in HIDL/hwbinder. Test: libhidl_test + hidl_test + internal marlin boots/works. Bug: 36099713 Change-Id: I12b20e1ae54d9df802a961bc9bc99f616291b0fd
/system/libhidl/test_main.cpp
|
2be9418963c8d36bb1425093d224256e2a94b277 |
|
04-Mar-2017 |
Yifan Hong <elsk@google.com> |
Return<T>::description() provide more descriptive text than numbers Test: libhidl_test Change-Id: Ib7c591f261bb335b9fe348dd2fb0695a94bdf468
/system/libhidl/test_main.cpp
|
af4e43cc5b2eacbfe600f9ab0e459f09c35c0c3f |
|
04-Mar-2017 |
Yifan Hong <elsk@google.com> |
return_status::checkStatus => assertOk checkStatus doesn't really set mCheckStatus to true, and hence is a misnomer. Change it to assertOk (correspond to isOk()). And hence the move assignment operator should not call checkStatus (because it is valid to move into a Return<T> object that !isOk() but has mCheckedStatus to true), because caller is fully aware of the error that will be overwritten. Test: libhidl_test Change-Id: I86a44967b68619d1467d2cc9caaa39124b156121
/system/libhidl/test_main.cpp
|
153f87a2fd2972f862a88976bb884a90a1f9fd7d |
|
28-Feb-2017 |
Steven Moreland <smoreland@google.com> |
Make operator= and constructor behavior consistent. hidl_string(nullptr) == "". Now so does operator=(nullptr). Bug: 35421240 Test: libhidl_test Change-Id: Ib62fa71ab47a407b7017c809783dd5f421b98e39
/system/libhidl/test_main.cpp
|
a21d84f11f066315bd1476368d511286d129cb8a |
|
16-Feb-2017 |
Steven Moreland <smoreland@google.com> |
hidl_string: allow null cstr Automatically converting nullptr to empty string rather than erroring. This gets rid of null values at the hal layer (rather than passing them to the server) and simplifies code that would have to check for nulls. Test: libhidl_test Fixes: 35421240 Change-Id: Iada25da5407c3eccdd28704d993ffdee8b15c5fe
/system/libhidl/test_main.cpp
|
551396afe50bb40b046e9f16b4501e6df055c88a |
|
14-Feb-2017 |
Steven Moreland <smoreland@google.com> |
hidl_string: relational operators Fixes: 34524287 Test: libhidl_test Change-Id: Id0731e6df1fc1831d8c1bdf35b3aa17abd5273d2
/system/libhidl/test_main.cpp
|
a2a8184180557e980387f769ac3915eb9e072f8b |
|
20-Jan-2017 |
Steven Moreland <smoreland@google.com> |
hidl_string: add operator< This allows us to std::sort vecs of hidl_strings, etc.. Test: libhidl_test Change-Id: I6e9a7669bceb09d62791fe8c88cfd1977ea07194
/system/libhidl/test_main.cpp
|
53120f70b7d429c81fe47718182e829660ed5ef9 |
|
12-Jan-2017 |
Steven Moreland <smoreland@google.com> |
hidl_string: Add cstr + length constructor This will allow non-terminated strings to be turned into a hidl_string with only one copy among other things. Test: libhidl_test Fixes: 34221748 Change-Id: I443f7920a65d23b7bcb583db55c3cec7266ec2d4
/system/libhidl/test_main.cpp
|
0f3461db91b75d9acd7e0bec8b1e97cbb3bafdf7 |
|
03-Jan-2017 |
Jeff Tinker <jtinker@google.com> |
Permit assignment from a null hidl_memory Bug: 34102220 Test: libhidl_test, hidl_test Change-Id: Ic0da09f0f6a79643ee44c6b13ef606b2217956e0
/system/libhidl/test_main.cpp
|
9fcbb36250a4392b12012cf5177c4669ef10e654 |
|
21-Dec-2016 |
Yifan Hong <elsk@google.com> |
operator== and != for hidl_vec. Test: libhidl_test Bug: 32834072 Change-Id: Iae9b454f5c1e9d8b3b10394ef6de9d4ba4cdd959
/system/libhidl/test_main.cpp
|
19f4db58674560ebe6b85dad77af0155a3dddecf |
|
15-Dec-2016 |
Eino-Ville Talvala <etalvala@google.com> |
hidl_version: Add comparison operators Test: libhidl_test passes Change-Id: I8bb01599ae581484d80014bd99c4aff09a953824
/system/libhidl/test_main.cpp
|
64fdf4d1bdea86202c7526482d4d329724d33dcb |
|
10-Dec-2016 |
Yifan Hong <elsk@google.com> |
fix and expand hidl_vec::iterator; add const_iterator. * Fix flipped post++ and ++pre for hidl_vec::iterator * make it random accessible. * add const_iterator. Test: hidl_test Change-Id: I8649f8598a13e6054c994d30a4daf383d1dae55c
/system/libhidl/test_main.cpp
|
44ab623ccf386f0e470bdc6aa22f261b7dc047cd |
|
23-Nov-2016 |
Yifan Hong <elsk@google.com> |
Convert (by copying) hidl_array from and to std::array. * For example, hidl_array<T, 2, 3> can be converted to std::array<std::array<T, 3>, 2>. * Uses operator= for copying. * Also uses operator= when initializing an hidl_array (instead of memcpy'ing). This fixes potential memory issues for hidl_array<hidl_string>, for example. Bug: 32883329 Test: libhidl_test Change-Id: Idf7d080433aaed2c585fd4875f3411e5544a9e72
/system/libhidl/test_main.cpp
|
bb840f73dc33c5de153cc6959672cf05c55ae4e6 |
|
21-Nov-2016 |
Scott Randolph <randolphs@google.com> |
Add hidl_vec iterator and hidl_string == Add bidirectional iterator for hidl_vec Add operator== and operator!= for hidl_string to hidl_string comparision. Test: Added unit tests to libhidl_test Change-Id: I908bc71fae4016f335e2342f28b351f63f22fa66
/system/libhidl/test_main.cpp
|
3da68488e12ae8d487f43700582b775a4cdb6ec1 |
|
18-Nov-2016 |
Sasha Levitskiy <sanek@google.com> |
Libhidl: Add a constructor to hidl_array. Test: Added a test to libhidl native test suite. Change-Id: I4469200a40b4c05184f39754db6b4e732ae85ccd Signed-off-by: Sasha Levitskiy <sanek@google.com>
/system/libhidl/test_main.cpp
|
00f4a391c6f9a698bbcbbd05bf3d80e213c82884 |
|
16-Nov-2016 |
Yifan Hong <elsk@google.com> |
Split libhidl into base and transport. libhidlbase contains "pure" classes that has nothing to do with tranport level stuff. libhwbinder can now safely depend on libhidlbase. IBinder inheriting from IBase will be in a follow up CL. Bug: 32756130 Test: mma under hardware/interfaces Test: hidl_test Change-Id: Ie2bd80769df3816aab83789ba0d7fca9b5a2e52f
/system/libhidl/test_main.cpp
|