WebSVN – filter_foundry – Diff – /trunk/ui.c

 /*
     This file is part of "Filter Foundry", a filter plugin for Adobe Photoshop
     Copyright (C) 2003-2009 Toby Thain, toby@telegraphics.net
-    Copyright (C) 2018-2022 Daniel Marschall, ViaThinkSoft
+    Copyright (C) 2018-2023 Daniel Marschall, ViaThinkSoft
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
     the Free Software Foundation; either version 2 of the License, or
     (at your option) any later version.
         int i;
         doupdates = false;
         for(i = 0; i < 8; ++i){
-                SETSLIDERVALUE(dp,FIRSTCTLITEM+i,slider[i]);
+                SETSLIDERVALUE(dp,FIRSTCTLITEM+i, gdata->parm.val[i]);
-                SETCTLTEXTINT(dp,FIRSTCTLTEXTITEM+i,slider[i],false);
+                SETCTLTEXTINT(dp,FIRSTCTLTEXTITEM+i, gdata->parm.val[i],false);
+        }
         for(i = 0; i < 4; ++i){
-                if(!gdata->standalone)
+                if(!gdata->parm.standalone)
-                        SETCTLTEXT(dp,FIRSTEXPRITEM+i,expr[i] ? expr[i] : "");
+                        SETCTLTEXT(dp,FIRSTEXPRITEM+i, gdata->parm.szFormula[i]);
                 if(i < nplanes)
                         updateexpr(dp,FIRSTEXPRITEM+i);
+        }
-        if(!gdata->standalone)
+        if(!gdata->parm.standalone)
                 SELECTCTLTEXT(dp,FIRSTEXPRITEM,0,-1);
         doupdates = true;
+}
 void updateglobals(DIALOGREF dp){
         UNREFERENCED_PARAMETER(dp);
-        // DM 28 Nov 2021: Removed this function. It makes no sense! The internal state is in the memory
-        // and the dialog is only the view!
-        /*
-        int i;
-        char s[MAXEXPR];
-        for(i = 0; i < 8; ++i)
-                slider[i] = (value_type)(GETSLIDERVALUE(dp,FIRSTCTLITEM+i));
-        if(!gdata->standalone)
-                for(i = 0; i < 4; ++i){
-                        // stash expression strings
-                        if(GETCTLTEXT(dp,FIRSTEXPRITEM+i,s,MAXEXPR)){ // cchMax: NULL is included, so MAXEXPR is correct
-                                if(expr[i])
-                                        free(expr[i]);
-                                expr[i] = _strdup(s);
-                        }
-                        if(!expr[i])
-                                expr[i] = _strdup("c");
-                }
-        */
+}
 struct node *updateexpr(DIALOGREF dp,int item){
         char s[MAXEXPR];
         int i;
         i = item - FIRSTEXPRITEM;
         freetree(tree[i]);
-        if(!gdata->standalone){
+        if(!gdata->parm.standalone){
                 GETCTLTEXT(dp,item,s,MAXEXPR); // cchMax: NULL is included, so MAXEXPR is correct
-                if(expr[i])
-                        free(expr[i]);
-                expr[i] = _strdup(s);
+                strcpy(gdata->parm.szFormula[i], s);
+        }
-        tree[i] = parseexpr(expr[i]);
+        tree[i] = parseexpr(gdata->parm.szFormula[i]);
-        if(!gdata->standalone){
+        if(!gdata->parm.standalone){
                 if(tree[i])
                         HideDialogItem(dp,FIRSTICONITEM+i);
                 else{
                         err[i] = errstr;
                         errstart[i] = tokstart;
+        }
         return result;
+}
-int checksliders(int exprs,int ctlflags[],int mapflags[]){
+int checksliders(int exprs){
         int i, result;
         result = 0;
         for(i = 4; i--;)
-                mapflags[i] = 0;
+                gdata->parm.map_used[i] = 0;
         for(i = 8; i--;)
-                ctlflags[i] = 0;
+                gdata->parm.ctl_used[i] = 0;
         for(i = 0; i < exprs; i++)
-                result |= _checksl(tree[i],ctlflags,mapflags);
+                result |= _checksl(tree[i], gdata->parm.ctl_used, gdata->parm.map_used);
         return result;
+}
 void slidermoved(DIALOGREF dp,int i){
         int v = GETSLIDERVALUE(dp,i);
         if (v < 0) v = 0;
         else if (v > 255) v = 255;
         i -= FIRSTCTLITEM;
-        slider[i] = (uint8_t)v;
+        gdata->parm.val[i] = (uint8_t)v;
         SETCTLTEXTINT(dp,i+FIRSTCTLTEXTITEM,v,false);
+}
 void slidertextchanged(DIALOGREF dp,int i){
         int v = GETCTLTEXTINT(dp,i,NULL,false);
         if (v < 0) v = 0;
         else if (v > 255) v = 255;
         i -= FIRSTCTLTEXTITEM;
         SETSLIDERVALUE(dp,i+FIRSTCTLITEM,v);
-        slider[i] = (uint8_t)v;
+        gdata->parm.val[i] = (uint8_t)v;
+}
 void maindlgupdate(DIALOGREF dp){
-        int i,unknown,ctls[8],maps[4];
+        int i,unknown;
-        unknown = checksliders(nplanes,ctls,maps);
+        unknown = checksliders(nplanes);
         for(i = 0; i < 8; i++)
-                if(unknown || ctls[i]){
+                if(unknown || gdata->parm.ctl_used[i]){
                         ENABLEDLGITEM(dp,FIRSTCTLITEM+i); // TODO: slider is still shown as disabled
                         REPAINTCTL(dp, FIRSTCTLITEM+i); // required for PLUGIN.DLL sliders
                         ENABLEDLGITEM(dp,FIRSTCTLLABELITEM+i);
                         ShowDialogItem(dp,FIRSTCTLTEXTITEM+i); /* FIXME: this changes keyboard focus */
                 }else{
         for(i = 0; i < nplanes; i++)
                 if(!tree[i]){
                         /* uh oh, couldn't parse one of the saved expressions...this is fatal */
                         DISABLEDLGITEM(dp,IDOK);
-                        if(gdata->standalone){
+                        if(gdata->parm.standalone){
                                 // TODO: But before this happens, we get filterBadParameters in filterSelectorStart, since setup() failed
                                 //       so, do we need this message here at all?
                                 simplealert_id(MSG_SAVED_EXPR_ERR_ID);
                         }else{
                                 DISABLEDLGITEM(dp,SAVEITEM);
         updateglobals(dp);
         if(setup(gpb))
                 recalc_preview(gpb,dp);
         ENABLEDLGITEM(dp,IDOK);
-        if(!gdata->standalone){
+        if(!gdata->parm.standalone){
                 ENABLEDLGITEM(dp,SAVEITEM);
                 ENABLEDLGITEM(dp,MAKEITEM);
                 ENABLEDLGITEM(dp,HELPITEM);
+        }
+}
                 "CMYK", "HSL", "HSB", "1234",
                 "DA", "LabA"
         };
         /* hide unused expression items */
-        if(gdata->standalone){
+        if(gdata->parm.standalone){
                 strcpy_win_replace_ampersand(&s[0], &gdata->parm.szAuthor[0]);
                 SETCTLTEXT(dp,PARAMAUTHORITEM,s);
                 strcpy_win_replace_ampersand(&s[0], &gdata->parm.szCopyright[0]);
                 SETCTLTEXT(dp,PARAMCOPYITEM,s);
                 strcpy_advance_id(&tmp1, MSG_ALL_FILES_ID);
                 strcpy_advance(&tmp1, (TCHAR*)TEXT(" (*.*)")); tmp1++;
                 strcpy_advance(&tmp1, (TCHAR*)TEXT("*.*")); tmp1++;
-                loadDlgRet = !gdata->standalone && choosefiletypes(
+                loadDlgRet = !gdata->parm.standalone && choosefiletypes(
 #ifdef MAC_ENV
                         "\pChoose filter settings", // "\p" means "Pascal string" // TODO (Not important yet): TRANSLATE
                         &sfr, &reply, types, 2,
                         filters
 #else
                 strcpy_advance_id(&tmp1, MSG_ALL_FILES_ID);
                 strcpy_advance(&tmp1, (TCHAR*)TEXT(" (*.*)")); tmp1++;
                 strcpy_advance(&tmp1, (TCHAR*)TEXT("*.*")); tmp1++;
-                saveDlgRet = !gdata->standalone && putfile(
+                saveDlgRet = !gdata->parm.standalone && putfile(
 #ifdef MAC_ENV
                         "\pSave filter settings", // "\p" means "Pascal string" // TODO (Not important yet): TRANSLATE
                         "\0",
                         TEXT_FILETYPE, SIG_SIMPLETEXT, &reply, &sfr,
                         "afs",
+                }
                 break;
+        }
         case MAKEITEM:
-                if (gdata->standalone) return true; // should not happen since the button should be grayed out
+                if (gdata->parm.standalone) return true; // should not happen since the button should be grayed out
                 builddialog(gpb);
                 break;
         case HELPITEM:

Subversion Repositories filter_foundry

filter_foundry/trunk/ui.c – Rev 544 → 550