Minor Android lints

Author: ospfranco

OpacityCore/src/main/cpp/CMakeLists.txt

@@ -8,7 +8,6 @@ add_library(opacity_core STATIC IMPORTED)
 set_target_properties(opacity_core PROPERTIES IMPORTED_LOCATION ${OPACITY_CORE_LIB})
 
 add_library(${CMAKE_PROJECT_NAME} SHARED
-    # List C/C++ source files with relative paths to this CMakeLists.txt.
     OpacityCore.cpp)
 
 target_include_directories(${CMAKE_PROJECT_NAME} PUBLIC ${CMAKE_SOURCE_DIR}/../jni/include)

OpacityCore/src/main/kotlin/com/opacitylabs/opacitycore/OpacityCore.kt

@@ -6,8 +6,19 @@ import android.os.Bundle
 import androidx.localbroadcastmanager.content.LocalBroadcastManager
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.withContext
-import kotlinx.serialization.*
-import kotlinx.serialization.json.*
+import kotlinx.serialization.encodeToString
+import kotlinx.serialization.json.Json
+import kotlinx.serialization.json.JsonArray
+import kotlinx.serialization.json.JsonElement
+import kotlinx.serialization.json.JsonObject
+import kotlinx.serialization.json.JsonPrimitive
+import kotlinx.serialization.json.boolean
+import kotlinx.serialization.json.booleanOrNull
+import kotlinx.serialization.json.double
+import kotlinx.serialization.json.doubleOrNull
+import kotlinx.serialization.json.int
+import kotlinx.serialization.json.intOrNull
+import kotlinx.serialization.json.jsonObject
 import org.mozilla.geckoview.GeckoRuntime
 
 object OpacityCore {
@@ -67,8 +78,8 @@ object OpacityCore {
     fun presentBrowser() {
         val intent = Intent()
         intent.setClassName(
-                appContext.packageName,
-                "com.opacitylabs.opacitycore.InAppBrowserActivity"
+            appContext.packageName,
+            "com.opacitylabs.opacitycore.InAppBrowserActivity"
         )
         intent.putExtra("url", _url)
         intent.putExtra("headers", headers)
@@ -91,12 +102,15 @@ object OpacityCore {
                     else -> throw Exception("Could not convert JSON primitive $jsonElement")
                 }
             }
+            
             is JsonObject -> {
                 jsonElement.toMap().mapValues { parseJsonElementToAny(it.value) }
             }
+
             is JsonArray -> {
                 jsonElement.map { parseJsonElementToAny(it) }
             }
+
             else -> throw Exception("Could not convert JSON primitive $jsonElement")
         }
     }
@@ -111,9 +125,9 @@ object OpacityCore {
             }
 
             val map: Map<String, Any> =
-                    Json.parseToJsonElement(res.data!!).jsonObject.mapValues {
-                        parseJsonElementToAny(it.value)
-                    }
+                Json.parseToJsonElement(res.data!!).jsonObject.mapValues {
+                    parseJsonElementToAny(it.value)
+                }
             map
         }
     }

app/src/main/AndroidManifest.xml

@@ -6,6 +6,7 @@
     <uses-permission android:name="android.permission.ACCESS_FINE_LOCATION" />
     <uses-permission android:name="android.permission.ACCESS_COARSE_LOCATION" />
     <uses-permission android:name="android.permission.CAMERA" />
+    <uses-permission android:name="android.permission.POST_NOTIFICATIONS" />
 
     <uses-feature android:name="android.hardware.camera" />
     <application

gradle.properties

@@ -6,7 +6,7 @@
 # http://www.gradle.org/docs/current/userguide/build_environment.html
 # Specifies the JVM arguments used for the daemon process.
 # The setting is particularly useful for tweaking memory settings.
-org.gradle.jvmargs=-Xmx2048m -Dfile.encoding=UTF-8
+org.gradle.jvmargs=-Xmx4096m -Dfile.encoding=UTF-8
 # When configured, Gradle will run in incubating parallel mode.
 # This option should only be used with decoupled projects. For more details, visit
 # https://developer.android.com/r/tools/gradle-multi-project-decoupled-projects