changeset 13:914fed025adb

Fixed two little bugs. Fixed a bug preventing mergetag from saving headers it read. Fixed a bug where Input.event() would cause a segfault when no config was loaded. Now it fails silently, to avoid message spam on every single event. As a result, input config filtering via headers tag "Configs" now works (not particularly useful though). committer: Diggory Hardy <diggory.hardy@gmail.com>
author Diggory Hardy <diggory.hardy@gmail.com>
date Mon, 25 Feb 2008 11:11:30 +0000
parents bff0d802cb7d
children 0047b364b6d9
files mde/mergetag/datasection.d
diffstat 1 files changed, 51 insertions(+), 0 deletions(-) [+]
line wrap: on
line diff
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/mde/mergetag/datasection.d	Mon Feb 25 11:11:30 2008 +0000
@@ -0,0 +1,51 @@
+/** This module contains the interface DataSection used by DataSet.
+*
+* It has been moved from the dataset module to avoid cyclic dependancies, since defaultdata depends
+* on DataSection and dataset depends on defaultdata.
+*
+* Also some base mergetag symbols have been moved here.
+*/
+module mde.mergetag.datasection;
+
+/** Typedef for data & section indexes (can be changed to ulong if necessary.) */
+typedef char[] ID;
+
+/**
+ * Interface for data storage classes, which contain all data-tags loaded from a single section of a
+ * file.
+ *
+ * A class implementing this may implement the addTag function to do whatever it likes with the
+ * data passed. DefaultData is one implementation which separates this data out into supported
+ * types and stores it appropriately (allowing merging with existing entries by keeping whichever
+ * tag was last loaded), while ignoring unsupported types. A different
+ * implementation could filter out the tags desired and use them directly, and ignore the rest.
+ *
+ * The tango.scrapple.text.convert.parseTo module provides a useful set of templated functions to
+ * convert the data accordingly. It is advised to keep the type definitions as defined in the file-
+ * format except for user-defined types, although this isn't necessary for library operation
+ * (addTag and writeAll are solely responsible for using and setting the type, ID and data fields).
+ *
+ * Another idea for a DataSection class:
+ * Use a void*[ID] variable to store all data (may also need a type var for each item).
+ * addTag should call a templated function which calls parse then casts to a void* and stores the data.
+ * Use a templated get(T)(ID) method which checks the type and casts to T.
+ */
+interface DataSection
+{
+    /** Delegate passed to writeAll. */
+    typedef void delegate (char[],ID,char[]) ItemDelg;
+    
+    /** Handles parsing of data items for all recognised types.
+     *
+     * Should ignore unsupported types/unwanted tags.
+     *
+     * TextExceptions (thrown by parseTo/parseFrom) are caught and a warning logged; execution
+     * then continues (so the offending tag gets dropped). */
+    void addTag (char[],ID,char[]);
+    
+    /** Responsible for getting all data tags saved.
+    *
+    * writeAll should call the ItemDelg once for each tag to be saved with parameters in the same
+    * form as received by addTag (char[] type, ID id, char[] data). */
+    void writeAll (ItemDelg);
+}