commit | 7ac610b7edc059fbcc378cdac9ca235975470563 | [log] [tgz] |
---|---|---|
author | Wade Guthrie <wdg@chromium.org> | Tue Oct 01 17:48:14 2013 -0700 |
committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | Thu Oct 10 18:06:33 2013 +0000 |
tree | 7b1a1d15573621360ba9933f5bade3b9bf6478f9 | |
parent | 13804a1acd238a3c7ba8f4f44db96a97fa04fbda [diff] |
shill: Converts many metrics calls to take references, not pointers. I noticed some code that takes pointers that could crash if given a NULL pointer. While I fixed that, I decided that I'd modify other methods that take pointers but where NULL is not a valid parameter. This seems consistent with Google style. BUG=None. TEST=unittest Change-Id: I368fd66e1aa9d3ff81ec0fc2902af4721d6c1dde Reviewed-on: https://chromium-review.googlesource.com/171510 Reviewed-by: mukesh agrawal <quiche@chromium.org> Reviewed-by: Wade Guthrie <wdg@chromium.org> Tested-by: Wade Guthrie <wdg@chromium.org> Commit-Queue: Wade Guthrie <wdg@chromium.org>