Rename chars used as numbers to int8_t/uint8_t. Fix some other type usage.

This should help against the signed char problems, and is good for
code readability.  Now, if you have a char, it's either an untyped
in-memory byte, or a symbol inside a string.  Small numbers are instead
[u]int8_t, ints, an enum type, or, in so many cases, bools.

I didn't touch any of the tiles code, as it's currently broken and I don't
want to risk making it unbroken harder.

1 files changed, 2 insertions, 2 deletions

diff --git a/crawl-ref/source/stash.h b/crawl-ref/source/stash.h
index 4c2f8c2a1b..5a34379193 100644
--- a/crawl-ref/source/stash.h
+++ b/crawl-ref/source/stash.h
@@ -41,7 +41,7 @@ public:
     Stash(int xp = -1, int yp = -1);
 
     static bool is_boring_feature(dungeon_feature_type feat);
-    static void filter(object_class_type base_type, unsigned char sub_type);
+    static void filter(object_class_type base_type, uint8_t sub_type);
     static void filter(const std::string &filt);
 
     static std::string stash_item_name(const item_def &item);
@@ -98,7 +98,7 @@ private:
 
 private:
     bool verified;      // Is this correct to the best of our knowledge?
-    unsigned char x, y;
+    uint8_t x, y;
     int  abspos;
     dungeon_feature_type feat;
     trap_type trap;