Remove `TOX_DEBUG` and have asserts always enabled.

These are cheap asserts. I've also replaced the fprintf's with
`LOGGER_ERROR` calls.

1 files changed, 3 insertions, 6 deletions

diff --git a/toxcore/DHT.c b/toxcore/DHT.c
index 94bbbe01..611cb0f8 100644
--- a/toxcore/DHT.c
+++ b/toxcore/DHT.c
@@ -2813,13 +2813,10 @@ static int dht_load_state_callback(void *outer, const uint8_t *data, uint32_t le
 
             break;
 
-#ifdef TOX_DEBUG
-
         default:
-            fprintf(stderr, "Load state (DHT): contains unrecognized part (len %u, type %u)\n",
-                    length, type);
+            LOGGER_ERROR(dht->log, "Load state (DHT): contains unrecognized part (len %u, type %u)\n",
+                         length, type);
             break;
-#endif
     }
 
     return 0;
@@ -2839,7 +2836,7 @@ int DHT_load(DHT *dht, const uint8_t *data, uint32_t length)
         lendian_to_host32(&data32, data);
 
         if (data32 == DHT_STATE_COOKIE_GLOBAL) {
-            return load_state(dht_load_state_callback, dht, data + cookie_len,
+            return load_state(dht_load_state_callback, dht->log, dht, data + cookie_len,
                               length - cookie_len, DHT_STATE_COOKIE_TYPE);
         }
     }