commit 903cc9916869a2e4cb696f1de9474b73c2ed0ce0
author Christian Georg Becker <krischan@flunzel.de> Sat Jun 24 15:54:36 2017 +0200
committer Dominik Schäfer <zesstra@zesstra.de> Sun Jun 25 18:08:02 2017 +0200
tree 292bfddb8c2413e67869fd476dd5e4d302d9c1af
parent a87519059a9628014821d02c7bf5ffe93240004a

AddCmd: Nach Warnungen auch abbrechen und nicht weitere Warnungen generieren

* durch die Umwandlung der harten raise_error in gefangene Warnungen
  muss jetzt haendisch an den richtigen Stellen ein Nullwert fuer die
  Cache-Closure zurueckgegeben werden

Change-Id: I109da020eb6f036074317bddb848c5f183427bb4

std/thing/commands.c

diff --git a/std/thing/commands.c b/std/thing/commands.c
index 5cb9673..f684288 100644
--- a/std/thing/commands.c
+++ b/std/thing/commands.c
@@ -146,36 +146,35 @@
 }
 
 private closure _check_stringmethod(mixed func, int ex, string resp) {
-  closure cl;
-  if(!function_exists(func))
+  closure cl = symbol_function(func, this_object());
+  if(!cl)
+  {
     catch(raise_error(sprintf(
       resp+"string-Methode '%s' fehlt oder ist private.\n", func));
       publish);
+    return 0;
+  }
 
   // extern erstellte auf Sichtbarkeit pruefen und abbrechen/davor warnen
   if(ex) {
     mixed *fl = functionlist(this_object(), RETURN_FUNCTION_NAME|
                                             RETURN_FUNCTION_FLAGS);
     int index = member(fl, func)+1;
-    if(index<=0 || sizeof(fl)<=index)
-      catch(raise_error(sprintf(
-        resp+"string-Methode '%s' nicht in functionlist.\n", func));
-        publish);
-    else if(fl[index]&TYPE_MOD_PROTECTED || fl[index]&TYPE_MOD_STATIC)
+    if(index<=0 || sizeof(fl)<=index) // sollte nicht passieren, s.o.
+      raise_error(sprintf(
+        resp+"string-Methode '%s' trotz function_exists() nicht in "
+             "functionlist. WTF?\n", func));
+    else if(fl[index]&TYPE_MOD_PROTECTED || fl[index]&TYPE_MOD_STATIC) {
       catch(raise_error(sprintf(
         resp+"string-Methode '%s' ist protected/static. Extern "
         "definiertes Kommando darf so eine Methode nicht aufrufen!\n",
         func));
         publish);
+      return 0;
+    }
   }
 
   // ansonsten Methode erstmal cachen
-  cl = symbol_function(func, this_object());
-  if(!cl)
-    catch(raise_error(sprintf(
-      resp+"string-Methode '%s' nicht in Closure wandelbar?\n", func));
-      publish);
-
   return cl;
 }
 
@@ -334,19 +333,19 @@
     while(j--) {
      int k;
      // DBG(rule[j]);
-    	// Regeln nicht löschen und Regel?
+    	// Regeln nicht löschen und Regel?
      if(!(del_norule && pointerp(rule[j])) &&
-        // nur bestimmte ID löschen und ID passt nicht?
+        // nur bestimmte ID löschen und ID passt nicht?
         !(onlyid && (!pointerp(ids) || sizeof(ids)<=j || ids[j]!=onlyid)) &&
-        // Löschregel existiert und passt nicht auf Regel?
+        // Löschregel existiert und passt nicht auf Regel?
         !(delrule && (k=sizeof(rule[j]))!=sizeof(delrule))) {
-      // partielles Testen einer Löschregel ...
+      // partielles Testen einer Löschregel ...
       if(delrule) {
        while(k--)
         if(!sizeof(rule[j][k]&delrule[k])) break;
        if(k>=0) continue;
       }
-      // alles korrekt: Löschen!
+      // alles korrekt: Löschen!
       // (Arraybereich durch leeres Array loeschen)
       flag[j..j]    = allocate(0);
       fun[j..j]     = allocate(0);
@@ -465,7 +464,7 @@
      while((range=r_end-q_start)>=0) {
       mixed tmpobj;
 
-      // wie weit wollen wir zurückgehen?
+      // wie weit wollen wir zurückgehen?
       if(range)
         if(!(check_present&CHECK_PUTGET))
           range=range>2?2:range; // 3 Fragmente fuer @ID/@PRESENT (Adverb/Nr)