writefile: use const for old address
authorCampbell Barton <ideasman42@gmail.com>
Tue, 21 Jun 2016 22:44:26 +0000 (08:44 +1000)
committerCampbell Barton <ideasman42@gmail.com>
Tue, 21 Jun 2016 22:44:26 +0000 (08:44 +1000)
Also remove temp preview overwriting.

source/blender/blenloader/intern/readfile.c
source/blender/blenloader/intern/writefile.c
source/blender/makesdna/DNA_sdna_types.h

index 5c7b664e5476c83dee76cc471e0959b15467fdb5..689a975e19f54ec1e28f9928b5d021d6245dc597 100644 (file)
@@ -8519,7 +8519,7 @@ BlendFileData *blo_read_file_internal(FileData *fd, const char *filepath)
 
 struct BHeadSort {
        BHead *bhead;
-       void *old;
+       const void *old;
 };
 
 static int verg_bheadsort(const void *v1, const void *v2)
index 327b7b5aed44680ad6b0b3a2ba9a40e89bc11170..b5a2115fa5f97c115828327c23b2aa8ad8e85440 100644 (file)
@@ -449,7 +449,9 @@ static int endwrite(WriteData *wd)
 
 /* ********** WRITE FILE ****************** */
 
-static void writestruct_at_address(WriteData *wd, int filecode, const char *structname, int nr, void *adr, void *data)
+static void writestruct_at_address(
+        WriteData *wd, int filecode, const char *structname, int nr,
+        const void *adr, const void *data)
 {
        BHead bh;
        const short *sp;
@@ -476,7 +478,9 @@ static void writestruct_at_address(WriteData *wd, int filecode, const char *stru
        mywrite(wd, data, bh.len);
 }
 
-static void writestruct(WriteData *wd, int filecode, const char *structname, int nr, void *adr)
+static void writestruct(
+        WriteData *wd, int filecode, const char *structname, int nr,
+        const void *adr)
 {
        writestruct_at_address(wd, filecode, structname, nr, adr, adr);
 }
@@ -493,7 +497,7 @@ static void writedata(WriteData *wd, int filecode, int len, const void *adr)  /*
 
        /* init BHead */
        bh.code   = filecode;
-       bh.old    = (void *)adr;  /* this is safe to cast from const */
+       bh.old    = adr;
        bh.nr     = 1;
        bh.SDNAnr = 0;
        bh.len    = len;
@@ -503,9 +507,9 @@ static void writedata(WriteData *wd, int filecode, int len, const void *adr)  /*
 }
 
 /* use this to force writing of lists in same order as reading (using link_list) */
-static void writelist(WriteData *wd, int filecode, const char *structname, ListBase *lb)
+static void writelist(WriteData *wd, int filecode, const char *structname, const ListBase *lb)
 {
-       Link *link = lb->first;
+       const Link *link = lb->first;
        
        while (link) {
                writestruct(wd, filecode, structname, 1, link);
@@ -515,10 +519,10 @@ static void writelist(WriteData *wd, int filecode, const char *structname, ListB
 
 /* *************** writing some direct data structs used in more code parts **************** */
 /*These functions are used by blender's .blend system for file saving/loading.*/
-void IDP_WriteProperty_OnlyData(IDProperty *prop, void *wd);
-void IDP_WriteProperty(IDProperty *prop, void *wd);
+void IDP_WriteProperty_OnlyData(const IDProperty *prop, void *wd);
+void IDP_WriteProperty(const IDProperty *prop, void *wd);
 
-static void IDP_WriteArray(IDProperty *prop, void *wd)
+static void IDP_WriteArray(const IDProperty *prop, void *wd)
 {
        /*REMEMBER to set totalen to len in the linking code!!*/
        if (prop->data.pointer) {
@@ -534,11 +538,11 @@ static void IDP_WriteArray(IDProperty *prop, void *wd)
        }
 }
 
-static void IDP_WriteIDPArray(IDProperty *prop, void *wd)
+static void IDP_WriteIDPArray(const IDProperty *prop, void *wd)
 {
        /*REMEMBER to set totalen to len in the linking code!!*/
        if (prop->data.pointer) {
-               IDProperty *array = prop->data.pointer;
+               const IDProperty *array = prop->data.pointer;
                int a;
 
                writestruct(wd, DATA, "IDProperty", prop->len, array);
@@ -548,13 +552,13 @@ static void IDP_WriteIDPArray(IDProperty *prop, void *wd)
        }
 }
 
-static void IDP_WriteString(IDProperty *prop, void *wd)
+static void IDP_WriteString(const IDProperty *prop, void *wd)
 {
        /*REMEMBER to set totalen to len in the linking code!!*/
        writedata(wd, DATA, prop->len, prop->data.pointer);
 }
 
-static void IDP_WriteGroup(IDProperty *prop, void *wd)
+static void IDP_WriteGroup(const IDProperty *prop, void *wd)
 {
        IDProperty *loop;
 
@@ -564,7 +568,7 @@ static void IDP_WriteGroup(IDProperty *prop, void *wd)
 }
 
 /* Functions to read/write ID Properties */
-void IDP_WriteProperty_OnlyData(IDProperty *prop, void *wd)
+void IDP_WriteProperty_OnlyData(const IDProperty *prop, void *wd)
 {
        switch (prop->type) {
                case IDP_GROUP:
@@ -582,13 +586,13 @@ void IDP_WriteProperty_OnlyData(IDProperty *prop, void *wd)
        }
 }
 
-void IDP_WriteProperty(IDProperty *prop, void *wd)
+void IDP_WriteProperty(const IDProperty *prop, void *wd)
 {
        writestruct(wd, DATA, "IDProperty", 1, prop);
        IDP_WriteProperty_OnlyData(prop, wd);
 }
 
-static void write_iddata(void *wd, ID *id)
+static void write_iddata(void *wd, const ID *id)
 {
        /* ID_WM's id->properties are considered runtime only, and never written in .blend file. */
        if (id->properties && !ELEM(GS(id->name), ID_WM)) {
@@ -596,29 +600,24 @@ static void write_iddata(void *wd, ID *id)
        }
 }
 
-static void write_previews(WriteData *wd, PreviewImage *prv)
+static void write_previews(WriteData *wd, const PreviewImage *prv_orig)
 {
        /* Never write previews when doing memsave (i.e. undo/redo)! */
-       if (prv && !wd->current) {
-               short w = prv->w[1];
-               short h = prv->h[1];
-               unsigned int *rect = prv->rect[1];
+       if (prv_orig && !wd->current) {
+               PreviewImage prv = *prv_orig;
 
                /* don't write out large previews if not requested */
                if (!(U.flag & USER_SAVE_PREVIEWS)) {
-                       prv->w[1] = 0;
-                       prv->h[1] = 0;
-                       prv->rect[1] = NULL;
-               }
-               writestruct(wd, DATA, "PreviewImage", 1, prv);
-               if (prv->rect[0]) writedata(wd, DATA, prv->w[0] * prv->h[0] * sizeof(unsigned int), prv->rect[0]);
-               if (prv->rect[1]) writedata(wd, DATA, prv->w[1] * prv->h[1] * sizeof(unsigned int), prv->rect[1]);
-
-               /* restore preview, we still want to keep it in memory even if not saved to file */
-               if (!(U.flag & USER_SAVE_PREVIEWS) ) {
-                       prv->w[1] = w;
-                       prv->h[1] = h;
-                       prv->rect[1] = rect;
+                       prv.w[1] = 0;
+                       prv.h[1] = 0;
+                       prv.rect[1] = NULL;
+               }
+               writestruct_at_address(wd, DATA, "PreviewImage", 1, prv_orig, &prv);
+               if (prv.rect[0]) {
+                       writedata(wd, DATA, prv.w[0] * prv.h[0] * sizeof(unsigned int), prv.rect[0]);
+               }
+               if (prv.rect[1]) {
+                       writedata(wd, DATA, prv.w[1] * prv.h[1] * sizeof(unsigned int), prv.rect[1]);
                }
        }
 }
index 28204ebeb8888120f373897102a54fd6063202eb..cb7a371c7c5205d9a541276ce2c5901687aa0a61 100644 (file)
@@ -67,7 +67,7 @@ typedef struct SDNA {
 #
 typedef struct BHead {
        int code, len;
-       void *old;
+       const void *old;
        int SDNAnr, nr;
 } BHead;
 #