summaryrefslogtreecommitdiff
path: root/server/InterfaceController.cpp
diff options
context:
space:
mode:
authorGeorge Burgess IV <gbiv@google.com>2018-05-21 14:41:57 -0700
committerGeorge Burgess IV <gbiv@google.com>2018-05-22 09:13:31 -0700
commitc4a6d27fce2e8020bcf6c098ee2c4c5d4055743b (patch)
tree28a641047463262c488035216f8df6e6e6afca87 /server/InterfaceController.cpp
parenta2b8381e6e5ab1b0604ec90b4c8044f626096dd3 (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