Do not evaluate strings as booleans

src/op.cc

@@ -366,7 +366,7 @@ value_t expr_t::op_t::calc(scope_t& scope, ptr_op_t * locus, const int depth)
     break;
 
   case O_COLON:
-    assert(! "We should never calculate an O_COLON operator");
+    assert("We should never calculate an O_COLON operator" == NULL);
     break;
 
   case O_CONS:

src/post.cc

@@ -558,7 +558,7 @@ std::size_t post_t::xact_id() const
       return id;
     id++;
   }
-  assert(! "Failed to find posting within its transaction");
+  assert("Failed to find posting within its transaction" == NULL);
   return 0;
 }
 
@@ -570,7 +570,7 @@ std::size_t post_t::account_id() const
       return id;
     id++;
   }
-  assert(! "Failed to find posting within its transaction");
+  assert("Failed to find posting within its transaction" == NULL);
   return 0;
 }
 

src/textual.cc

@@ -783,7 +783,7 @@ void instance_t::master_account_directive(char * line)
     context.state_stack.push_front(acct);
 #if !defined(NO_ASSERTS)
   else
-    assert(! "Failed to create account");
+    assert("Failed to create account" == NULL);
 #endif
 }
 

src/utils.cc

@@ -189,7 +189,7 @@ static void trace_delete_func(void * ptr, const char * which)
   if (i == live_memory->end()) {
     i = freed_memory->find(ptr);
     if (i != freed_memory->end())
-      VERIFY(! "Freeing a block of memory twice");
+      VERIFY("Freeing a block of memory twice" == NULL);
 #if 0
     // There can be memory allocated by Boost or the standard library, which
     // was allocated before memory tracing got turned on, that the system