| 054afbd3037addb4f2e245ecbc36f31d3040f877 |
|
17-Oct-2015 |
Christopher Wiley <wiley@google.com> |
Change C++ output to match make expectations
- generate a single C++ output file
- put headers into an output directory
- create the output directory structure using an IoDelegate
Bug: 23599697
Test: Unittests pass
Change-Id: I0f7b821678cd5adc00b2fdba06b52b235c724467
/system/tools/aidl/options_unittest.cpp
|
| 4432ccfb09ae2c28e42b9c4c58f0e2c2c5a3ba7a |
|
19-Sep-2015 |
Christopher Wiley <wiley@google.com> |
Calculate C++ output file names from commandline args
Test: unittests
Bug: 24220933
Change-Id: I59c32d7a27241776181c0f707082b5433b7b7005
/system/tools/aidl/options_unittest.cpp
|
| a590de8d148bb42aa5f1f32aec6faae18a28e720 |
|
16-Sep-2015 |
Christopher Wiley <wiley@google.com> |
Define aidl-cpp executable
This executable has a distinct usage which reflects that it generates
a very different sort of code.
Bug: 23599341
Test: unittests pass, added new Options unit test
Change-Id: I6750e7146949659a3af0246ad1bca0fa23afb2d5
/system/tools/aidl/options_unittest.cpp
|
| 89eaab56f3ff913c247397b858fac3c8d3d4a206 |
|
15-Sep-2015 |
Christopher Wiley <wiley@google.com> |
Rename Options to JavaOptions
The aidl command line interface exposes a lot of knobs that
are not used in practice and complicate the internal logic to support.
For C++ generation we have an opportunity to reduce the exposed surface.
Begin by renaming Options to a more specific JavaOptions.
Test: unittests pass
Bug: 23599341
Change-Id: I76478ab2fb12612bf7120a1af28c8db1da9ed22f
/system/tools/aidl/options_unittest.cpp
|
| d93c5b761bf5a53b23a1f0ab954fa38109e72043 |
|
14-Sep-2015 |
Christopher Wiley <wiley@google.com> |
Clean up options parsing logic a little bit
We're about to add support for asking for C++ output.
Make this a little less tedious and flesh out the unittests a little.
Bug: 23599341
Test: unittests
Change-Id: Ie08414db43b48ae1a90640260b0badf4c4eb60f8
/system/tools/aidl/options_unittest.cpp
|
| 8f8cc9b91a8d0b4942a978e9d2edc5337503d583 |
|
14-Sep-2015 |
Christopher Wiley <wiley@google.com> |
Follow style guide for Options field naming
Bug: 23599341
Test: unittests
Change-Id: I113810186e67efe7ef657195664e683ab55e829f
/system/tools/aidl/options_unittest.cpp
|
| 4427d8654985209c1ebf2fb4705bb3e1a4262be2 |
|
14-Sep-2015 |
Christopher Wiley <wiley@google.com> |
Basic cleanup of parsed options
1) Turn it into a proper C++ object
2) Don't leak Options objects
3) Don't mutate Options objects
4) clang-format -style=Google -i options.cpp
Bug: 23599341
Test: unittests
Change-Id: I06597dd21e94ff391a08ea1bdb8f3a526123b3dc
/system/tools/aidl/options_unittest.cpp
|
| 3616d138056b61bd151820b9799820b70d797061 |
|
01-Sep-2015 |
Christopher Wiley <wiley@google.com> |
aidl: Separate aidl logic into its own static library
Compile aidl's logic into a static library that can be
linked against both the unittest binary and the executable. Validate
that this is working by writing a basic unittest against options
parsing.
Provide default values for Options struct members. These were
previously taking on values according to the the bits in the memory
allocated to hold them for preprocessed inputs.
Bug: 23516947
Change-Id: Ib57cd7f473995f7851d788deac39f9c4d9018e8f
Test: compiles and unittests pass
/system/tools/aidl/options_unittest.cpp
|