NonRealDeveloper
diff --git a/‎library/VersionInfo.cpp‎
Lines changed: 3 additions & 0 deletions b/‎library/VersionInfo.cpp‎
Lines changed: 3 additions & 0 deletions
diff --git a/‎library/include/dfhack/modules/Maps.h‎
Lines changed: 1 addition & 2 deletions b/‎library/include/dfhack/modules/Maps.h‎
Lines changed: 1 addition & 2 deletions
diff --git a/‎library/modules/Maps.cpp‎
Lines changed: 14 additions & 5 deletions b/‎library/modules/Maps.cpp‎
Lines changed: 14 additions & 5 deletions
diff --git a/‎tools/supported/probe.cpp‎
Lines changed: 2 additions & 2 deletions b/‎tools/supported/probe.cpp‎
Lines changed: 2 additions & 2 deletions
diff --git a/‎tools/supported/veinlook.cpp‎
Lines changed: 2 additions & 2 deletions b/‎tools/supported/veinlook.cpp‎
Lines changed: 2 additions & 2 deletions
@@ -173,6 +173,9 @@ void OffsetGroup::setOffset (const string & key, const string & value, const INV
     if(it != OGd->offsets.end())
     {
         int32_t offset = strtol(value.c_str(), NULL, 16);
+        // REPORT pointless rewrites
+        if((*it).second.second == offset)
+            std::cout << "Pointless offset setting: " << this->getFullName() + key << endl;
         (*it).second.second = offset;
         if(inval != NOT_SET)
             (*it).second.first = inval;
 
@@ -8,7 +8,6 @@
 #include "dfhack/DFExport.h"
 #include "dfhack/DFModule.h"
 #include "Vegetation.h"
-
 namespace DFHack
 {
     /***************************************************************************
@@ -23,7 +22,7 @@ namespace DFHack
         feature_Hell_Temple
     };
 
-    extern const char * sa_feature[];
+    extern DFHACK_EXPORT const char * sa_feature(int index);
 
     /// used as a key for the local feature map. combines X an Y coords.
     union planecoord
 
@@ -41,12 +41,21 @@ distribution.
 #define MAPS_GUARD if(!d->Started) throw DFHack::Error::ModuleNotInitialized();
 using namespace DFHack;
 
-const char * sa_feature[]=
+const char * DFHack::sa_feature(int index)
 {
-    "Other",
-    "Adamantine Tube",
-    "Underworld",
-    "Hell Temple"
+    switch(index)
+    {
+        case 0:
+            return "Other";
+        case 1:
+            return "Adamantine Tube";
+        case 2:
+            return "Underworld";
+        case 3:
+            return "Hell Temple";
+        default:
+            return "Unknown/Error";
+    }
 };
 
 struct Maps::Private
 
@@ -139,7 +139,7 @@ int main (int numargs, const char ** args)
                 printf("%-16s  %4d (%2d) %s\n", "",
                     block.local_feature,
                     local_features[pc][block.local_feature]->type,
-                    sa_feature[local_features[pc][block.local_feature]->type]
+                    sa_feature(local_features[pc][block.local_feature]->type)
                     );
             }
 
@@ -148,7 +148,7 @@ int main (int numargs, const char ** args)
                 printf("%-16s  %4d (%2d) %s\n", "",
                     block.global_feature,
                     global_features[block.global_feature].type,
-                    sa_feature[global_features[block.global_feature].type]
+                    sa_feature(global_features[block.global_feature].type)
                     );
             }
             #undef PRINT_FLAG
 
@@ -441,7 +441,7 @@ void do_features(Context* DF, mapblock40d * block, uint32_t blockX, uint32_t blo
             matname = stonetypes[ftr.sub_material].id;
         }
         gotoxy(printX,printY+2);
-        cprintf("%d:%s, material %d/%d : %s", ftr.type, sa_feature[ftr.type], ftr.main_material, ftr.sub_material, matname);
+        cprintf("%d:%s, material %d/%d : %s", ftr.type, sa_feature(ftr.type), ftr.main_material, ftr.sub_material, matname);
         {
             gotoxy(printX,printY+3);
             string name = p->readClassName(p->readDWord( ftr.origin ));
@@ -469,7 +469,7 @@ void do_features(Context* DF, mapblock40d * block, uint32_t blockX, uint32_t blo
                 matname = stonetypes[ftr.sub_material].id;
             }
             gotoxy(printX,printY+6);
-            cprintf("%d:%s, material %d/%d : %s", ftr.type, sa_feature[ftr.type], ftr.main_material, ftr.sub_material, matname);
+            cprintf("%d:%s, material %d/%d : %s", ftr.type, sa_feature(ftr.type), ftr.main_material, ftr.sub_material, matname);
 
             gotoxy(printX,printY+7);
             string name = p->readClassName(p->readDWord( ftr.origin ));
Original file line number	Diff line number	Diff line change
`@@ -441,7 +441,7 @@ void do_features(Context* DF, mapblock40d * block, uint32_t blockX, uint32_t blo`
`441`	`441`	`matname = stonetypes[ftr.sub_material].id;`
`442`	`442`	`}`
`443`	`443`	`gotoxy(printX,printY+2);`
`444`		`- cprintf("%d:%s, material %d/%d : %s", ftr.type, sa_feature[ftr.type], ftr.main_material, ftr.sub_material, matname);`
	`444`	`+ cprintf("%d:%s, material %d/%d : %s", ftr.type, sa_feature(ftr.type), ftr.main_material, ftr.sub_material, matname);`
`445`	`445`	`{`
`446`	`446`	`gotoxy(printX,printY+3);`
`447`	`447`	`string name = p->readClassName(p->readDWord( ftr.origin ));`
`@@ -469,7 +469,7 @@ void do_features(Context* DF, mapblock40d * block, uint32_t blockX, uint32_t blo`
`469`	`469`	`matname = stonetypes[ftr.sub_material].id;`
`470`	`470`	`}`
`471`	`471`	`gotoxy(printX,printY+6);`
`472`		`- cprintf("%d:%s, material %d/%d : %s", ftr.type, sa_feature[ftr.type], ftr.main_material, ftr.sub_material, matname);`
	`472`	`+ cprintf("%d:%s, material %d/%d : %s", ftr.type, sa_feature(ftr.type), ftr.main_material, ftr.sub_material, matname);`
`473`	`473`
`474`	`474`	`gotoxy(printX,printY+7);`
`475`	`475`	`string name = p->readClassName(p->readDWord( ftr.origin ));`