diff options
| author | Christopher Ferris <cferris@google.com> | 2019-01-08 14:58:07 -0800 |
|---|---|---|
| committer | Christopher Ferris <cferris@google.com> | 2019-01-10 15:15:15 -0800 |
| commit | 658b16fd9078882730861cc2c29cb44828989e52 (patch) | |
| tree | bff861d3acba602010b5f3043a605f34ad13e1d6 /libc/malloc_hooks/malloc_hooks.cpp | |
| parent | e7964158cedf37b50e42ab1988e67eb2154c74f2 (diff) | |
Rewrite removeVarsAndFuncs.
The current version has these bugs:
- Adding a semicolon after a function results in the removal of structures
following the function.
- Function like macros get removed on accident rather than on purpose.
- It removes extern "C" { completely, which might not be a bug, but doesn't
seem right.
I couldn't easily fix any of these problems because the code depends heavily
on the header being correct.
New unit tests added for the function to cover all of these cases.
A follow-on CL will include the updated headers.
Bug: 112290385
Test: Passes all new unit tests.
Test: When run on the current kernel headers, the generated headers are
Test: nearly the same, missing data is being added.
Change-Id: Ib22a5f2e78873544e8a9d54e385af1156b2a72bb
Diffstat (limited to 'libc/malloc_hooks/malloc_hooks.cpp')
0 files changed, 0 insertions, 0 deletions