summaryrefslogtreecommitdiff
path: root/chromium/styleguide/java/java.md
diff options
context:
space:
mode:
Diffstat (limited to 'chromium/styleguide/java/java.md')
-rw-r--r--chromium/styleguide/java/java.md9
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.
+ ...
}
```