Merge pull request #290 from chrisws/12_35

12 35

Author: chrisws

ChangeLog

@@ -1,5 +1,14 @@
+2026-04-28 (12.34)
+	COMMON: implement plugin_refresh_id for variable lifecycle handling
+	COMMON: fix bug with comma in string assigned as local #283
+	COMMON: fix crash when missing () on module function call
+	CONSOLE: fix argument parsing to void adding switches to opt_command
+	ANDROID: Fix stability issue with chromebooks
+
 2026-02-26 (12.33)
 	COMMON: Fix build for FreeBSD
+	ANDROID: fix race with graphics drawing while pausing
+	ANDROID: fix back-button handling
 
 2026-01-23 (12.33)
 	SDL: Fix issue with flatpak release

configure.ac

@@ -7,7 +7,7 @@ dnl This program is distributed under the terms of the GPL v2.0
 dnl Download the GNU Public License (GPL) from www.gnu.org
 dnl
 
-AC_INIT([smallbasic], [12.34])
+AC_INIT([smallbasic], [12.35])
 AC_CONFIG_SRCDIR([configure.ac])
 
 AC_CANONICAL_TARGET

samples/distro-examples/tests/strings.bas

@@ -215,3 +215,7 @@ REM Bug: RTRIM trimed input string on the right side
 s1 = "   test   "
 s2 = rtrim(s1)
 if(s1 != "   test   ") then throw "err: RTRIM changed input string"
+
+REM bug with comma in string assigned as local
+local d="ab,"
+if d != "ab," then throw "err: comma bug"

src/common/eval.c

@@ -786,7 +786,11 @@ static inline void eval_var(var_t *r, var_t *var_p) {
     v_set(r, var_p);
     break;
   case V_FUNC:
-    var_p->v.fn.cb(var_p, r);
+    if (!var_p->v.fn.cb) {
+      err_missing_sep();
+    } else {
+      var_p->v.fn.cb(var_p, r);
+    }
     break;
   case V_NIL:
     r->type = V_NIL;

src/common/scan.c

@@ -1583,9 +1583,13 @@ int comp_getlist(char *source, char_p_t *args, int maxarg) {
   int square = 0;
   int round = 0;
   int brace = 0;
+  int quote = 0;
 
   while (*p && count < maxarg) {
     switch (*p) {
+    case '"':
+      quote = !quote;
+      break;
     case '[':
       square++;
       break;
@@ -1605,7 +1609,7 @@ int comp_getlist(char *source, char_p_t *args, int maxarg) {
       brace--;
       break;
     case ',':
-      if (!square && !round && !brace) {
+      if (!square && !round && !brace && !quote) {
         *p = '\0';
         SKIP_SPACES(ps);
         args[count] = ps;

src/lib/maapi.h

@@ -37,6 +37,7 @@
 #define EVENT_TYPE_KEY_PRESSED 11
 #define EVENT_TYPE_OPTIONS_BOX_BUTTON_CLICKED 41
 #define EVENT_TYPE_SCREEN_CHANGED 21
+#define EVENT_TYPE_SCREEN_CHANGING 22
 
 typedef int MAExtent;
 typedef intptr_t MAHandle;

src/platform/android/app/build.gradle

@@ -9,8 +9,8 @@ android {
     applicationId 'net.sourceforge.smallbasic'
     minSdkVersion 23
     targetSdkVersion 36
-    versionCode 91
-    versionName '12.33'
+    versionCode 94
+    versionName '12.35'
     resourceConfigurations += ['en']
   }
 

src/platform/android/app/src/main/assets/webui/sb-desktop-32x32.png

@@ -434,6 +434,14 @@ public int getWindowHeight() {
     return rect.height();
   }
 
+  public boolean isInsetBasedOnResize() {
+    return Build.VERSION.SDK_INT >= Build.VERSION_CODES.BAKLAVA;
+  }
+
+  public boolean isPredictiveBack() {
+    return Build.VERSION.SDK_INT >= Build.VERSION_CODES.BAKLAVA;
+  }
+
   /**
    * Check if traditional 3-button navigation is enabled
    * @return true if 3-button navigation is active
@@ -449,14 +457,6 @@ public boolean isThreeButtonNavigationEnabled() {
     return false;
   }
 
-  public boolean isInsetBasedOnResize() {
-    return Build.VERSION.SDK_INT >= Build.VERSION_CODES.BAKLAVA;
-  }
-
-  public boolean isPredictiveBack() {
-    return Build.VERSION.SDK_INT >= Build.VERSION_CODES.BAKLAVA;
-  }
-
   public boolean loadModules() {
     Log.i(TAG, "loadModules: " + getActivity());
     boolean result;
@@ -589,6 +589,17 @@ public String request(String endPoint, String data, String apiKey) throws IOExce
     return connection.invoke();
   }
 
+  public boolean requestApplyInsets() {
+    final View view = getWindow().getDecorView();
+    view.postDelayed(new Runnable() {
+      @Override
+      public void run() {
+        ViewCompat.requestApplyInsets(view);
+      }
+    }, 100);
+    return true;
+  }
+
   public boolean requestLocationUpdates() {
     final LocationManager locationService = (LocationManager) getSystemService(Context.LOCATION_SERVICE);
     boolean result = false;

src/platform/android/app/src/main/java/net/sourceforge/smallbasic/MainActivity.java

@@ -5,7 +5,7 @@ buildscript {
     mavenCentral()
   }
   dependencies {
-    classpath 'com.android.tools.build:gradle:9.1.1'
+    classpath 'com.android.tools.build:gradle:9.3.1'
   }
 }
 
@@ -19,7 +19,7 @@ allprojects {
 apply plugin: 'idea'
 
 tasks.register('clean', Delete) {
-  delete rootProject.buildDir
+  delete rootProject.layout.buildDirectory
 }
 
 tasks.register('test') {