Commit be6fc77a authored by Bartek Fabiszewski's avatar Bartek Fabiszewski
Browse files

Fix null pointer dereference warning

parent b81fbfae

app/src/main/java/net/fabiszewski/ulogger/LoggerTask.java

+1 −1
Original line number Diff line number Diff line
@@ -238,7 +238,7 @@ class LoggerTask implements LocationListener, Runnable { 
    }



    interface LoggerTaskCallback {

        void onLoggerTaskCompleted(Location location);

        void onLoggerTaskCompleted(@NonNull Location location);

        void onLoggerTaskFailure(int reason);

        Activity getActivity();

    }

app/src/main/java/net/fabiszewski/ulogger/WaypointFragment.java

+8 −6
Original line number Diff line number Diff line
@@ -333,12 +333,14 @@ public class WaypointFragment extends Fragment implements LoggerTask.LoggerTaskC 
     * Display location details

     */

    private void setLocationText() {

        if (location != null) {

            LocationFormatter formatter = new LocationFormatter(location);

            locationNotFoundTextView.setVisibility(View.GONE);

            locationTextView.setText(String.format("%s\n—\n%s", formatter.getLongitudeDMS(), formatter.getLatitudeDMS()));

            locationTextView.setVisibility(View.VISIBLE);

            locationDetailsTextView.setText(formatter.getDetails(requireContext()));

        }

    }





    private void clearLocationText() {
@@ -474,7 +476,7 @@ public class WaypointFragment extends Fragment implements LoggerTask.LoggerTaskC 
     * @param location Current location

     */

    @Override

    public void onLoggerTaskCompleted(Location location) {

    public void onLoggerTaskCompleted(@NonNull Location location) {

        if (Logger.DEBUG) { Log.d(TAG, "[onLoggerTaskCompleted: " + location + "]"); }

        this.location = location;

        if (imageTask == null || !imageTask.isRunning()) {