added (currently unchecked) global rendermode option support

src/main.c

@@ -42,6 +42,9 @@ static PyMethodDef COverviewerMethods[] = {
     {"get_render_mode_children", get_render_mode_children, METH_VARARGS,
      "returns (direct) children for a particular render mode"},
     
+    {"set_render_mode_options", set_render_mode_options, METH_VARARGS,
+     "sets the default options for a given render mode"},
+    
     {"extension_version", get_extension_version, METH_VARARGS, 
         "Returns the extension version"},
     

src/rendermode-cave.c

@@ -161,13 +161,13 @@ rendermode_cave_occluded(void *data, RenderState *state) {
 }
 
 static int
-rendermode_cave_start(void *data, RenderState *state) {
+rendermode_cave_start(void *data, RenderState *state, PyObject *options) {
     RenderModeCave* self;
     int ret;
     self = (RenderModeCave *)data;
 
     /* first, chain up */
-    ret = rendermode_normal.start(data, state);
+    ret = rendermode_normal.start(data, state, options);
     if (ret != 0)
         return ret;
 

src/rendermode-lighting.c

@@ -169,11 +169,11 @@ do_shading_with_mask(RenderModeLighting *self, RenderState *state,
 }
 
 static int
-rendermode_lighting_start(void *data, RenderState *state) {
+rendermode_lighting_start(void *data, RenderState *state, PyObject *options) {
     RenderModeLighting* self;
 
     /* first, chain up */
-    int ret = rendermode_normal.start(data, state);
+    int ret = rendermode_normal.start(data, state, options);
     if (ret != 0)
         return ret;
     

src/rendermode-night.c

@@ -26,11 +26,11 @@ static float calculate_darkness(unsigned char skylight, unsigned char blocklight
 }
 
 static int
-rendermode_night_start(void *data, RenderState *state) {
+rendermode_night_start(void *data, RenderState *state, PyObject *options) {
     RenderModeNight* self;    
 
     /* first, chain up */
-    int ret = rendermode_lighting.start(data, state);
+    int ret = rendermode_lighting.start(data, state, options);
     if (ret != 0)
         return ret;
     

src/rendermode-normal.c

@@ -18,7 +18,7 @@
 #include "overviewer.h"
 
 static int
-rendermode_normal_start(void *data, RenderState *state) {
+rendermode_normal_start(void *data, RenderState *state, PyObject *options) {
     PyObject *chunk_x_py, *chunk_y_py, *world, *use_biomes, *worlddir;
     RenderModeNormal *self = (RenderModeNormal *)data;
     

src/rendermode-overlay.c

@@ -26,7 +26,7 @@ static void get_color(void *data, RenderState *state,
 }
 
 static int
-rendermode_overlay_start(void *data, RenderState *state) {
+rendermode_overlay_start(void *data, RenderState *state, PyObject *options) {
     PyObject *facemasks_py;
     RenderModeOverlay *self = (RenderModeOverlay *)data;
     

src/rendermode-spawn.c

@@ -62,11 +62,11 @@ static void get_color(void *data, RenderState *state,
 }
 
 static int
-rendermode_spawn_start(void *data, RenderState *state) {
+rendermode_spawn_start(void *data, RenderState *state, PyObject *options) {
     RenderModeSpawn* self;
 
     /* first, chain up */
-    int ret = rendermode_overlay.start(data, state);
+    int ret = rendermode_overlay.start(data, state, options);
     if (ret != 0)
         return ret;
     

src/rendermodes.c

@@ -30,8 +30,41 @@ static RenderModeInterface *render_modes[] = {
     NULL
 };
 
+static PyObject *render_mode_options = NULL;
+
+/* rendermode encapsulation */
+
+/* helper to recursively find options for a given mode */
+static inline PyObject *
+render_mode_create_options(RenderModeInterface *iface) {
+    PyObject *base_options, *ret, *parent_options;
+    if (render_mode_options == NULL)
+        return PyDict_New();
+    
+    base_options = PyDict_GetItemString(render_mode_options, iface->name);
+    if (base_options) {
+        ret = PyDict_Copy(base_options);
+    } else {
+        ret = PyDict_New();
+    }
+    
+    if (iface->parent) {
+        parent_options = render_mode_create_options(iface->parent);
+        if (parent_options) {
+            if (PyDict_Merge(ret, parent_options, 0) == -1) {
+                Py_DECREF(parent_options);
+                return NULL;
+            }
+            Py_DECREF(parent_options);
+        }
+    }
+    
+    return ret;
+}
+
 RenderMode *render_mode_create(const char *mode, RenderState *state) {
     unsigned int i;
+    PyObject *options;
     RenderMode *ret = NULL;
     RenderModeInterface *iface = NULL;
     for (i = 0; render_modes[i] != NULL; i++) {
@@ -44,12 +77,19 @@ RenderMode *render_mode_create(const char *mode, RenderState *state) {
     if (iface == NULL)
         return NULL;
     
-    ret = malloc(sizeof(RenderMode));
-    if (ret == NULL)
+    options = render_mode_create_options(iface);
+    if (options == NULL)
         return NULL;
     
+    ret = malloc(sizeof(RenderMode));
+    if (ret == NULL) {
+        Py_DECREF(options);
+        return NULL;
+    }
+    
     ret->mode = malloc(iface->data_size);
     if (ret->mode == NULL) {
+        Py_DECREF(options);
         free(ret);
         return NULL;
     }
@@ -57,12 +97,14 @@ RenderMode *render_mode_create(const char *mode, RenderState *state) {
     ret->iface = iface;
     ret->state = state;
     
-    if (iface->start(ret->mode, state)) {
+    if (iface->start(ret->mode, state, options)) {
+        Py_DECREF(options);
         free(ret->mode);
         free(ret);
         return NULL;
     }
     
+    Py_DECREF(options);
     return ret;
 }
     
@@ -213,3 +255,19 @@ PyObject *get_render_mode_children(PyObject *self, PyObject *args) {
     
     return children;
 }
+
+/* python rendermode options bindings */
+PyObject *set_render_mode_options(PyObject *self, PyObject *args) {
+    const char *rendermode;
+    PyObject *opts;
+    if (!PyArg_ParseTuple(args, "sO!", &rendermode, &PyDict_Type, &opts))
+        return NULL;
+    
+    /* check options here */
+    
+    if (render_mode_options == NULL)
+        render_mode_options = PyDict_New();
+    
+    PyDict_SetItemString(render_mode_options, rendermode, opts);
+    Py_RETURN_NONE;
+}

src/rendermodes.h

@@ -50,8 +50,8 @@ struct _RenderModeInterface {
     /* the size of the local storage for this rendermode */
     unsigned int data_size;
     
-    /* may return non-zero on error */
-    int (*start)(void *, RenderState *);
+    /* may return non-zero on error, last arg is options */
+    int (*start)(void *, RenderState *, PyObject *);
     void (*finish)(void *, RenderState *);
     /* returns non-zero to skip rendering this block */
     int (*occluded)(void *, RenderState *);
@@ -72,13 +72,16 @@ void render_mode_destroy(RenderMode *self);
 int render_mode_occluded(RenderMode *self);
 void render_mode_draw(RenderMode *self, PyObject *img, PyObject *mask, PyObject *mask_light);
 
-/* python bindings */
+/* python metadata bindings */
 PyObject *get_render_modes(PyObject *self, PyObject *args);
 PyObject *get_render_mode_info(PyObject *self, PyObject *args);
 PyObject *get_render_mode_parent(PyObject *self, PyObject *args);
 PyObject *get_render_mode_inheritance(PyObject *self, PyObject *args);
 PyObject *get_render_mode_children(PyObject *self, PyObject *args);
 
+/* python rendermode options bindings */
+PyObject *set_render_mode_options(PyObject *self, PyObject *args);
+
 /* individual rendermode interface declarations follow */
 
 /* NORMAL */