[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [57200] trunk/blender: Better API design for making text datablocks after loading.
Tamito Kajiyama
rd6t-kjym at asahi-net.or.jp
Sun Jun 2 19:52:06 CEST 2013
Revision: 57200
http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=57200
Author: kjym3
Date: 2013-06-02 17:52:06 +0000 (Sun, 02 Jun 2013)
Log Message:
-----------
Better API design for making text datablocks after loading.
An optional 'internal' argument was added to the bpy.data.texts.load() operator.
The changes in revision 57153 were reverted, so that the is_in_memory and is_dirty
properties of text datablocks are not editable again.
In the C API layer, BKE_text_load_ex() was introduced to allow for optionally
making text datablocks internal after loading.
Revision Links:
--------------
http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=57153
Modified Paths:
--------------
trunk/blender/release/scripts/startup/bl_operators/freestyle.py
trunk/blender/source/blender/blenkernel/BKE_text.h
trunk/blender/source/blender/blenkernel/intern/text.c
trunk/blender/source/blender/editors/space_text/text_ops.c
trunk/blender/source/blender/makesrna/intern/rna_main_api.c
trunk/blender/source/blender/makesrna/intern/rna_text.c
Modified: trunk/blender/release/scripts/startup/bl_operators/freestyle.py
===================================================================
--- trunk/blender/release/scripts/startup/bl_operators/freestyle.py 2013-06-02 17:32:45 UTC (rev 57199)
+++ trunk/blender/release/scripts/startup/bl_operators/freestyle.py 2013-06-02 17:52:06 UTC (rev 57200)
@@ -166,10 +166,6 @@
return {'RUNNING_MODAL'}
def execute(self, context):
- text = bpy.data.texts.load(self.filepath)
- if self.make_internal:
- text.is_in_memory = True
- text.is_dirty = True
- text.filepath = ""
+ text = bpy.data.texts.load(self.filepath, self.make_internal)
self.freestyle_module.script = text
return {'FINISHED'}
Modified: trunk/blender/source/blender/blenkernel/BKE_text.h
===================================================================
--- trunk/blender/source/blender/blenkernel/BKE_text.h 2013-06-02 17:32:45 UTC (rev 57199)
+++ trunk/blender/source/blender/blenkernel/BKE_text.h 2013-06-02 17:52:06 UTC (rev 57200)
@@ -48,6 +48,8 @@
struct Text *BKE_text_add (struct Main *bmain, const char *name);
int txt_extended_ascii_as_utf8(char **str);
int BKE_text_reload (struct Text *text);
+struct Text *BKE_text_load_ex(struct Main *bmain, const char *file, const char *relpath,
+ const bool is_internal);
struct Text *BKE_text_load (struct Main *bmain, const char *file, const char *relpath);
struct Text *BKE_text_copy (struct Text *ta);
void BKE_text_unlink (struct Main *bmain, struct Text *text);
Modified: trunk/blender/source/blender/blenkernel/intern/text.c
===================================================================
--- trunk/blender/source/blender/blenkernel/intern/text.c 2013-06-02 17:32:45 UTC (rev 57199)
+++ trunk/blender/source/blender/blenkernel/intern/text.c 2013-06-02 17:52:06 UTC (rev 57200)
@@ -362,7 +362,7 @@
return 1;
}
-Text *BKE_text_load(Main *bmain, const char *file, const char *relpath)
+Text *BKE_text_load_ex(Main *bmain, const char *file, const char *relpath, const bool is_internal)
{
FILE *fp;
int i, llen, len;
@@ -392,8 +392,13 @@
len = ftell(fp);
fseek(fp, 0L, SEEK_SET);
- ta->name = MEM_mallocN(strlen(file) + 1, "text_name");
- strcpy(ta->name, file);
+ if (is_internal == false) {
+ ta->name = MEM_mallocN(strlen(file) + 1, "text_name");
+ strcpy(ta->name, file);
+ }
+ else {
+ ta->flags |= TXT_ISMEM | TXT_ISDIRTY;
+ }
init_undo_text(ta);
@@ -460,6 +465,11 @@
return ta;
}
+Text *BKE_text_load(Main *bmain, const char *file, const char *relpath)
+{
+ return BKE_text_load_ex(bmain, file, relpath, false);
+}
+
Text *BKE_text_copy(Text *ta)
{
Text *tan;
Modified: trunk/blender/source/blender/editors/space_text/text_ops.c
===================================================================
--- trunk/blender/source/blender/editors/space_text/text_ops.c 2013-06-02 17:32:45 UTC (rev 57199)
+++ trunk/blender/source/blender/editors/space_text/text_ops.c 2013-06-02 17:52:06 UTC (rev 57200)
@@ -238,7 +238,7 @@
RNA_string_get(op->ptr, "filepath", str);
- text = BKE_text_load(bmain, str, G.main->name);
+ text = BKE_text_load_ex(bmain, str, G.main->name, internal);
if (!text) {
if (op->customdata) MEM_freeN(op->customdata);
@@ -264,13 +264,6 @@
st->text = text;
st->top = 0;
}
-
- if (internal) {
- if (text->name)
- MEM_freeN(text->name);
-
- text->name = NULL;
- }
text_drawcache_tag_update(st, 1);
WM_event_add_notifier(C, NC_TEXT | NA_ADDED, text);
Modified: trunk/blender/source/blender/makesrna/intern/rna_main_api.c
===================================================================
--- trunk/blender/source/blender/makesrna/intern/rna_main_api.c 2013-06-02 17:32:45 UTC (rev 57199)
+++ trunk/blender/source/blender/makesrna/intern/rna_main_api.c 2013-06-02 17:52:06 UTC (rev 57200)
@@ -725,12 +725,12 @@
RNA_POINTER_INVALIDATE(text_ptr);
}
-static Text *rna_Main_texts_load(Main *bmain, ReportList *reports, const char *filepath)
+static Text *rna_Main_texts_load(Main *bmain, ReportList *reports, const char *filepath, int is_internal)
{
Text *txt;
errno = 0;
- txt = BKE_text_load(bmain, filepath, bmain->name);
+ txt = BKE_text_load_ex(bmain, filepath, bmain->name, is_internal);
if (!txt)
BKE_reportf(reports, RPT_ERROR, "Cannot read '%s': %s", filepath,
@@ -1701,6 +1701,7 @@
RNA_def_function_ui_description(func, "Add a new text to the main database from a file");
parm = RNA_def_string_file_path(func, "filepath", "Path", FILE_MAX, "", "path for the datablock");
RNA_def_property_flag(parm, PROP_REQUIRED);
+ parm = RNA_def_boolean(func, "internal", 0, "Make internal", "Make text file internal after loading");
/* return type */
parm = RNA_def_pointer(func, "text", "Text", "", "New text datablock");
RNA_def_function_return(func, parm);
Modified: trunk/blender/source/blender/makesrna/intern/rna_text.c
===================================================================
--- trunk/blender/source/blender/makesrna/intern/rna_text.c 2013-06-02 17:32:45 UTC (rev 57199)
+++ trunk/blender/source/blender/makesrna/intern/rna_text.c 2013-06-02 17:52:06 UTC (rev 57200)
@@ -148,6 +148,7 @@
prop = RNA_def_property(srna, "is_dirty", PROP_BOOLEAN, PROP_NONE);
RNA_def_property_boolean_sdna(prop, NULL, "flags", TXT_ISDIRTY);
+ RNA_def_property_clear_flag(prop, PROP_EDITABLE);
RNA_def_property_ui_text(prop, "Dirty", "Text file has been edited since last save");
prop = RNA_def_property(srna, "is_modified", PROP_BOOLEAN, PROP_NONE);
@@ -157,6 +158,7 @@
prop = RNA_def_property(srna, "is_in_memory", PROP_BOOLEAN, PROP_NONE);
RNA_def_property_boolean_sdna(prop, NULL, "flags", TXT_ISMEM);
+ RNA_def_property_clear_flag(prop, PROP_EDITABLE);
RNA_def_property_ui_text(prop, "Memory", "Text file is in memory, without a corresponding file on disk");
prop = RNA_def_property(srna, "use_module", PROP_BOOLEAN, PROP_NONE);
More information about the Bf-blender-cvs
mailing list