[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