diff options
| author | newt <newt@chromium.org> | 2015-08-19 09:42:59 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-08-19 16:44:09 +0000 |
| commit | 9a9993f2b61f7b0eb947dda9b4c777bbdcf61e66 (patch) | |
| tree | 0a61626cec2298e8cd68969057e3a7963fc47e52 /sync | |
| parent | f62ea41d40ecf774dbb44233252388a55c3cf0ee (diff) | |
| download | chromium_src-9a9993f2b61f7b0eb947dda9b4c777bbdcf61e66.zip chromium_src-9a9993f2b61f7b0eb947dda9b4c777bbdcf61e66.tar.gz chromium_src-9a9993f2b61f7b0eb947dda9b4c777bbdcf61e66.tar.bz2 | |
Remove redundant null checks before instanceof checks.
In Java, "x instanceof Y" can only be true if x is non-null, so it's not
necessary to first check that "x != null". See:
http://docs.oracle.com/javase/specs/jls/se7/html/jls-15.html#jls-15.20.2
This removes all such unnecessary null checks.
Review URL: https://codereview.chromium.org/1303533004
Cr-Commit-Position: refs/heads/master@{#344251}
Diffstat (limited to 'sync')
| -rw-r--r-- | sync/test/android/javatests/src/org/chromium/sync/test/util/AccountHolder.java | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sync/test/android/javatests/src/org/chromium/sync/test/util/AccountHolder.java b/sync/test/android/javatests/src/org/chromium/sync/test/util/AccountHolder.java index 01f0ff4b..34b52e9 100644 --- a/sync/test/android/javatests/src/org/chromium/sync/test/util/AccountHolder.java +++ b/sync/test/android/javatests/src/org/chromium/sync/test/util/AccountHolder.java @@ -141,8 +141,8 @@ public class AccountHolder { @Override public boolean equals(Object that) { - return that != null && that instanceof AccountHolder && mAccount - .equals(((AccountHolder) that).getAccount()); + return that instanceof AccountHolder + && mAccount.equals(((AccountHolder) that).getAccount()); } public static Builder create() { |