diff options
author | Harsh Shandilya <harsh@prjkt.io> | 2018-04-27 22:59:24 +0530 |
---|---|---|
committer | Jason A. Donenfeld <Jason@zx2c4.com> | 2018-04-28 06:07:48 +0200 |
commit | 27bdfd14187ad9c804856e2c0ef76f0bf565283e (patch) | |
tree | 8f4f08ccec905f27062a4bb337a6c44185726e9a /app/src/main/java/com/wireguard/android/fragment/TunnelEditorFragment.java | |
parent | Version bump (diff) | |
download | wireguard-android-27bdfd14187ad9c804856e2c0ef76f0bf565283e.tar.xz wireguard-android-27bdfd14187ad9c804856e2c0ef76f0bf565283e.zip |
Quick cleanup across the board
- Clean up imports
- Adjust method visibility
- Drop in NonNull annotations where needed
- Prevent potential NPEs from nulled getActivity calls
- Remove unused methods
Signed-off-by: Harsh Shandilya <harsh@prjkt.io>
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, 4 insertions, 2 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 f3316490..0b4d74d8 100644 --- a/app/src/main/java/com/wireguard/android/fragment/TunnelEditorFragment.java +++ b/app/src/main/java/com/wireguard/android/fragment/TunnelEditorFragment.java @@ -5,6 +5,7 @@ import android.content.Context; import android.os.Bundle; import android.os.Parcel; import android.os.Parcelable; +import android.support.annotation.NonNull; import android.util.Log; import android.view.LayoutInflater; import android.view.Menu; @@ -101,7 +102,7 @@ public class TunnelEditorFragment extends BaseFragment { } @Override - public View onCreateView(final LayoutInflater inflater, final ViewGroup container, + public View onCreateView(@NonNull final LayoutInflater inflater, final ViewGroup container, final Bundle savedInstanceState) { super.onCreateView(inflater, container, savedInstanceState); binding = TunnelEditorFragmentBinding.inflate(inflater, container, false); @@ -118,6 +119,7 @@ public class TunnelEditorFragment extends BaseFragment { private void onFinished() { // Hide the keyboard; it rarely goes away on its own. final Activity activity = getActivity(); + if (activity == null) return; final View focusedView = activity.getCurrentFocus(); if (focusedView != null) { final Object service = activity.getSystemService(Context.INPUT_METHOD_SERVICE); @@ -174,7 +176,7 @@ public class TunnelEditorFragment extends BaseFragment { } @Override - public void onSaveInstanceState(final Bundle outState) { + public void onSaveInstanceState(@NonNull final Bundle outState) { outState.putParcelable(KEY_LOCAL_CONFIG, localConfig); outState.putString(KEY_ORIGINAL_NAME, originalName); super.onSaveInstanceState(outState); |