diff options
| author | Alec Mouri <alecmouri@google.com> | 2021-01-11 17:32:29 -0800 |
|---|---|---|
| committer | Alec Mouri <alecmouri@google.com> | 2021-01-11 17:38:19 -0800 |
| commit | f2d993ae9338397a4ecdf67f8715df0a24682469 (patch) | |
| tree | 40e4cf8fe201083ee57c68783e338b38df1f06ae /tests/DynamicCodeLoggerIntegrationTests/src/cpp/test_executable.cpp | |
| parent | 82231edfc7aa5e14b0fba7f5ff96f594bb1855aa (diff) | |
Remove Point#dump
This is in response to API feedback - dump() methods for most of the
public classes describe complex internal state, which does not fit for
the Point class. Point#dump does not need to exist anyways, because
1. Point#toString exists, and
2. Point#dump only printed out the x,y coordinates, which are already
part of the public api, so printing out a more concise string
representation is trivial.
So, this patch removes Point#dump and just has RemoteAnimationTarget
implement its own print method for Point, since it was the only caller.
Bug: 159683987
Test: builds
Change-Id: I5c947330768b3e4e450eba3971de939a904d30d4
Diffstat (limited to 'tests/DynamicCodeLoggerIntegrationTests/src/cpp/test_executable.cpp')
0 files changed, 0 insertions, 0 deletions