code cleanup: use single define for undo string size, was 64 mostly, but 512 in the UI.
authorCampbell Barton <ideasman42@gmail.com>
Mon, 10 Sep 2012 01:55:58 +0000 (01:55 +0000)
committerCampbell Barton <ideasman42@gmail.com>
Mon, 10 Sep 2012 01:55:58 +0000 (01:55 +0000)
source/blender/blenkernel/BKE_blender.h
source/blender/blenkernel/intern/blender.c
source/blender/editors/gpencil/gpencil_paint.c
source/blender/editors/gpencil/gpencil_undo.c
source/blender/editors/interface/interface_handlers.c
source/blender/editors/sculpt_paint/paint_undo.c
source/blender/editors/util/editmode_undo.c
source/blender/editors/util/undo.c

index 0e7bdbbb4dd53ca33e9ef1cf445516cdb8232407..1e67027ae8c211a972cb44c04ae1ee8fc8b3f0f6 100644 (file)
@@ -83,6 +83,8 @@ void BKE_userdef_free(void);
 void set_blender_test_break_cb(void (*func)(void) );
 int blender_test_break(void);
 
+#define BKE_UNDO_STR_MAX 64
+
 /* global undo */
 extern void BKE_write_undo(struct bContext *C, const char *name);
 extern void BKE_undo_step(struct bContext *C, int step);
index 04b135df798501262ab19979697482745d81d07d..09519f68415db5f956706465c74ab1496395b81e 100644 (file)
@@ -463,11 +463,10 @@ int blender_test_break(void)
 
 #define UNDO_DISK   0
 
-#define MAXUNDONAME 64
 typedef struct UndoElem {
        struct UndoElem *next, *prev;
        char str[FILE_MAX];
-       char name[MAXUNDONAME];
+       char name[BKE_UNDO_STR_MAX];
        MemFile memfile;
        uintptr_t undosize;
 } UndoElem;
index dfd2783fad6f311f6026fe45c5bd80c65639049a..4e281b96fc385a134fa40e6ec308037e7d7c0f5d 100644 (file)
@@ -27,7 +27,6 @@
  *  \ingroup edgpencil
  */
 
-
 #include <stdio.h>
 #include <stddef.h>
 #include <stdlib.h>
index 61f7e1bb86f05129a5408322f3d5498df1b8dd52..fa9f51968665ab46341049703ca6b12a3972338d 100644 (file)
  * ***** END GPL LICENSE BLOCK *****
  */
 
+/** \file blender/editors/gpencil/gpencil_undo.c
+ *  \ingroup edgpencil
+ */
+
 #include <stdlib.h>
 #include <string.h>
 
@@ -34,6 +38,7 @@
 #include "DNA_listBase.h"
 #include "DNA_windowmanager_types.h"
 
+#include "BKE_blender.h"
 #include "BKE_context.h"
 #include "BKE_gpencil.h"
 
 
 #include "gpencil_intern.h"
 
-#define MAXUNDONAME 64
-
 typedef struct bGPundonode {
        struct bGPundonode *next, *prev;
 
-       char name[MAXUNDONAME];
+       char name[BKE_UNDO_STR_MAX];
        struct bGPdata *gpd;
 } bGPundonode;
 
index 23fb480cd1fcb8b6e3a4641599a51d2bb419c1d7..822794aae7b455fea03c3b3e06eb5ce027fde56a 100644 (file)
@@ -197,7 +197,7 @@ typedef struct uiAfterFunc {
 
        bContextStore *context;
 
-       char undostr[512];
+       char undostr[BKE_UNDO_STR_MAX];
 
        int autokey;
 } uiAfterFunc;
index 8e6c87c151050a82c9c46f73037d7c446fa83270..f5b9aa742c6ac516fee7eb62737cc3b916df467d 100644 (file)
@@ -23,7 +23,6 @@
  *  \brief Undo system for painting and sculpting.
  */
 
-
 #include <stdlib.h>
 #include <string.h>
 
@@ -35,7 +34,7 @@
 
 #include "DNA_userdef_types.h"
 
-
+#include "BKE_blender.h"
 #include "BKE_context.h"
 #include "BKE_global.h"
 
 
 #include "paint_intern.h"
 
-#define MAXUNDONAME 64
-
 typedef struct UndoElem {
        struct UndoElem *next, *prev;
-       char name[MAXUNDONAME];
+       char name[BKE_UNDO_STR_MAX];
        uintptr_t undosize;
 
        ListBase elems;
@@ -272,4 +269,3 @@ void ED_undo_paint_free(void)
        undo_stack_free(&ImageUndoStack);
        undo_stack_free(&MeshUndoStack);
 }
-
index 18610d57bbdd108ac7fa7f012267f9d7a6c3dcb2..9a3b0476205dbfdcd9e0f62976e123872faa35b1 100644 (file)
@@ -29,8 +29,6 @@
  *  \ingroup edutil
  */
 
-
-
 #include <stdlib.h>
 #include <string.h>
 #include <math.h>
@@ -44,7 +42,7 @@
 #include "BLI_dynstr.h"
 #include "BLI_utildefines.h"
 
-
+#include "BKE_blender.h"
 #include "BKE_context.h"
 #include "BKE_depsgraph.h"
 #include "BKE_global.h"
@@ -82,8 +80,6 @@
 static void error(const char *UNUSED(arg)) {}
 /* ****** XXX ***** */
 
-
-#define MAXUNDONAME 64
 typedef struct UndoElem {
        struct UndoElem *next, *prev;
        ID id;          // copy of editmode object ID
@@ -91,7 +87,7 @@ typedef struct UndoElem {
        int type;       // type of edited object
        void *undodata;
        uintptr_t undosize;
-       char name[MAXUNDONAME];
+       char name[BKE_UNDO_STR_MAX];
        void * (*getdata)(bContext * C);
        void (*freedata)(void *);
        void (*to_editmode)(void *, void *, void *);
index 4213ff68cf2d259cff1090e24ceedcae3fc472b6..2ffad3ad7968348dc23f24f49a4a8bc706f17a61 100644 (file)
@@ -74,8 +74,6 @@
 
 #include "util_intern.h"
 
-#define MAXUNDONAME 64 /* XXX, make common define */
-
 /* ***************** generic undo system ********************* */
 
 void ED_undo_push(bContext *C, const char *str)
@@ -283,7 +281,7 @@ static int ed_undo_exec(bContext *C, wmOperator *UNUSED(op))
 
 static int ed_undo_push_exec(bContext *C, wmOperator *op)
 {
-       char str[MAXUNDONAME];
+       char str[BKE_UNDO_STR_MAX];
        RNA_string_get(op->ptr, "message", str);
        ED_undo_push(C, str);
        return OPERATOR_FINISHED;
@@ -321,7 +319,7 @@ void ED_OT_undo_push(wmOperatorType *ot)
 
        ot->flag = OPTYPE_INTERNAL;
 
-       RNA_def_string(ot->srna, "message", "Add an undo step *function may be moved*", MAXUNDONAME, "Undo Message", "");
+       RNA_def_string(ot->srna, "message", "Add an undo step *function may be moved*", BKE_UNDO_STR_MAX, "Undo Message", "");
 }
 
 void ED_OT_redo(wmOperatorType *ot)