cdo_operator_add now only required once per operator

src/EcaEtccdi.cc

@@ -798,32 +798,23 @@ public:
     if (cdo_operator_argc() > 4) cdo_abort("Too many arguments!");
 
     int TX90P, TX10P, TN90P, TN10P, R99P, R95P;
-    if (cdo_operator_argc() == 4 && 'm' == cdo_operator_argv(3)[0])
-      {
-        TX90P = cdo_operator_add("etccdi_tx90p", func_selge, CMP_MONTH, nullptr);  // monthly mode
-        R99P = cdo_operator_add("etccdi_r99p", func_selge, CMP_MONTH, nullptr);    // monthly mode
-        R95P = cdo_operator_add("etccdi_r95p", func_selge, CMP_MONTH, nullptr);    // monthly mode
-        TX10P = cdo_operator_add("etccdi_tx10p", func_selle, CMP_MONTH, nullptr);  // monthly mode
-        TN90P = cdo_operator_add("etccdi_tn90p", func_selge, CMP_MONTH, nullptr);  // monthly mode
-        TN10P = cdo_operator_add("etccdi_tn10p", func_selle, CMP_MONTH, nullptr);  // monthly mode
-        cdo_operator_add("etccdi", 0, CMP_MONTH, nullptr);                  // monthly mode
-      }
-    else
-      {
-        TX90P = cdo_operator_add("etccdi_tx90p", func_selge, CMP_DATE, nullptr);
-        R99P = cdo_operator_add("etccdi_r99p", func_selge, CMP_DATE, nullptr);
-        R95P = cdo_operator_add("etccdi_r95p", func_selge, CMP_DATE, nullptr);
-        TX10P = cdo_operator_add("etccdi_tx10p", func_selle, CMP_DATE, nullptr);
-        TN90P = cdo_operator_add("etccdi_tn90p", func_selge, CMP_DATE, nullptr);
-        TN10P = cdo_operator_add("etccdi_tn10p", func_selle, CMP_DATE, nullptr);
-        cdo_operator_add("etccdi", 0, CMP_DATE, nullptr);
-      }
+    TX90P = cdo_operator_add("etccdi_tx90p", func_selge, CMP_DATE, nullptr);
+    R99P = cdo_operator_add("etccdi_r99p", func_selge, CMP_DATE, nullptr);
+    R95P = cdo_operator_add("etccdi_r95p", func_selge, CMP_DATE, nullptr);
+    TX10P = cdo_operator_add("etccdi_tx10p", func_selle, CMP_DATE, nullptr);
+    TN90P = cdo_operator_add("etccdi_tn90p", func_selge, CMP_DATE, nullptr);
+    TN10P = cdo_operator_add("etccdi_tn10p", func_selle, CMP_DATE, nullptr);
+    cdo_operator_add("etccdi", 0, CMP_DATE, nullptr);
+
 
+    if (cdo_operator_argc() == 4 && 'm' == cdo_operator_argv(3)[0]) { request.compare_type = CMP_YEAR; }
 
     request.ndates = parameter_to_int(cdo_operator_argv(0));
     request.startboot = parameter_to_int(cdo_operator_argv(1));
 
     const auto operatorID = cdo_operator_id();
+    request.compare_type = cdo_operator_f2(cdo_operator_id());
+
     if (operatorID == TX90P || operatorID == TN90P || operatorID == R95P || operatorID == R99P)
       {
         if (operatorID == TX90P || operatorID == TN90P)