diff options
Diffstat (limited to 'chromium/styleguide/java/java.md')
-rw-r--r-- | chromium/styleguide/java/java.md | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/chromium/styleguide/java/java.md b/chromium/styleguide/java/java.md index 76d5f753353..24f98a3b3c1 100644 --- a/chromium/styleguide/java/java.md +++ b/chromium/styleguide/java/java.md @@ -94,7 +94,7 @@ try { somethingThatThrowsIOException(); somethingThatThrowsParseException(); } catch (IOException | ParseException e) { - Log.e(TAG, "Failed to do something with exception: ", e) + Log.e(TAG, "Failed to do something with exception: ", e); } ``` @@ -111,16 +111,15 @@ where C++ DCHECK()s make sense. For multi-statement asserts, use Example assert: ```java -assert someCallWithSideEffects() : "assert description"; +assert someCallWithoutSideEffects() : "assert description"; ``` Example use of `DCHECK_IS_ON`: ```java if (org.chromium.base.BuildConfig.DCHECK_IS_ON) { - if (!someCallWithSideEffects()) { - throw new AssertionError("assert description"); - } + // Any code here will be stripped in Release by ProGuard. + ... } ``` |