Changeset 551

Timestamp:

07/30/16 03:19:31 (8 years ago)

Author:

Maciej Komosinski

Message:

PreconfiguredGenetics? warns about unexpected use case and unregisters its objects on destruction

File:

• cpp/frams/genetics/preconfigured.h (modified) (1 diff)

cpp/frams/genetics/preconfigured.h

@@ -24 +24 @@
         {
         gcm.addDefaultConverters(); //without converters, the application would only handle "format 0" genotypes
-        Geno::useConverters(&gcm);
-
-        Geno::useValidators(&validators);
+        if (Geno::useConverters(&gcm)!=NULL)
+                logPrintf("PreconfiguredGenetics", "init", LOG_WARN, "Geno converters already configured"); //someone is using multiple PreconfiguredGenetics objects? (or other potentially unsafe configuration)
+        if (Geno::useValidators(&validators)!=NULL)
+                logPrintf("PreconfiguredGenetics", "init", LOG_WARN, "Geno validators already configured");
         validators+=&genman; //primary validation: use the extended validity checking (through dedicated genetic operators)
         validators+=&model_validator; //secondary validation: this simple validator handles all cases when there is no dedicated genetic validation operator, but a converter for a particular format is available. Converters may be less strict in detecting invalid genotypes but using them and checking whether they produced a valid f0 genotype is also some way to tell whether the initial genotype was valid. Otherwise, without dedicated genetic validation operator, we would have no validity check at all.
+        }
+
+~PreconfiguredGenetics()
+        {
+        Geno::useConverters(NULL);
+        Geno::useValidators(NULL);
         }
 };