X-Git-Url: http://winboard.nl/cgi-bin?a=blobdiff_plain;f=main.c;h=ae3c5aaf4a7e87cb45e8b82ede8eda8f34d0df0a;hb=5b56e51eeae04c352b95b1706b9d69b61feba3e9;hp=fad2ed8b9a806b21371342d105d0d1191915a39b;hpb=cb9522491af43508c47cb927247e3b5769b9259b;p=polyglot.git

diff --git a/main.c b/main.c
index fad2ed8..ae3c5aa 100644
--- a/main.c
+++ b/main.c
@@ -7,6 +7,7 @@
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
+#include <time.h>
 
 #include "attack.h"
 #include "board.h"
@@ -32,11 +33,13 @@
 #include "xboard2uci.h"
 #include "uci2uci.h"
 #include "ini.h"
+#include "util.h"
+
 
 // constants
 
 
-static const char * const Version = "1.4.42b";
+static const char * const Version = "1.4.53b";
 static const char * const HelpMessage = "\
 SYNTAX\n\
 * polyglot [configfile] [-noini] [-ec engine] [-ed enginedirectory] [-en enginename] [-log] [-lf logfile] [-hash value] [-bk book] [-pg <name>=<value>]* [-uci <name>=<value>]*\n\
@@ -58,7 +61,6 @@ static bool Init;
 
 // prototypes
 
-static void init_book ();
 static void stop_search  ();
 
 // functions
@@ -72,62 +74,209 @@ static void arg_shift_left(char **argv, int index){
     }
 }
 
+// parse_args()
+
+static void parse_args(ini_t *ini, char **argv){
+    int arg_index;
+    char *arg;
+    arg_index=0;
+    while((arg=argv[arg_index])){
+        if(my_string_equal(arg,"-ec") && argv[arg_index+1]){
+            ini_insert_ex(ini,"PolyGlot","EngineCommand",argv[arg_index+1]);
+            arg_shift_left(argv,arg_index);
+            arg_shift_left(argv,arg_index);
+            continue;
+        }if(my_string_equal(arg,"-ed") && argv[arg_index+1]){
+            ini_insert_ex(ini,"PolyGlot","EngineDir",argv[arg_index+1]);
+            arg_shift_left(argv,arg_index);
+            arg_shift_left(argv,arg_index);
+            continue;
+        }
+        if(my_string_equal(arg,"-en") && argv[arg_index+1]){
+            ini_insert_ex(ini,"PolyGlot","EngineName",argv[arg_index+1]);
+            arg_shift_left(argv,arg_index);
+            arg_shift_left(argv,arg_index);
+            continue;
+        }
+        if(my_string_equal(arg,"-log") &&
+           argv[arg_index+1] &&
+           IS_BOOL(argv[arg_index+1])){
+            ini_insert_ex(ini,
+                          "PolyGlot",
+                          "Log",
+                          TO_BOOL(argv[arg_index+1])?"true":"false");
+            arg_shift_left(argv,arg_index);
+            arg_shift_left(argv,arg_index);
+            continue;
+        }
+        if(my_string_equal(arg,"-lf") && argv[arg_index+1]){
+            ini_insert_ex(ini,"PolyGlot","LogFile",argv[arg_index+1]);
+            arg_shift_left(argv,arg_index);
+            arg_shift_left(argv,arg_index);
+            continue;
+        }
+        if(my_string_equal(arg,"-wb") &&
+           argv[arg_index+1]&&
+           IS_BOOL(argv[arg_index+1])){
+               ini_insert_ex(ini,"PolyGlot",
+                             "OnlyWbOptions",
+                             TO_BOOL(argv[arg_index+1])?"true":"false");
+               arg_shift_left(argv,arg_index);
+               arg_shift_left(argv,arg_index);
+               continue;
+        }
+        if((my_string_equal(arg,"-pg")||my_string_equal(arg,"-uci")) &&
+           argv[arg_index+1]){
+            int ret;
+            char section[StringSize];
+            char name[StringSize];
+            char value[StringSize];
+            ret=ini_line_parse(argv[arg_index+1],section,name,value);
+            if(ret==NAME_VALUE){
+                if(my_string_equal(arg,"-pg")){
+                    ini_insert_ex(ini,"PolyGlot",name,value);
+                }else{
+                    ini_insert_ex(ini,"Engine",name,value);
+                }
+            }
+            arg_shift_left(argv,arg_index);
+            arg_shift_left(argv,arg_index);
+            continue;
+        }
+        arg_index++;
+    }
+}
+
+
+// make_ini()
+
+static void make_ini(ini_t *ini){
+    option_t *opt;
+    ini_insert_ex(ini,"polyglot",
+		  "EngineCommand",
+		  option_get(Option,"EngineCommand"));
+    ini_insert_ex(ini,"polyglot",
+		  "EngineDir",
+		  option_get(Option,"EngineDir"));
+    option_start_iter(Option);
+    while((opt=option_next(Option))){
+        if(my_string_case_equal(opt->name,"SettingsFile")) continue;
+        if(my_string_case_equal(opt->name,"EngineCommand")) continue;
+        if(my_string_case_equal(opt->name,"EngineDir")) continue;
+        if(!my_string_equal(opt->value,opt->default_)&& !IS_BUTTON(opt->type))
+        {
+            ini_insert_ex(ini,"polyglot",opt->name,opt->value);
+        }
+    }
+    option_start_iter(Uci->option);
+    while((opt=option_next(Uci->option))){
+        if(!strncmp(opt->name,"UCI_",4) &&
+            !my_string_case_equal(opt->name,"UCI_LimitStrength") &&
+            !my_string_case_equal(opt->name,"UCI_Elo"))continue;
+        if(!my_string_equal(opt->value,opt->default_)&&
+           !IS_BUTTON(opt->type)){
+            ini_insert_ex(ini,"engine",opt->name,opt->value);
+        }
+    }
+}
+
+
 // write_ini()
 
 static void write_ini(const char *filename,
-                      option_list_t *pg_options,
-                      option_list_t *uci_options){
-    option_t *opt;
+			 ini_t *ini){
+    const char *quote;
+    ini_entry_t *entry;
     char tmp[StringSize];
     FILE *f;
+    time_t t=time(NULL);
     f=fopen(filename,"w");
     if(!f){
-        my_fatal("ini_create_pg(): Cannot open %s for writing.\n",filename);
+      gui_send(GUI,"tellusererror write_ini(): %s: %s.",filename,strerror(errno));
+      my_log("POLYGLOT write_ini(): %s: %s.\n",filename,strerror(errno));
+      return;
     }
-    fprintf(f,"; You may edit this file to set options for the\n"
-              "; UCI engine whose PolyGlot name is %s.\n"
-              "; You may also safely delete this file\n"
-              "; to restore the default options.\n",
-            option_get_string(Option,"EngineName"));
+    fprintf(f,"; Created: %s\n",ctime(&t));
     fprintf(f,"[PolyGlot]\n");
-    option_start_iter(pg_options);
-    while((opt=option_next(pg_options))){
-        if(!my_string_equal(opt->value,opt->default_)&&
-           !my_string_case_equal(opt->type,"button") &&
-           (opt->mode & XBOARD)){
-            snprintf(tmp,sizeof(tmp),"%s=%s\n",opt->name,opt->value);
-            tmp[sizeof(tmp)-1]='\0';
-            fprintf(f,"%s",tmp);
-        }
+    ini_start_iter(ini);
+    while((entry=ini_next(ini))){
+      if(my_string_case_equal(entry->section,"polyglot")){
+          if(strchr(entry->value,';')|| strchr(entry->value,'#')|| strchr(entry->value,'"')){
+              quote="\"";
+          }else{
+              quote="";
+          }
+          snprintf(tmp,sizeof(tmp),"%s=%s%s%s\n",
+                   entry->name,
+                   quote,
+                   entry->value,
+                   quote);
+	tmp[sizeof(tmp)-1]='\0';
+	fprintf(f,"%s",tmp);
+      }
     }
     fprintf(f,"[Engine]\n");
-    option_start_iter(uci_options);
-    while((opt=option_next(uci_options))){
-        if(!my_string_equal(opt->value,opt->default_)&&
-           !my_string_case_equal(opt->type,"button")){
-            snprintf(tmp,sizeof(tmp),"%s=%s\n",opt->name,opt->value);
-            tmp[sizeof(tmp)-1]='\0';
-            fprintf(f,"%s",tmp);
-        }
+    ini_start_iter(ini);
+    while((entry=ini_next(ini))){
+        if(my_string_case_equal(entry->section,"engine")){
+            if(strchr(entry->value,';')|| strchr(entry->value,'#')|| strchr(entry->value,'"')){
+                quote="\"";
+            }else{
+                quote="";
+            }
+            snprintf(tmp,sizeof(tmp),"%s=%s%s%s\n",
+                     entry->name,
+                     quote,
+                     entry->value,
+                     quote);
+	tmp[sizeof(tmp)-1]='\0';
+	fprintf(f,"%s",tmp);
+      }
     }
     fclose(f);
 }
 
+// welcome_message()
+
+void welcome_message(char *buf){
+    if(!DEBUG){
+        sprintf(buf,
+                "PolyGlot %s by Fabien Letouzey.\n",
+                Version);
+    }else{
+        sprintf(buf,
+                "PolyGlot %s by Fabien Letouzey (debug build).\n",
+                Version);
+    }
+}
+
+int wb_select(){
+    option_t *opt;
+    option_start_iter(Option);
+    while((opt=option_next(Option))){
+        opt->mode&=~XBOARD;
+        if(opt->mode & XBSEL){
+            opt->mode|=XBOARD; 
+        }
+    }
+}
 
 // main()
 
 int main(int argc, char * argv[]) {
-    ini_t ini[1],ini_save[1];
+    ini_t ini[1], ini_command[1];
     ini_entry_t *entry;
     char *arg;
     int arg_index;
     bool NoIni;
+    option_t *opt;
+    char welcome[StringSize];
+
+
+    welcome_message(welcome);
  
-    if(!DEBUG){
-        printf("PolyGlot %s by Fabien Letouzey.\n",Version);
-    }else{
-        printf("PolyGlot %s by Fabien Letouzey (debug build).\n",Version);
-    }
+    printf("%s",welcome);
+
 
     if(argc>=2 && ((my_string_case_equal(argv[1],"help")) || (my_string_case_equal(argv[1],"-help")) || (my_string_case_equal(argv[1],"--help")) ||  (my_string_case_equal(argv[1],"-h")) ||  my_string_case_equal(argv[1],"/?"))){
         printf("%s\n",HelpMessage);
@@ -150,9 +299,9 @@ int main(int argc, char * argv[]) {
     my_random_init();
 
     ini_init(ini);
-    ini_init(ini_save);
+    ini_init(ini_command);
 
-        // book utilities
+        // book utilities: do not touch these
     
     if (argc >= 2 && my_string_equal(argv[1],"make-book")) {
         book_make(argc,argv);
@@ -180,9 +329,6 @@ int main(int argc, char * argv[]) {
         do_perft(argc,argv);
         return EXIT_SUCCESS;
     }
-
-        // TODO: If logging is enabled on the command line turn it on NOW
-        // and do not allow it to be overridden later. 
     
         // What is the config file? This is very hacky right now.
 
@@ -197,8 +343,9 @@ int main(int argc, char * argv[]) {
         }
     }
     arg_shift_left(argv,arg_index-1);
+    parse_args(ini_command,argv+1);
     if(NoIni){
-        option_set(Option,"OptionFile","<empty>");
+        option_set(Option,"SettingsFile","<empty>");
     }
 
         // Ok see if first argument looks like config file
@@ -206,7 +353,7 @@ int main(int argc, char * argv[]) {
     if(argv[1] && !my_string_equal(argv[1],"epd-test") && !(argv[1][0]=='-')){
                 // first argument must be  config file
         if(!NoIni){
-            option_set(Option,"OptionFile",argv[1]);
+            option_set(Option,"SettingsFile",argv[1]);
         }else{
                 // ignore
         }
@@ -215,33 +362,59 @@ int main(int argc, char * argv[]) {
             // Config file is the default.
             // This has already been set above or in "option_init_pg()"
     }
-    
+
+
 
         // if we use a config file: load it!
     
-    if(!my_string_equal(option_get_string(Option,"OptionFile"),"<empty>")){
-        if(ini_parse(ini,option_get_string(Option,"OptionFile"))){
+    if(!my_string_equal(option_get_string(Option,"SettingsFile"),"<empty>")){
+        if(ini_parse(ini,option_get_string(Option,"SettingsFile"))){
             my_fatal("main(): Can't open file \"%s\": %s\n",
-                   option_get_string(Option,"OptionFile"),
+                   option_get_string(Option,"SettingsFile"),
                    strerror(errno));
         }
     }
-        // remind the reader of what options are in effect
 
-    my_log("POLYGLOG Options from ini file\n");
-    ini_disp(ini);
+        // Extract some important options
 
-        // extract PG options
+    if((entry=ini_find(ini,"polyglot","EngineCommand"))){
+        option_set(Option,entry->name,entry->value);
+    }
+    if((entry=ini_find(ini,"polyglot","EngineDir"))){
+        option_set(Option,entry->name,entry->value);
+    }
+    if((entry=ini_find(ini,"polyglot","EngineName"))){
+        option_set(Option,entry->name,entry->value);
+    }
+    if((entry=ini_find(ini,"polyglot","Log"))){
+        polyglot_set_option(entry->name,entry->value);
+    }
+    if((entry=ini_find(ini,"polyglot","LogFile"))){
+        polyglot_set_option(entry->name,entry->value);
+    }
     
-    ini_start_iter(ini);
-    while((entry=ini_next(ini))){
-        if(my_string_case_equal(entry->section,"polyglot")){
-            option_set(Option,entry->name,entry->value);
-            option_set_default(Option,entry->name,entry->value);
+        // Concession to WB 4.4.0
+        // Treat "polyglot_1st.ini" and "polyglot_2nd.ini" specially
+
+    if(option_get_bool(Option,"WbWorkArounds3")){
+        const char *SettingsFile=option_get(Option,"SettingsFile");
+        if(strstr(SettingsFile,"polyglot_1st.ini")||
+           strstr(SettingsFile,"polyglot_2nd.ini")){
+            option_set(Option,"SettingsFile","<empty>");
         }
     }
 
-        // start logging if required
+        // Look at command line for logging option. It is important
+        // to start logging as soon as possible.
+
+     if((entry=ini_find(ini_command,"PolyGlot","Log"))){
+         option_set(Option,entry->name,entry->value);
+    }
+    if((entry=ini_find(ini_command,"PolyGlot","LogFile"))){
+        option_set(Option,entry->name,entry->value);
+    }
+    
+       // start logging if required
     
     if (option_get_bool(Option,"Log")) {
         my_log_open(option_get_string(Option,"LogFile"));
@@ -249,41 +422,24 @@ int main(int argc, char * argv[]) {
 
         // log welcome stuff
     
-    if(!DEBUG){
-        my_log("PolyGlot %s by Fabien Letouzey\n",Version);
-    }else{
-        my_log("PolyGlot %s by Fabien Letouzey (debug build)\n",Version);
-    }    
+    my_log("%s",welcome);
     my_log("POLYGLOT *** START ***\n");
-    my_log("POLYGLOT INI file \"%s\"\n",option_get_string(Option,"OptionFile"));
+    if(!my_string_equal(option_get_string(Option,"SettingsFile"),"<empty>")){
+        my_log("POLYGLOT INI file \"%s\"\n",option_get_string(Option,"SettingsFile"));
+    }
 
-        // open book (presumably this should go else where)
-    
-    init_book();
 
         // scavenge command line for options necessary to start the engine
-    
-    arg_index=1;
-    while((arg=argv[arg_index])){
-        if(my_string_equal(arg,"-ec") && argv[arg_index+1]){
-            option_set(Option,"EngineCommand",argv[arg_index+1]);
-            arg_shift_left(argv,arg_index);
-            arg_shift_left(argv,arg_index);
-            continue;
-        }
-        if(my_string_equal(arg,"-ed") && argv[arg_index+1]){
-            option_set(Option,"EngineDir",argv[arg_index+1]);
-            arg_shift_left(argv,arg_index);
-            arg_shift_left(argv,arg_index);
-            continue;
-        }
-        if(my_string_equal(arg,"-en") && argv[arg_index+1]){
-            option_set(Option,"EngineName",argv[arg_index+1]);
-            arg_shift_left(argv,arg_index);
-            arg_shift_left(argv,arg_index);
-            continue;
-        }
-        arg_index++;
+
+
+    if((entry=ini_find(ini_command,"PolyGlot","EngineCommand"))){
+        option_set(Option,entry->name,entry->value);
+    }
+    if((entry=ini_find(ini_command,"PolyGlot","EngineDir"))){
+        option_set(Option,entry->name,entry->value);
+    }
+    if((entry=ini_find(ini_command,"PolyGlot","EngineName"))){
+        option_set(Option,entry->name,entry->value);
     }
 
         // start engine
@@ -299,101 +455,83 @@ int main(int argc, char * argv[]) {
         my_log("POLYGLOT *** Switching to UCI mode ***\n");
     }
 
-        // initialize uci parsing and send uci command. Parse options and wait
-        // for uciok
+        // initialize uci parsing and send uci command. 
+        // Parse options and wait for uciok
     
     uci_open(Uci,Engine);
 
-        // get engine name from engine if not supplied in config file
-    
+    option_set_default(Option,"EngineName",Uci->name);
+
+        // get engine name from engine if not supplied in config file or on
+        // the command line
+
     if (my_string_equal(option_get_string(Option,"EngineName"),"<empty>")) {
         option_set(Option,"EngineName",Uci->name);
     }
 
-        // what is the name of the persist file?
 
-    if(my_string_equal(option_get_string(Option,"PersistFile"),"<empty>")){
+        // In the case we have been invoked with NoIni or StandardIni
+        // we still have to load a config file.
+
+    if(my_string_equal(option_get_string(Option,"SettingsFile"),"<empty>")){
+
+            //  construct the name of the ConfigFile from the EngineName
+        
         char tmp[StringSize];
-        snprintf(tmp,sizeof(tmp),"PG_%s.ini",
+        char option_file[StringSize];
+        int i;
+        snprintf(tmp,sizeof(tmp),"%s.ini",
                  option_get_string(Option,"EngineName"));
         tmp[sizeof(tmp)-1]='\0';
-        option_set(Option,"PersistFile",tmp);
-    }
-
-        // if "Persist" is true, load the persist file!
-    
-    if(option_get_bool(Option,"Persist")){
-        my_log("POLYGLOT PersistFile=%s\n",option_get_string(Option,"PersistFile"));
-        if(ini_parse(ini_save,option_get_string(Option,"PersistFile"))){
-            my_log("POLYGLOT Unable to open PersistFile\n"); 
+        for(i=0;i<strlen(tmp);i++){
+            if(tmp[i]==' '){
+                tmp[i]='_';
+            }
+        }
+        my_path_join(option_file,
+                     option_get_string(Option,"SettingsDir"),
+                     tmp);
+    // Load the config file
+        option_set(Option,"SettingsFile",option_file);
+
+        my_log("POLYGLOT INI file \"%s\"\n",option_get_string(Option,"SettingsFile"));
+        if(ini_parse(ini,option_file)){
+            my_log("POLYGLOT Unable to open %s\n",
+                   option_get_string(Option,"SettingsFile")); 
         }
     }
-    
-        // parse the command line and merge remaining options
 
-    arg_index=1;
-    while((arg=argv[arg_index])){
-        if(my_string_equal(arg,"-log")){
-            ini_insert_ex(ini_save,"PolyGlot","Log","true");
-            arg_shift_left(argv,arg_index);
-            continue;
-        }
-        if(my_string_equal(arg,"-lf") && argv[arg_index+1]){
-            ini_insert_ex(ini_save,"PolyGlot","LogFile",argv[arg_index+1]);
-            arg_shift_left(argv,arg_index);
-            arg_shift_left(argv,arg_index);
-            continue;
-        }
-        if(my_string_equal(arg,"-hash") && argv[arg_index+1]){
-            ini_insert_ex(ini_save,"Engine","Hash",argv[arg_index+1]);
-            arg_shift_left(argv,arg_index);
-            arg_shift_left(argv,arg_index);
-            continue;
-        }
-        if(my_string_equal(arg,"-bk") && argv[arg_index+1]){
-            ini_insert_ex(ini_save,"PolyGlot","Book","true");
-            ini_insert_ex(ini_save,"PolyGlot","BookFile",argv[arg_index+1]);
-            arg_shift_left(argv,arg_index);
-            arg_shift_left(argv,arg_index);
-            continue;
-        }
-        if((my_string_equal(arg,"-pg")||my_string_equal(arg,"-uci")) &&
-           argv[arg_index]){
-            int ret;
-            char section[StringSize];
-            char name[StringSize];
-            char value[StringSize];
-            ret=ini_line_parse(argv[arg_index++],section,name,value);
-            if(ret==NAME_VALUE){
-                if(my_string_equal(arg,"-pg")){
-                    ini_insert_ex(ini_save,"PolyGlot",name,value);
-                }else{
-                    ini_insert_ex(ini_save,"Engine",name,value);
-                }
-            }
-            arg_shift_left(argv,arg_index);
-            arg_shift_left(argv,arg_index);
-            continue;
-        }
-        arg_index++;
+
+    // Parse the command line and merge remaining options.
+
+    ini_start_iter(ini_command);
+    while((entry=ini_next(ini_command))){
+        ini_insert(ini,entry);
     }
 
-        // remind the reader once again about options
+        // Remind the reader about the options that are now in effect.
 
-    my_log("POLYGLOG Options from save file and command line\n");
-    ini_disp(ini_save);
+    my_log("POLYGLOG OPTIONS \n");
+    ini_disp(ini);
 
-        // extract PG options; this time do not set the default
-        // polyglot_set_option() performs the necessary actions such
-        // as opening the log file/opening book etcetera.
+            // extract PG options
     
-    ini_start_iter(ini_save);
-    while((entry=ini_next(ini_save))){
+    ini_start_iter(ini);
+    while((entry=ini_next(ini))){
         if(my_string_case_equal(entry->section,"polyglot")){
-            polyglot_set_option(entry->name,entry->value);
+            opt=option_find(Option,entry->name);
+            if(opt && !IS_BUTTON(opt->type)){
+                polyglot_set_option(entry->name,entry->value);
+            }
         }
     }
 
+        // Cater to our biggest customer:-)
+    
+    if(option_get_bool(Option,"OnlyWbOptions")){
+        wb_select();
+    }
+
         // done initializing
     
     Init = TRUE;
@@ -405,28 +543,10 @@ int main(int argc, char * argv[]) {
         if(my_string_case_equal(entry->section,"engine")){
                 // also updates value in Uci->option
             uci_send_option(Uci,entry->name,"%s",entry->value);
-                // since this comes from the ini file, also update default
-            option_set_default(Uci->option,entry->name,entry->value);
-                // this is inherited, it probably does not work correctly
-           if(my_string_case_equal(entry->name,"MultiPV") &&
-              atoi(entry->value)>1){
-               Uci->multipv_mode=TRUE;
-           }
         }
     }
-    ini_start_iter(ini_save);
-    while((entry=ini_next(ini_save))){
-        if(my_string_case_equal(entry->section,"engine")){
-                // also updates value in Uci->option
-            uci_send_option(Uci,entry->name,"%s",entry->value);
-                // this is inherited, it probably does not work correctly
-            if(my_string_case_equal(entry->name,"MultiPV") &&
-               atoi(entry->value)>1){
-                Uci->multipv_mode=TRUE;
-            }
-        }
-    }
-   
+
+    
     
         // EPD test
     
@@ -438,12 +558,16 @@ int main(int argc, char * argv[]) {
     }
     
         // Anything that hasn't been parsed yet is a syntax error
+        // It seems that XBoard sometimes passes empty strings as arguments
+        // to PolyGlot. We ignore these. 
 
-    if(argv[1]){
-        my_fatal("main(): Unknown option: %s\n",argv[1]);
+    argc=1;
+    while((arg=argv[argc++])){
+        if(!my_string_equal(arg,"")){
+            my_fatal("main(): Option: \"%s\" not found\n",argv[argc-1]);
+        }
     }
 
-
     gui_init(GUI);
     mainloop();
     return EXIT_SUCCESS; 
@@ -452,7 +576,27 @@ int main(int argc, char * argv[]) {
 // polyglot_set_option()
 
 void polyglot_set_option(const char *name, const char *value){ // this must be cleaned up!
+    ini_t ini[1];
+    int ret;
+    ini_init(ini);
     my_log("POLYGLOT Setting PolyGlot option %s=\"%s\"\n",name,value);
+    if(my_string_case_equal(name,"Save")){
+        ret=my_mkdir(option_get(Option,"SettingsDir"));
+        if(ret){
+            my_log("POLYGLOT polyglot_set_option(): %s: %s\n",
+                   option_get(Option,"SettingsDir"),
+                   strerror(errno));
+        }
+        make_ini(ini);
+        write_ini(option_get(Option,"SettingsFile"),ini);
+        return;
+    }
+//    if(my_string_equal(option_get(Option,name),value)){
+//        my_log("Not setting PolyGlot option \"%s\" "
+//               "since it already as the correct value.\n",
+//               name);
+//        return;
+//    }
     option_set(Option,name,value);
     if(option_get_bool(Option,"Book")&&(my_string_case_equal(name,"BookFile")||my_string_case_equal(name,"Book"))){
         my_log("POLYGLOT *** SETTING BOOK ***\n");
@@ -464,8 +608,8 @@ void polyglot_set_option(const char *name, const char *value){ // this must be c
             my_log("POLYGLOT Unable to open book \"%s\"\n",option_get_string(Option,"BookFile"));
         }
     }else if(option_get_bool(Option,"Log")&&(my_string_case_equal(name,"LogFile") ||my_string_case_equal(name,"Log"))){
-        my_log("POLYGLOT *** SETTING LOGFILE ***\n");
-        my_log("POLYGLOT LOGFILE \"%s\"\n",option_get_string(Option,"LogFile"));
+        my_log("POLYGLOT *** SWITCHING LOGFILE ***\n");
+        my_log("POLYGLOT NEW LOGFILE \"%s\"\n",option_get_string(Option,"LogFile"));
         my_log_close();
         my_log_open(option_get_string(Option,"LogFile"));
     }else if(option_get_bool(Option,"UseNice") &&(my_string_case_equal(name,"NiceValue")||my_string_case_equal(name,"UseNice"))){
@@ -484,21 +628,6 @@ void polyglot_set_option(const char *name, const char *value){ // this must be c
 }
 
 
-// init_book()
-
-static void init_book(){
-    book_clear();
-    if (option_get_bool(Option,"Book")){
-        my_log("POLYGLOT *** SETTING BOOK ***\n");
-        my_log("POLYGLOT BOOK \"%s\"\n",option_get_string(Option,"BookFile"));
-        book_open(option_get_string(Option,"BookFile"));
-        if(!book_is_open()){
-            my_log("POLYGLOT Unable to open book \"%s\"\n",
-                   option_get_string(Option,"BookFile"));
-        }
-    }
-}
-
 
 // quit()
 
@@ -514,11 +643,7 @@ void quit() {
         engine_close(Engine);
         
     }
-    if(option_get_bool(Option,"Persist")){
-        write_ini(option_get_string(Option,"PersistFile"),
-                  Option,Uci->option);
-    }
-    my_log("POLYGLOT Calling exit\n");
+	my_log("POLYGLOT Calling exit\n");
     exit(EXIT_SUCCESS);
 }
 
@@ -542,5 +667,5 @@ static void stop_search() {
 }
 
 
-// end of main.cpp
+// end of main.c