| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
| |
Bug: 24410295
Change-Id: I28b70254c8e1846eaeab8523da5975843e86e0de
Test: Full android build
Signed-off-by: Casey Dahlin <sadmac@google.com>
|
|
|
|
|
|
|
| |
Change-Id: Ib367a9183e734a008acecf90c5f3f25e51e16b37
Test: Full android build
Bug: 24410295
Signed-off-by: Casey Dahlin <sadmac@google.com>
|
|
|
|
|
|
|
|
|
|
| |
This class delegates to an IoDelegate for file I/O.
This replaces the search_path.* files.
Bug: 24703046
Test: Unittests pass
Change-Id: I261f6066cef514926b45174988863b745a37e87c
|
|
|
|
|
|
|
|
|
| |
We also take this opportunity to neaten up the lexer code.
Change-Id: I0220f827144be768bcf771e240125e805c93d670
Test: Full android build
Bug: 24680682
Signed-off-by: Casey Dahlin <sadmac@google.com>
|
|
|
|
|
|
|
| |
Change-Id: I223caa8358e4f7f0554eaa9d198eb6366c141ce1
Test: Unit tests and full android rebuild
Bug: 24680682
Signed-off-by: Casey Dahlin <sadmac@google.com>
|
|
|
|
|
|
|
| |
Change-Id: Ib93032113c92fe23e1a7e4ac9b990a065f2b5d19
Test: Unit tests, Full Android Rebuild
Bugs: 24680682
Signed-off-by: Casey Dahlin <sadmac@google.com>
|
|
|
|
|
|
|
| |
Change-Id: I453ab9bcbbc610df4109d691283e7b872e0078a4
Test: Unit tests
Bug: 24410295
Signed-off-by: Casey Dahlin <sadmac@google.com>
|
|
|
|
|
|
|
|
|
|
|
| |
As usual, we're leaving most of the same members in place and public. However,
we have removed the linked list pointer and switched to std::vector for lists
of methods.
Change-Id: Id0aa55340e2c56bfa1001c6c8a8343bb4815d38e
Test: unit tests
Bug: 24410295
Signed-off-by: Casey Dahlin <sadmac@google.com>
|
|
|
|
|
|
|
|
|
|
| |
This actually simplifies things a great deal, and means we don't need
buffer types back from the lexer for these three tokens. Small victories.
Change-Id: I5de6ef01852012ac60dd91238ba013662089ed0e
Test: unit tests
Bug: 24410295
Signed-off-by: Casey Dahlin <sadmac@google.com>
|
|
|
|
|
|
|
|
|
| |
This reverts commit b2116869ba878222d607807be594366b6e9392f7.
Test: unittests
Bug: 24469748
Change-Id: I82de5a12cf851ffdacc2c7969a2429746ba7a772
Signed-off-by: Casey Dahlin <sadmac@google.com>
|
|
|
|
|
|
|
|
| |
This reverts commit b992b450093546d275d2da4b544a611700536e29.
The build's address sanitizer is complaining that we're calling delete[] on a 3 byte region allocated with malloc().
Change-Id: I03e667e1b64a9b298607d0709747c02c8f7d0da6
|
|
|
|
|
|
|
| |
Change-Id: Iee22541081caf78e295f7a05e248bfdee0bc8770
Test: unittests
Bug: 24469748
Signed-off-by: Casey Dahlin <sadmac@google.com>
|
|
|
|
|
|
|
|
|
|
| |
1) Move some helper functions to parse_helpers.cpp
2) Refactor some helper functions to take advantage of C++ built ins
3) Add unittest coverage of refactored logic
4) clang-format -i -style=Google parse_helpers.cpp
5) Move remaining helper functions of aidl.cpp into anonymous namespace
Change-Id: I3a2cf7be113ac7f71a1a3502b12a8ed910509546
|
|
|
|
|
|
|
|
|
|
|
| |
No more callbacks. No more unsafe pointer cast (fixes Bug 23977313).
The best part is that the entire YACC/FLEX portion is now behind an object
from the rest of the program's POV.
Test: Ran unit tests
Change-Id: I9e38953bd0e65b8f1fad87ff2df049643ef6cc73
Signed-off-by: Casey Dahlin <sadmac@google.com>
|
|
|
|
|
|
|
|
|
|
|
| |
This might introduce some minor changes in the grammar, as we have to change
the way we lex whitespace in order to count lines, and the result is linebreaks
aren't valid everywhere they used to be. But the whole android code base
builds, and in coming changes we'll be able to put some of that back (we'll be
reducing complexity in the lexer and moving that in to the grammar).
Change-Id: I6a688b2dc45e6ea4838af466d2cca5e1b26f916c
Signed-off-by: Casey Dahlin <sadmac@google.com>
|
|
|
|
|
| |
Change-Id: I5481fef962e70263dc791339c0493407d3dabaf1
Signed-off-by: Casey Dahlin <sadmac@google.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This allows us to write unittests of Java/Cpp AST elements without
doing file I/O each time. Introduce a CodeWriter class which
may write to either a string or a file dependening on the contect.
While here, mark the write methods as const.
Test: Compiles, unittests pass, added new unittest demonstrating usage.
Bug: 23598995
Change-Id: I240ede41f0f9c04c0d14d1a3680ce5ab332babbc
|
|
|
|
|
|
|
|
| |
This is attempting to fix build breakage caused by redefining the isatty stub
for win32.
Change-Id: I5f854934cb9d2b2981f6349045e85d9ad737aa77
Signed-off-by: Casey Dahlin <sadmac@google.com>
|
|
|
|
|
|
|
|
|
|
| |
We have to step up to a GLR parser to do this without exceptions (for no
reason other than Bison happens to use exceptions for the LALR(1) template and
not for the GLR one), but this should let us smooth out integration going
forward.
Change-Id: Iff44662914b4a65dfa5612d07c3a1ede07e6e4a9
Signed-off-by: Casey Dahlin <sadmac@google.com>
|
|
|
|
|
|
|
|
|
|
|
| |
We're flipping Flex/Bison in to reentrant mode, cutting down on global
variables, and exposing a more C++-like interface earlier. This is the first
phase. There's still a couple of weird hacks to deal with the previous code's
reckless disregard for scope sanity, but the outline of things to come is
starting to appear, and this CL really doesn't need to get any bigger.
Change-Id: Ife2d70db026d7ab1319bdf6c586959315666d0bb
Signed-off-by: Casey Dahlin <sadmac@google.com>
|
|
|
|
|
|
|
|
|
|
|
| |
AIDL previously supported "rpc" and "flatten" types, which are not used
anywhere in the Android tree. This patch removes them, which significantly cuts
down code and complexity.
Bug: 23517584
Test: Rebuilt AOSP tree against new version
Change-Id: I6cf38d43c1ce109ffca987cc14520945aa22431f
Signed-off-by: Casey Dahlin <sadmac@google.com>
|
|
|
|
| |
This reverts commit 9f6a119c8aa276432ece4fe2118bd8a3c9b1067e.
|
|
|
|
| |
Change-Id: I3ffafdab27cc4aca256c3a5806b630795b75d5c8
|
|
|
|
|
|
|
|
|
|
|
|
| |
This adds an annotation to methods in AIDL of the form
"void myMethod() = 3;" to explicitly set the onTransact
id for the method. Either all methods must have explicitly
annotated id's or none of them should be explicitly annotated.
There is error checking in the AIDL compiler
for duplicate id's and id's outside of the valid range.
Bug: 7353910
Change-Id: I868045e3f112c9a279c573cea368a621116cbf77
|
| |
|
| |
|
|
|