diff options
| author | George Burgess IV <gbiv@google.com> | 2018-05-21 14:41:57 -0700 |
|---|---|---|
| committer | George Burgess IV <gbiv@google.com> | 2018-05-22 09:13:31 -0700 |
| commit | c4a6d27fce2e8020bcf6c098ee2c4c5d4055743b (patch) | |
| tree | 28a641047463262c488035216f8df6e6e6afca87 /server/InterfaceController.cpp | |
| parent | a2b8381e6e5ab1b0604ec90b4c8044f626096dd3 (diff) | |
Simplify code
So, our static analyzer complained about this code having a
use-after-free because of the const string&s here. This appears to be a
broken static analyzer, but I'm unsure:
- why this array has to be static, and
- why the string fields have to be const&
By the letter of the standard, this code will work without this patch,
but unless there's something subtle going on that requires these strings
to live forever (in which case, we should probably have a comment), I
think that making the code slightly simpler is a good thing.
No functional change is intended.
Bug: None
Test: Builds; tool is happier.
Change-Id: I1b953c579cea95f2a71d820b5026737057faeeb6
Diffstat (limited to 'server/InterfaceController.cpp')
0 files changed, 0 insertions, 0 deletions
