diff options
| author | Kweku Adams <kwekua@google.com> | 2019-07-11 12:33:10 -0700 |
|---|---|---|
| committer | Kweku Adams <kwekua@google.com> | 2019-07-12 17:10:39 -0700 |
| commit | 6acbca3a9ae74d94c70637e09cd41d617f48ac8d (patch) | |
| tree | 8c0d0be410729f116f0c977176f96d2f19831ffa /docs/html/sdk/api_diff/20/changes | |
| parent | 3f8cdda0497f8dc4fbba56892777327f9bee9273 (diff) | |
Rename ProtoInputStream.isNextField to nextField.
ProtoInputStream.isNextField() skips over the current field to read the
next tag. This side effect means that calling isNextField prevents the
caller from reading the field it was just on. The "is" prefix makes it
easy to assume that the method has no side effects. Renaming to
nextField(long) with a parameter should help make it clear that the call
is as dangerous as calling nextField().
Fixes: 137294060
Test: atest android.app.usage.UsageStatsTest
Test: atest android.app.usage.cts.UsageStatsTest
Test: atest com.android.server.usage.UsageStatsDatabaseTest
Test: atest ProtoInputStreamTests
Change-Id: Ibfb8ef741f27a211f076dc0b715b64acbfc0696a
Diffstat (limited to 'docs/html/sdk/api_diff/20/changes')
0 files changed, 0 insertions, 0 deletions