opendev/gerrit
Code Issues Proposed changes

Remove usages of Throwables#propagate

Browse Source 
Guava intends to deprecate these, and recommends explicitly wrapping
with RuntimeException instead.

Change-Id: Id148cb96164d31fae2d12945baa6db462b31fd36
This commit is contained in:
Dave Borowitz
2016-03-21 12:05:15 -04:00
parent 54e19df08f
commit 00f0839714
2 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
gerrit-server/src/main/java/com/google/gerrit/common/ChangeHookRunner.java
View File

@@ -17,7 +17,6 @@ package com.google.gerrit.common;
import com.google.common.base.Optional; import com.google.common.base.Optional;
import com.google.common.base.Supplier; import com.google.common.base.Supplier;
import com.google.common.base.Suppliers; import com.google.common.base.Suppliers;
import com.google.common.base.Throwables;
import com.google.common.collect.Sets; import com.google.common.collect.Sets;
import com.google.common.util.concurrent.ThreadFactoryBuilder; import com.google.common.util.concurrent.ThreadFactoryBuilder;
import com.google.gerrit.common.data.LabelType; import com.google.gerrit.common.data.LabelType;
@@ -845,7 +844,7 @@ public class ChangeHookRunner implements ChangeHooks, LifecycleListener,
return eventFactory.asPatchSetAttribute( return eventFactory.asPatchSetAttribute(
revWalk, change, patchSet); revWalk, change, patchSet);
} catch (IOException e) { } catch (IOException e) {
throw Throwables.propagate(e); throw new RuntimeException(e);
} }
} }
}); });

2
plugins/replication

Submodule plugins/replication updated: 77ae05e49c...8419e92cb0