diff options
| author | Greg Hackmann <ghackmann@google.com> | 2017-02-14 16:42:44 -0800 |
|---|---|---|
| committer | Greg Hackmann <ghackmann@google.com> | 2017-02-22 21:18:48 +0000 |
| commit | dc80973726d709371d70f4cc85a12c8f18bf2490 (patch) | |
| tree | 76230c4b328212e2df79cc37b12ea07980080d16 /adb/adb_utils.cpp | |
| parent | 52ae36ed57902508b1440c7ff3570c34e2cf154c (diff) | |
| download | system_core-dc80973726d709371d70f4cc85a12c8f18bf2490.tar.gz system_core-dc80973726d709371d70f4cc85a12c8f18bf2490.tar.bz2 system_core-dc80973726d709371d70f4cc85a12c8f18bf2490.zip | |
libadf: convert to C++
Both humans and the clang static analyzer find libadf's error handling
confusing.
Now that the platform has better C++ support, we can clean up things up
by switching to C++ and using STL + RAII in targeted parts of the code.
This isn't a complete rewrite in idiomatic C++, but it's enough to get
rid of all the "goto" statements (and the false-positive memory leaks
found by clang's static analyzer).
Bug: 27125399
Test: WITH_STATIC_ANALYZER=1 mmm system/core/adf/libadf
Test: /data/nativetest/adf-unit-tests/adf-unit-tests (on Nexus 9)
Test: /data/nativetest64/adf-unit-tests/adf-unit-tests (on Nexus 9)
Change-Id: Ie9dd5d5dc424d1a3ddcc3cba836fce04190f46fd
Signed-off-by: Greg Hackmann <ghackmann@google.com>
Diffstat (limited to 'adb/adb_utils.cpp')
0 files changed, 0 insertions, 0 deletions