A function die() that does the equivalent of ASSERT(!"foo") but always works.

It's what the old DEBUGSTR() did except for failing to be fatal in non-assert
builds.

Rationale: not aborting in known failure leads to further damage, and unlike
actual (conditional) assertions, there's no gain in skipping checks.

1 files changed, 3 insertions, 3 deletions

diff --git a/crawl-ref/source/store.cc b/crawl-ref/source/store.cc
index d1ca966932..33f4e93409 100644
--- a/crawl-ref/source/store.cc
+++ b/crawl-ref/source/store.cc
@@ -351,11 +351,11 @@ void CrawlStoreValue::unset(bool force)
     }
 
     case SV_NONE:
-        DEBUGSTR("CrawlStoreValue::unset: unsetting nothing");
+        die("CrawlStoreValue::unset: unsetting nothing");
         break;
 
     default:
-        DEBUGSTR("CrawlStoreValue::unset: unsetting invalid type");
+        die("CrawlStoreValue::unset: unsetting invalid type");
         break;
     }
 
@@ -427,7 +427,7 @@ CrawlStoreValue &CrawlStoreValue::operator = (const CrawlStoreValue &other)
         break;
 
      default:
-        DEBUGSTR("CrawlStoreValue has invalid type");
+        die("CrawlStoreValue has invalid type");
         break;
     }