Changeset 1900

Timestamp:

06/06/12 00:55:04 (13 months ago)

Author:

philmoz

Message:

Add min/max range option to script parameters to control allowed values on parameters.

Location:

trunk

Files:

• core/gui.c (modified) (1 diff)
• core/gui_menu.h (modified) (1 diff)
• core/script.c (modified) (11 diffs)
• include/script.h (modified) (1 diff)
• lib/ubasic/tokenizer.c (modified) (1 diff)

trunk/core/gui.c

@@ -1345 +1345 @@
             script_submenu_items[p].type=MENUITEM_INT;
             script_submenu_items[p].value=&conf.script_vars[script_param_order[i]-1];
+            if (script_range_values[script_param_order[i]-1] != 0)
+            {
+                if (script_range_values[script_param_order[i]-1] == MENU_MINMAX(0,1))
+                {
+                    script_submenu_items[p].type = MENUITEM_BOOL;
+                }
+                else
+                {
+                    script_submenu_items[p].type |= MENUITEM_F_MINMAX;
+                    script_submenu_items[p].arg = script_range_values[script_param_order[i]-1];
+                }
+            }
             ++p;
         }

trunk/core/gui_menu.h

@@ -33 +33 @@
 #define MENUITEM_ARG_CALLBACK   0x0300
 
-#define MENU_MINMAX(min, max)   (((max)<<16)|(min))
+#define MENU_MINMAX(min, max)   (((max)<<16)|(min&0xFFFF))
 
 //-------------------------------------------------------------------

trunk/core/script.c

@@ -5 +5 @@
 #include "gui.h"
 #include "gui_draw.h"
+#include "gui_menu.h"
 #include "conf.h"
 #include "script.h"
@@ -106 +107 @@
 int script_param_order[SCRIPT_NUM_PARAMS];                  // Ordered as_in_script list of variables ( [idx] = id_of_var )
                                                             // to display in same order in script
+int script_range_values[SCRIPT_NUM_PARAMS];                 // Min/Max values for param validation
 static char script_params_update[SCRIPT_NUM_PARAMS];        // Flag is such parameter exist
 static int script_loaded_params[SCRIPT_NUM_PARAMS];         // Copy of original values of parameters 
@@ -113 +115 @@
 
 //-------------------------------------------------------------------
+
+const char* skip_whitespace(const char* p)  { while (*p==' ' || *p=='\t') p++; return p; }  // Skip past whitespace
+const char* skip_token(const char* p)       { while (*p!=' ' && *p!='\t') p++; return p; }  // Skip to next whitespace
+
+//-------------------------------------------------------------------
 static void process_title(const char *title) {
     register const char *ptr = title;
     register int i=0;
 
-    while (ptr[0]==' ' || ptr[0]=='\t') ++ptr; // whitespaces
+    ptr = skip_whitespace(ptr);
     while (i<(sizeof(script_title)-1) && ptr[i] && ptr[i]!='\r' && ptr[i]!='\n') {
         script_title[i]=ptr[i];
@@ -138 +145 @@
     register int n, i=0;
 
-    while (ptr[0]==' ' || ptr[0]=='\t') ++ptr; // whitespaces
+    ptr = skip_whitespace(ptr);
     if (ptr[0] && (ptr[0]>='a' && ptr[0]<='a'+SCRIPT_NUM_PARAMS) && (ptr[1]==' ' || ptr[1]=='\t')) {
         n=ptr[0]-'a';
-        ptr+=2;
-        while (ptr[0]==' ' || ptr[0]=='\t') ++ptr; // whitespaces
+        ptr = skip_whitespace(ptr+2);
         script_params_update[n] = 1;
         while (i<(sizeof(script_params[0])-1) && ptr[i] && ptr[i]!='\r' && ptr[i]!='\n') {
@@ -169 +175 @@
     register int n;
 
-    while (ptr[0]==' ' || ptr[0]=='\t') ++ptr; // whitespaces
+    ptr = skip_whitespace(ptr);
     if (ptr[0] && (ptr[0]>='a' && ptr[0]<='a'+SCRIPT_NUM_PARAMS) && (ptr[1]==' ' || ptr[1]=='\t')) {
         n=ptr[0]-'a';
-        ptr+=2;
+        ptr = skip_whitespace(ptr+2);
         if (!update || script_params_update[n])
         {
             conf.script_vars[n] = strtol(ptr, NULL, 0);
             script_loaded_params[n] = conf.script_vars[n];
+        }
+    } // ??? else produce error message
+}
+
+//-------------------------------------------------------------------
+// Process one entry "@range VAR MIN MAX"
+//      param = ptr right after descriptor (should point to var)
+//      update = 0 - get
+//               1 - only if updated
+//-------------------------------------------------------------------
+static void process_range(const char *param, char update) {
+    register const char *ptr = param;
+    register int n;
+
+    ptr = skip_whitespace(ptr);
+    if (ptr[0] && (ptr[0]>='a' && ptr[0]<='a'+SCRIPT_NUM_PARAMS) && (ptr[1]==' ' || ptr[1]=='\t')) {
+        n=ptr[0]-'a';
+        ptr = skip_whitespace(ptr+2);
+        if (!update || script_params_update[n])
+        {
+            short min = strtol(ptr,NULL,0);
+            ptr = skip_whitespace(skip_token(ptr));
+            short max = strtol(ptr,NULL,0);
+            script_range_values[n] = MENU_MINMAX(min,max);
         }
     } // ??? else produce error message
@@ -201 +231 @@
         script_params[i][0]=0;
         script_param_order[i]=0;
+        script_range_values[i] = 0;
     }
 
     while (ptr[0]) {
-        while (ptr[0]==' ' || ptr[0]=='\t') ++ptr; // whitespaces
+        ptr = skip_whitespace(ptr);
         if (ptr[0]=='@') {
             if (strncmp("@title", ptr, 6)==0) {
@@ -219 +250 @@
                 ptr+=8;
                 process_default(ptr, 0);
+            } else if (update_vars && strncmp("@range", ptr, 6)==0) {
+                ptr+=6;
+                process_range(ptr, 0);
             }
         }
@@ -329 +363 @@
                 ptr+=8;
                 process_default(ptr, 1);
+            } else if (strncmp("@range", ptr, 6)==0) {
+                ptr+=6;
+                process_range(ptr, 1);
             }
         }
@@ -353 +390 @@
 {
     int i, n, fd, changed=0;
-    char *buf,*p;
+    char *buf;
     for(i = 0; i < SCRIPT_NUM_PARAMS; i++)
     {
@@ -375 +412 @@
 
     // max possible params * (param description + some extra for @default etc)
-    buf=umalloc(SCRIPT_NUM_PARAMS*(28 + 20));
+    buf=umalloc(256);
     if(!buf)
         return;
@@ -385 +422 @@
     }
     buf[0] = 0;
-    p=buf;
     for(n = 0; n < SCRIPT_NUM_PARAMS; ++n)
     {
         if (script_params[n][0] != 0)
         {
-            p+=sprintf(p,"@param %c %s\n@default %c %d\n",'a'+n,script_params[n],'a'+n,conf.script_vars[n]);
-        }
-    }
-    write(fd, buf, strlen(buf));
+            sprintf(buf,"@param %c %s\n@default %c %d\n",'a'+n,script_params[n],'a'+n,conf.script_vars[n]);
+            if (script_range_values[n] != 0)
+                sprintf(buf+strlen(buf),"@range %c %d %d\n",'a'+n,(short)(script_range_values[n]&0xFFFF),(short)(script_range_values[n]>>16));
+            write(fd, buf, strlen(buf));
+        }
+    }
     close(fd);
     ufree(buf);

trunk/include/script.h

@@ -19 +19 @@
 extern char script_title[36];
 extern char script_params[SCRIPT_NUM_PARAMS][28];
+extern int script_range_values[SCRIPT_NUM_PARAMS];
 extern int script_param_order[SCRIPT_NUM_PARAMS];
 

trunk/lib/ubasic/tokenizer.c

@@ -202 +202 @@
   {"@param",                  TOKENIZER_REM},
   {"@default",                TOKENIZER_REM},
+  {"@range",                  TOKENIZER_REM},
 
   {"md_detect_motion",        TOKENIZER_MD_DETECT_MOTION},