diff options
author | Harsh Shandilya <me@msfjarvis.dev> | 2018-05-05 16:26:52 +0530 |
---|---|---|
committer | Jason A. Donenfeld <Jason@zx2c4.com> | 2018-05-09 18:16:14 +0200 |
commit | 0b9bcf0f9e25122f9ee351aa9c889ebb01a238f5 (patch) | |
tree | e7e66720eca79145348adb82b763a0ee10685b3a /app/src/main/java/com/wireguard/android/fragment/TunnelEditorFragment.java | |
parent | MainActivity: Nip out as early as possible when moving to same fragment (diff) | |
download | wireguard-android-0b9bcf0f9e25122f9ee351aa9c889ebb01a238f5.tar.xz wireguard-android-0b9bcf0f9e25122f9ee351aa9c889ebb01a238f5.zip |
global: Cleanup line lengths and misc lint warnings
I know we decided to ditch the idea of shutting up "Exception
thrown with empty param" warnings but this pesters me too
much and we can instead just treat this as a weird future proofing
thing if and when we end up needing the exception messages.
Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>
Diffstat (limited to 'app/src/main/java/com/wireguard/android/fragment/TunnelEditorFragment.java')
-rw-r--r-- | app/src/main/java/com/wireguard/android/fragment/TunnelEditorFragment.java | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/app/src/main/java/com/wireguard/android/fragment/TunnelEditorFragment.java b/app/src/main/java/com/wireguard/android/fragment/TunnelEditorFragment.java index 91d5131b..f7ed5e77 100644 --- a/app/src/main/java/com/wireguard/android/fragment/TunnelEditorFragment.java +++ b/app/src/main/java/com/wireguard/android/fragment/TunnelEditorFragment.java @@ -46,7 +46,7 @@ public class TunnelEditorFragment extends BaseFragment { binding.setConfig(new Config.Observable(config, name)); } - private void onConfigSaved(final Tunnel savedTunnel, final Config config, + private void onConfigSaved(final Tunnel savedTunnel, final Throwable throwable) { final String message; if (throwable == null) { @@ -141,7 +141,7 @@ public class TunnelEditorFragment extends BaseFragment { } else { Log.d(TAG, "Attempting to save config of " + tunnel.getName()); tunnel.setConfig(newConfig) - .whenComplete((a, b) -> onConfigSaved(tunnel, a, b)); + .whenComplete((a, b) -> onConfigSaved(tunnel, b)); } return true; default: @@ -192,7 +192,7 @@ public class TunnelEditorFragment extends BaseFragment { Log.d(TAG, message); // Now save the rest of configuration changes. Log.d(TAG, "Attempting to save config of renamed tunnel " + tunnel.getName()); - renamedTunnel.setConfig(newConfig).whenComplete((a, b) -> onConfigSaved(renamedTunnel, a, b)); + renamedTunnel.setConfig(newConfig).whenComplete((a, b) -> onConfigSaved(renamedTunnel, b)); } else { final String error = ExceptionLoggers.unwrap(throwable).getMessage(); message = getString(R.string.tunnel_rename_error, error); |