diff options
| author | Nathan Harold <nharold@google.com> | 2018-09-07 11:50:05 -0700 |
|---|---|---|
| committer | Nathan Harold <nharold@google.com> | 2018-09-07 11:56:08 -0700 |
| commit | edf6ccaeb3c66baa331aa55fe0fcdc5c4697f3e7 (patch) | |
| tree | 11e66ff0ba3d6d50bc0ff0603fd62a7269a72a7b /tools/aapt2/java/JavaClassGenerator_test.cpp | |
| parent | 608fa51d55ea09caa31c2a4ec2d365db858cd86f (diff) | |
Don't Use NPE for Flow Control in getCellLocation
Remove catch of NPE from getCellLocation and do explicit
null checks. This will make it easier to differentiate
between actual problems and local NPEs. In pre-P, there
were multiple conditions that would intentionally return
null over the binder interface. In Post-P null is returned
when location is disabled, which means that null is a very
common return value. We shouldn't use NPE to handle the
case where location is disabled.
Bug: 114294321
Test: manual w/ SL4A (no automated tests for TM)
Change-Id: Ie5b3afa56f280ff901ff9fa7c93f958d754f109b
Diffstat (limited to 'tools/aapt2/java/JavaClassGenerator_test.cpp')
0 files changed, 0 insertions, 0 deletions