]> git.ipfire.org Git - thirdparty/plymouth.git/commitdiff
[script] Remove some asserts which are no longer useful
authorCharlie Brej <cbrej@cs.man.ac.uk>
Wed, 8 Jul 2009 21:48:08 +0000 (22:48 +0100)
committerRay Strode <rstrode@redhat.com>
Fri, 24 Jul 2009 13:30:27 +0000 (09:30 -0400)
src/plugins/splash/script/script-execute.c
src/plugins/splash/script/script-object.c

index 72d64e71261c93469993a2dc2d93f8767f83f5f0..9856b0ed161d036db601ec0ffa282d9ca21be4c6 100644 (file)
@@ -426,7 +426,7 @@ static script_obj_t *script_evaluate (script_state_t *state,
           return script_evaluate_func (state, exp);
         }
     }
-  assert (0);
+  return script_obj_new_null ();
 }
 
 static script_return_t script_execute_list (script_state_t *state,
index 29aefbe7485d6a71cff05c2a308d5337c000670f..7c12c1eb9eff61521574d70876813279916000cd 100644 (file)
@@ -220,10 +220,6 @@ char *script_obj_print (script_obj_t *obj)
                     obj->refcount);
           return reply;
         }
-
-      default:
-        printf ("unhandeled object type %d\n", obj->type);
-        assert (0);
     }
   return NULL;
 }
@@ -324,9 +320,7 @@ int script_obj_as_int (script_obj_t *obj)
       case SCRIPT_OBJ_TYPE_NULL:
         return 0;
 
-      case SCRIPT_OBJ_TYPE_REF:     /* should have been de-reffed already */
-        assert (0);
-
+      case SCRIPT_OBJ_TYPE_REF:
       case SCRIPT_OBJ_TYPE_HASH:
       case SCRIPT_OBJ_TYPE_FUNCTION:
       case SCRIPT_OBJ_TYPE_NATIVE:
@@ -335,8 +329,6 @@ int script_obj_as_int (script_obj_t *obj)
       case SCRIPT_OBJ_TYPE_STRING:
         return 0;
     }
-
-  assert (0);       /* Abort on uncaught */
   return 0;
 }
 
@@ -354,9 +346,7 @@ float script_obj_as_float (script_obj_t *obj)
       case SCRIPT_OBJ_TYPE_NULL:
         return NAN;
 
-      case SCRIPT_OBJ_TYPE_REF:     /* should have been de-reffed already */
-        assert (0);
-
+      case SCRIPT_OBJ_TYPE_REF:
       case SCRIPT_OBJ_TYPE_HASH:
       case SCRIPT_OBJ_TYPE_FUNCTION:
       case SCRIPT_OBJ_TYPE_NATIVE:
@@ -365,8 +355,6 @@ float script_obj_as_float (script_obj_t *obj)
       case SCRIPT_OBJ_TYPE_STRING:
         return NAN;
     }
-
-  assert (0);       /* Abort on uncaught */
   return NAN;
 }
 
@@ -386,9 +374,7 @@ bool script_obj_as_bool (script_obj_t *obj)
       case SCRIPT_OBJ_TYPE_NULL:
         return false;
 
-      case SCRIPT_OBJ_TYPE_REF:     /* should have been de-reffed already */
-        assert (0);
-
+      case SCRIPT_OBJ_TYPE_REF:
       case SCRIPT_OBJ_TYPE_HASH:
       case SCRIPT_OBJ_TYPE_FUNCTION:
       case SCRIPT_OBJ_TYPE_NATIVE:
@@ -398,8 +384,6 @@ bool script_obj_as_bool (script_obj_t *obj)
         if (*obj->data.string) return true;
         return false;
     }
-
-  assert (0);       /* Abort on uncaught */
   return false;
 }
 
@@ -421,23 +405,16 @@ char *script_obj_as_string (script_obj_t *obj)              /* reply is strduppe
       case SCRIPT_OBJ_TYPE_NULL:
         return NULL;
 
-      case SCRIPT_OBJ_TYPE_REF:     /* should have been de-reffed already */
-        assert (0);
-
+      case SCRIPT_OBJ_TYPE_REF:
       case SCRIPT_OBJ_TYPE_HASH:
-        return NULL;
-
       case SCRIPT_OBJ_TYPE_FUNCTION:
+      case SCRIPT_OBJ_TYPE_NATIVE:
         return NULL;
 
       case SCRIPT_OBJ_TYPE_STRING:
         return strdup (obj->data.string);
-
-      case SCRIPT_OBJ_TYPE_NATIVE:
-        return NULL;
     }
 
-  assert (0);       /* Abort on uncaught */
   return NULL;
 }
 
@@ -560,9 +537,8 @@ void script_obj_assign (script_obj_t *obj_a,
         obj_a->data.string = strdup (obj_b->data.string);
         break;
 
-      case SCRIPT_OBJ_TYPE_REF:     /* should have been de-reffed already */
-        assert (0);
-
+      case SCRIPT_OBJ_TYPE_REF:
+        break;
       case SCRIPT_OBJ_TYPE_HASH:
       case SCRIPT_OBJ_TYPE_FUNCTION:
       case SCRIPT_OBJ_TYPE_NATIVE: