Merge pull request #174 from NordicSemiconductor/bigfix/no-logger

Fixed crash when nRF Logger is not installed
NordicSemiconductor · Aug 16, 2024 · 117d362 · 117d362
2 parents 1a22bcd + 858c2ce
commit 117d362
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/sample/src/main/java/io/runtime/mcumgr/sample/MainActivity.java b/sample/src/main/java/io/runtime/mcumgr/sample/MainActivity.java
@@ -54,6 +54,7 @@ public class MainActivity extends AppCompatActivity
     @Inject
     BluetoothDevice device;
     @Inject
+    @Nullable
     Uri logSessionUri;
 
     private Fragment deviceFragment;

diff --git a/sample/src/main/java/io/runtime/mcumgr/sample/di/component/McuMgrSubComponent.java b/sample/src/main/java/io/runtime/mcumgr/sample/di/component/McuMgrSubComponent.java
@@ -9,6 +9,8 @@
 import android.bluetooth.BluetoothDevice;
 import android.net.Uri;
 
+import androidx.annotation.Nullable;
+
 import dagger.BindsInstance;
 import dagger.Subcomponent;
 import io.runtime.mcumgr.sample.application.Dagger2Application;
@@ -40,7 +42,7 @@ interface Builder {
         Builder target(final BluetoothDevice device);
 
         @BindsInstance
-        Builder logSessionUri(final Uri uri);
+        Builder logSessionUri(final @Nullable Uri uri);
 
         McuMgrSubComponent build();
     }