clean dead code and solve warnings

Author: SliDeeN

app/src/main/AndroidManifest.xml

@@ -1,5 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <manifest xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:tools="http://schemas.android.com/tools"
     android:sharedUserId="android.uid.system">
 
     <!-- Permissions — same as original DriveHub Dort -->
@@ -13,12 +14,14 @@
 
     <!-- Consommation de donnees : signature|privileged, accessible car l app est signee
          avec la cle plateforme. Sans elle, NetworkStatsManager leve une SecurityException. -->
-    <uses-permission android:name="android.permission.READ_NETWORK_USAGE_HISTORY" />
+    <uses-permission android:name="android.permission.READ_NETWORK_USAGE_HISTORY"
+        tools:ignore="ProtectedPermissions" />
     <!-- Pilotage des sessions média (piste suivante, lecture/pause). Voie du launcher
          d'origine sur les firmwares A9, et seule voie pour CarPlay/Android Auto sur
          SWI68/165. signature|privileged : obtenue parce que l'app tourne en
          android.uid.system. -->
-    <uses-permission android:name="android.permission.MEDIA_CONTENT_CONTROL" />
+    <uses-permission android:name="android.permission.MEDIA_CONTENT_CONTROL"
+        tools:ignore="ProtectedPermissions" />
     <!-- Vehicle control permissions (Android Automotive / SAIC) -->
     <uses-permission android:name="android.car.permission.CAR_ENERGY" />
     <uses-permission android:name="android.car.permission.CAR_POWERTRAIN" />

app/src/main/java/com/mg4/control/MG4App.kt

@@ -31,7 +31,7 @@ class MG4App : Application() {
         }
 
         // ── Migration + initialisation du thème ───────────────────────────────
-        val prefs = getSharedPreferences("mg4_settings", Context.MODE_PRIVATE)
+        val prefs = getSharedPreferences("mg4_settings", MODE_PRIVATE)
 
         if (!prefs.contains(ThemeHelper.PREF_THEME_MODE)) {
             // Migration depuis l'ancien booléen "dark_theme" (version < 2.x)

app/src/main/java/com/mg4/control/MainActivity.kt

@@ -169,10 +169,10 @@ class MainActivity : AppCompatActivity() {
 
     // ── Navigation vers l'écran par défaut au démarrage ─────────────────────
 
-    private fun navigateToDefaultScreen(savedInstanceState: android.os.Bundle?) {
+    private fun navigateToDefaultScreen(savedInstanceState: Bundle?) {
         // Ne naviguer que si c'est un vrai démarrage (pas une rotation / recreate)
         if (savedInstanceState != null) return
-        val prefs = getSharedPreferences("mg4_settings", android.content.Context.MODE_PRIVATE)
+        val prefs = getSharedPreferences("mg4_settings", MODE_PRIVATE)
         when (prefs.getString("default_screen", "dashboard")) {
             "profiles"  -> navController.navigate(R.id.profileFragment)
             "shortcuts" -> navController.navigate(R.id.shortcutsFragment)
@@ -184,7 +184,7 @@ class MainActivity : AppCompatActivity() {
 
     private fun checkForUpdates() {
         if (BuildConfig.OFFLINE) return  // build offline : aucune vérif réseau
-        val prefs = getSharedPreferences("mg4_settings", android.content.Context.MODE_PRIVATE)
+        val prefs = getSharedPreferences("mg4_settings", MODE_PRIVATE)
         if (!prefs.getBoolean("auto_check_update", true)) return
 
         UpdateChecker.check(

app/src/main/java/com/mg4/control/accessibility/AdvancedShortcuts.kt

@@ -122,7 +122,7 @@ object AdvancedShortcuts {
                              else reste.substring(coupure + 1).takeIf { it.isNotEmpty() }
                 // Le suffixe est cherché par correspondance exacte, sans valeur de repli : une
                 // clé inconnue doit être IGNORÉE, pas rattachée arbitrairement à un type d'appui.
-                val press = PressType.values().firstOrNull { partieTouche.endsWith("_${it.key}") }
+                val press = PressType.entries.firstOrNull { partieTouche.endsWith("_${it.key}") }
                     ?: return@mapNotNull null
                 val code = partieTouche.removeSuffix("_${press.key}").toIntOrNull()
                     ?: return@mapNotNull null
@@ -209,7 +209,7 @@ object AdvancedShortcuts {
      * même le délai d'un renvoi.
      */
     fun isClaimedNow(context: Context, keyCode: Int): Boolean =
-        PressType.values().any { resolve(context, keyCode, it) != null }
+        PressType.entries.any { resolve(context, keyCode, it) != null }
 
     /**
      * Vrai si CETTE TOUCHE porte au moins un raccourci — quel que soit le type d'appui ET quel que

app/src/main/java/com/mg4/control/accessibility/KeyCaptureService.kt

@@ -14,7 +14,6 @@ import com.mg4.control.debug.AppLogger
 import com.mg4.control.service.MG4ControlService
 import com.mg4.control.service.ProfilePickerOverlay
 import com.mg4.control.shortcut.PressType
-import com.mg4.control.shortcut.ShortcutAction
 import com.mg4.control.util.GarageMode
 import java.util.concurrent.ConcurrentHashMap
 import java.util.concurrent.Executors

app/src/main/java/com/mg4/control/api/ExternalApi.kt

@@ -177,15 +177,14 @@ object ExternalApi {
 
     // ── Lecture (ContentProvider) ────────────────────────────────────────────
 
-    /**
+    /*
      * `content://<applicationId>.state/state` → une ligne, une colonne par valeur.
      *
      * ⚠️ L'authority suit l'applicationId, elle n'est donc PAS une constante : la variante
      * offline s'installe à côté de l'online et deux paquets ne peuvent pas déclarer la même
      * (INSTALL_FAILED_CONFLICTING_PROVIDER). Les intégrateurs doivent viser
      * `com.mg4.control.state` ou `com.mg4.control.offline.state` selon la variante installée.
      */
-    fun authority(context: Context): String = context.packageName + ".state"
 
     const val PATH_STATE = "state"
 

app/src/main/java/com/mg4/control/debug/AppLogger.kt

@@ -1,20 +1,17 @@
 package com.mg4.control.debug
 
-import android.os.Handler
-import android.os.Looper
 import android.util.Log
 import java.text.SimpleDateFormat
 import java.util.Date
 import java.util.Locale
-import java.util.concurrent.CopyOnWriteArrayList
 
 /**
  * In-app log buffer — mirrors every Log.* call to an in-memory ring buffer
- * so the ConsoleFragment can display them without ADB.
+ * so the Diagnostic report can include them without ADB.
  *
  * Le buffer est un ArrayDeque sous verrou : la CopyOnWriteArrayList précédente recopiait
  * les 400 entrées deux fois par ligne de log, sur le thread appelant — c'est-à-dire
- * pendant l'application d'un profil. Les listeners sont notifiés hors du chemin chaud.
+ * pendant l'application d'un profil.
  */
 object AppLogger {
 
@@ -46,14 +43,6 @@ object AppLogger {
     val entries: List<Entry>
         get() = synchronized(lock) { buffer.toList() }
 
-    private val listeners = CopyOnWriteArrayList<() -> Unit>()
-
-    private val mainHandler = Handler(Looper.getMainLooper())
-
-    /** Notification déjà programmée : une rafale de logs ne donne qu'un seul réveil UI. */
-    @Volatile
-    private var notifyPending = false
-
     // ---- Public log methods (mirror android.util.Log) ----
 
     fun d(tag: String, msg: String) { add(tag, Level.DEBUG, msg); Log.d(tag, msg) }
@@ -66,7 +55,6 @@ object AppLogger {
             buffer.clear()
             totalCount = 0L
         }
-        notifyListeners()
     }
 
     /**
@@ -84,11 +72,6 @@ object AppLogger {
         }
     }
 
-    // ---- Listener for live UI updates ----
-
-    fun addListener(l: () -> Unit)    { listeners.add(l) }
-    fun removeListener(l: () -> Unit) { listeners.remove(l) }
-
     // ---- Internal ----
 
     private fun add(tag: String, level: Level, msg: String) {
@@ -99,19 +82,5 @@ object AppLogger {
             buffer.addLast(entry)
             totalCount++
         }
-        notifyListeners()
-    }
-
-    /**
-     * Réveille l'UI sur le thread principal, au plus une fois par rafale : la notification
-     * ne doit pas s'exécuter sur le thread qui journalise.
-     */
-    private fun notifyListeners() {
-        if (listeners.isEmpty() || notifyPending) return
-        notifyPending = true
-        mainHandler.post {
-            notifyPending = false
-            listeners.forEach { runCatching { it.invoke() } }
-        }
     }
 }