Merge branch 'develop' into refactor/hotkey_module

Author: ab9rf

docs/changelog.txt

@@ -61,6 +61,7 @@ Template for new versions:
 ## Fixes
 
 ## Misc Improvements
+- `createitem`: created items can now be placed onto/into tables, nests, bookcases, display cases, and altars
 - `keybinding`: keybinds may now include the super key, and are no longer limited to particular keys ranges of keys, allowing any recognized by SDL.
 - The ``fpause`` console command can now be used to force world generation to pause (as it did prior to version 50).
 

library/Crashlog.cpp

@@ -1,4 +1,5 @@
 #include "DFHackVersion.h"
+
 #include <csignal>
 #include <iomanip>
 #include <filesystem>
@@ -42,7 +43,8 @@ void signal_crashlog_complete() {
 
 std::thread crashlog_thread;
 
-extern "C" void dfhack_crashlog_handle_signal(int sig) {
+// Force this method to be inlined so that it doesn't create a stack frame
+[[gnu::always_inline]] inline void handle_signal_internal(int sig) {
     if (shutdown.load() || crashed.exchange(true) || crashlog_ready.load()) {
         // Ensure the signal handler doesn't try to write a crashlog
         // whilst the crashlog thread is unavailable.
@@ -61,8 +63,12 @@ extern "C" void dfhack_crashlog_handle_signal(int sig) {
     std::quick_exit(1);
 }
 
+extern "C" void dfhack_crashlog_handle_signal(int sig) {
+    handle_signal_internal(sig);
+}
+
 void dfhack_crashlog_handle_terminate() {
-    dfhack_crashlog_handle_signal(0);
+    handle_signal_internal(0);
 }
 
 std::string signal_name(int sig) {
@@ -105,10 +111,6 @@ std::filesystem::path get_crashlog_path() {
 
 void dfhack_save_crashlog() {
     char** backtrace_strings = backtrace_symbols(crash_info.backtrace, crash_info.backtrace_entries);
-    if (!backtrace_strings) {
-        // Allocation failed, give up
-        return;
-    }
     try {
         std::filesystem::path crashlog_path = get_crashlog_path();
         std::ofstream crashlog(crashlog_path);
@@ -122,8 +124,14 @@ void dfhack_save_crashlog() {
             crashlog << "Signal " << signal << "\n";
         }
 
-        for (int i = 0; i < crash_info.backtrace_entries; i++) {
-            crashlog << i << "> " << backtrace_strings[i] << "\n";
+        if (crash_info.backtrace_entries >= 2 && backtrace_strings != nullptr) {
+            // Skip the first backtrace entry as it will always be dfhack_crashlog_handle_(signal|terminate)
+            for (int i = 1; i < crash_info.backtrace_entries; i++) {
+                crashlog << i - 1 << "> " << backtrace_strings[i] << "\n";
+            }
+        } else {
+            // Make it clear if no relevant backtrace was able to be obtained
+            crashlog << "Failed to obtain relevant backtrace\n";
         }
     } catch (...) {}
 

library/xml

@@ -280,6 +280,7 @@ command_result df_createitem (color_ostream &out, vector<string> &parameters) {
             }
             switch (building->getType())
             {   using namespace df::enums::building_type;
+                case Table:
                 case Coffin:
                 case Furnace:
                 case TradeDepot:
@@ -294,8 +295,12 @@ command_result df_createitem (color_ostream &out, vector<string> &parameters) {
                 case AnimalTrap:
                 case Cage:
                 case Wagon:
+                case Nest:
                 case NestBox:
                 case Hive:
+                case Bookcase:
+                case DisplayFurniture:
+                case OfferingPlace:
                     break;
                 default:
                     out.printerr("The selected building cannot be used for item storage!\n");