Add Drawable API

.clang-tidy

@@ -0,0 +1,56 @@
+---
+Checks:          'clang-diagnostic-*,clang-analyzer-*'
+WarningsAsErrors: ''
+HeaderFilterRegex: ''
+AnalyzeTemporaryDtors: false
+FormatStyle:     none
+User:            joshuam
+CheckOptions:
+  - key:             cert-dcl16-c.NewSuffixes
+    value:           'L;LL;LU;LLU'
+  - key:             cert-oop54-cpp.WarnOnlyIfThisHasSuspiciousField
+    value:           '0'
+  - key:             cert-str34-c.DiagnoseSignedUnsignedCharComparisons
+    value:           '0'
+  - key:             cppcoreguidelines-explicit-virtual-functions.IgnoreDestructors
+    value:           '1'
+  - key:             cppcoreguidelines-non-private-member-variables-in-classes.IgnoreClassesWithAllMemberVariablesBeingPublic
+    value:           '1'
+  - key:             google-readability-braces-around-statements.ShortStatementLines
+    value:           '1'
+  - key:             google-readability-function-size.StatementThreshold
+    value:           '800'
+  - key:             google-readability-namespace-comments.ShortNamespaceLines
+    value:           '10'
+  - key:             google-readability-namespace-comments.SpacesBeforeComments
+    value:           '2'
+  - key:             llvm-else-after-return.WarnOnConditionVariables
+    value:           '0'
+  - key:             llvm-else-after-return.WarnOnUnfixable
+    value:           '0'
+  - key:             llvm-qualified-auto.AddConstToQualified
+    value:           '0'
+  - key:             modernize-loop-convert.MaxCopySize
+    value:           '16'
+  - key:             modernize-loop-convert.MinConfidence
+    value:           reasonable
+  - key:             modernize-loop-convert.NamingStyle
+    value:           CamelCase
+  - key:             modernize-pass-by-value.IncludeStyle
+    value:           llvm
+  - key:             modernize-replace-auto-ptr.IncludeStyle
+    value:           llvm
+  - key:             modernize-use-nullptr.NullMacros
+    value:           'NULL'
+  - key:             readability-identifier-naming.StructCase
+    value:           'CamelCase'
+  - key:             readability-identifier-naming.FunctionCase
+    value:           'lower_case'
+  - key:             readability-identifier-naming.VariableCase
+    value:           'lower_case'
+  - key:             readability-identifier-naming.GlobalConstantCase
+    value:           'UPPER_CASE'
+  - key:             readability-identifier-naming.ConstantCase
+    value:           'UPPER_CASE'
+...
+

docs/drawable_api.md

@@ -0,0 +1,36 @@
+# Drawable API Design Document
+
+# Creating a Drawable
+
+```c
+DrawableTextures *textures = drawable_get_textures();
+Drawable drawable = {
+    .x = 0, 
+    .y = SCREEN_HEIGHT - 32,
+    .enabled = enabled,
+    .value.texture = textures->status_bar,
+    .type = DRAWABLE_TEXTURE
+};
+
+// Under the hood this uses the DrawableRegistry
+drawable_create_drawable(&drawable, "statusbar"); 
+```
+
+# Destroying a drawable
+
+```c 
+// Again, using the DrawableRegistry under the hood
+drawable_destroy_drawable(drawable, "statusbar"); 
+
+// All Drawables can be destroyed (such as at exit), just a wrapper
+around the DrawableRegistry.
+drawables_destroy_all();
+```
+
+## Accessing a Drawable
+
+```c
+// Again, wrapper around the Drawable registry
+Drawable *drawable = drawable_get_drawable(drawable, "statusbar");
+*drawable->value.number = &plyr_ammo;
+```

meson.build

@@ -1,5 +1,5 @@
 project('mindoom', 'c',
-       default_options: ['default_library=static', 'buildtype=debugoptimized'])
+       default_options: ['default_library=static', 'buildtype=debug'])
 add_project_arguments('-DLOG_USE_COLOR', language: 'c')
 
 sdl2 = dependency('SDL2')
@@ -38,6 +38,12 @@ dedicated_server_source_files = files(
     src_dir / 'net_structrw.c',
   )
 
+# Source files for the game engine
+engine_src_dir = src_dir / 'engine'
+engine_source_files = files (
+    engine_src_dir / 'graphics' / 'widget.c',
+    engine_src_dir / 'graphics' / 'drawable.c',
+)
 
 # Source files only used by the game binary 
 doom_source_dir = 'doom'
@@ -135,8 +141,7 @@ game_source_files = files(
     src_dir / doom_source_dir / 's_sound.c',
     src_dir / doom_source_dir / 'statdump.c',
 
-    # Code for the HUD
-    src_dir / doom_source_dir / 'hud' / 'widget.c',
+    # Code for the status bar 
     src_dir / doom_source_dir / 'hud' / 'statusbar.c',
 
     src_dir / doom_source_dir / 'wi_stuff.c'
@@ -210,6 +215,7 @@ executable('mindoom',
     sources: [
         common_source_files, 
         game_source_files,
+        engine_source_files
     ],
     link_with: [
         textscreen, 

scripts/README.md

@@ -0,0 +1,3 @@
+# mindoom
+
+These scripts are intended to be called by Meson as run targets. Don't call them directly, instead use `meson compile scriptname` from the build directory.

src/d_mode.h

@@ -26,6 +26,7 @@
 
 typedef enum
 {
+    //TODO: remove
     doom,    // Doom 1
     heretic, // Heretic
     hexen,   // Hexen
@@ -36,6 +37,7 @@ typedef enum
 // in: eg. shareware vs. registered.  So doom1.wad and doom.wad are the
 // same mission, but a different mode.
 
+// TODO: remove
 typedef enum
 {
     registered,  // Doom registered
@@ -44,6 +46,7 @@ typedef enum
 
 // What version are we emulating?
 
+// TODO: remove
 typedef enum
 {
     exe_doom_1_9, // Doom 1.9: "