Remove packed attr on map nodes

It was causing compiler to access as bytes. This patch makes
unpacked struct more obvious.

Author: jserv

src/doomdata.h

@@ -178,24 +178,6 @@ typedef struct
 // Indicate a leaf.
 #define NF_SUBSECTOR    0x8000
 
-typedef struct
-{
-  // Partition line from (x,y) to x+dx,y+dy)
-  short         x;
-  short         y;
-  short         dx;
-  short         dy;
-
-  // Bounding box for each child,
-  // clip against view frustum.
-  short         bbox[2][4];
-
-  // If NF_SUBSECTOR its a subsector,
-  // else it's a node of another subtree.
-  unsigned short        children[2];
-
-} PACKEDATTR mapnode_t;
-
 
 
 
@@ -211,7 +193,19 @@ typedef struct
 } PACKEDATTR mapthing_t;
 
 
+// This is used at runtime so not packed.
+// compiler uses byte access on packed structs.
 
+typedef struct {
+  short x;  // Partition line from (x,y) to x+dx,y+dy)
+  short y;
+  short dx;
+  short dy;
+  // Bounding box for each child, clip against view frustum.
+  short bbox[2][4];
+  // If NF_SUBSECTOR its a subsector, else it's a node of another subtree.
+  unsigned short children[2];
+} mapnode_t;
 
 
 #endif                  // __DOOMDATA__