[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [31007] trunk/blender/source/blender: minor cleanup, minimal functional change.

Campbell Barton ideasman42 at gmail.com
Tue Aug 3 14:34:42 CEST 2010


Revision: 31007
          http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=31007
Author:   campbellbarton
Date:     2010-08-03 14:34:42 +0200 (Tue, 03 Aug 2010)

Log Message:
-----------
minor cleanup, minimal functional change.
replace some long duplicated, ifdef'd if statements for image extension.
- new function: BLI_testextensie_array(), can take an array of extensions.
- define extension arrays: imb_ext_image, imb_ext_movie, imb_ext_sound - we could have more of these.
- removed amiga extensions iff and lbm

Modified Paths:
--------------
    trunk/blender/source/blender/blenlib/BLI_path_util.h
    trunk/blender/source/blender/blenlib/intern/path_util.c
    trunk/blender/source/blender/editors/space_file/filelist.c
    trunk/blender/source/blender/imbuf/IMB_imbuf_types.h
    trunk/blender/source/blender/imbuf/intern/util.c

Modified: trunk/blender/source/blender/blenlib/BLI_path_util.h
===================================================================
--- trunk/blender/source/blender/blenlib/BLI_path_util.h	2010-08-03 11:25:34 UTC (rev 31006)
+++ trunk/blender/source/blender/blenlib/BLI_path_util.h	2010-08-03 12:34:42 UTC (rev 31007)
@@ -99,6 +99,7 @@
 int BKE_rebase_path(char *abs, int abs_size, char *rel, int rel_size, const char *base_dir, const char *src_dir, const char *dest_dir);
 void BLI_getlastdir(const char* dir, char *last, int maxlen);
 int BLI_testextensie(const char *str, const char *ext);
+int BLI_testextensie_array(const char *str, const char **ext_array);
 int BLI_replace_extension(char *path, int maxlen, const char *ext);
 void BLI_uniquename(struct ListBase *list, void *vlink, const char defname[], char delim, short name_offs, short len);
 void BLI_newname(char * name, int add);

Modified: trunk/blender/source/blender/blenlib/intern/path_util.c
===================================================================
--- trunk/blender/source/blender/blenlib/intern/path_util.c	2010-08-03 11:25:34 UTC (rev 31006)
+++ trunk/blender/source/blender/blenlib/intern/path_util.c	2010-08-03 12:34:42 UTC (rev 31007)
@@ -1263,6 +1263,19 @@
 	return (retval);
 }
 
+int BLI_testextensie_array(const char *str, const char **ext_array)
+{
+	int i=0;
+	while(ext_array[i]) {
+		if(BLI_testextensie(str, ext_array[i])) {
+			return 1;
+		}
+
+		i++;
+	}
+	return 0;
+}
+
 int BLI_replace_extension(char *path, int maxlen, const char *ext)
 {
 	int a;

Modified: trunk/blender/source/blender/editors/space_file/filelist.c
===================================================================
--- trunk/blender/source/blender/editors/space_file/filelist.c	2010-08-03 11:25:34 UTC (rev 31006)
+++ trunk/blender/source/blender/editors/space_file/filelist.c	2010-08-03 12:34:42 UTC (rev 31007)
@@ -828,153 +828,13 @@
 				file->flags |= BTXFILE;
 		} else if(BLI_testextensie(file->relname, ".dae")) {
 			file->flags |= COLLADAFILE;
-		} else if (has_quicktime){
-			if(		BLI_testextensie(file->relname, ".int")
-				||  BLI_testextensie(file->relname, ".inta")
-				||  BLI_testextensie(file->relname, ".jpg")
-#ifdef WITH_OPENJPEG
-				||  BLI_testextensie(file->relname, ".jp2")
-#endif
-				||	BLI_testextensie(file->relname, ".jpeg")
-				||	BLI_testextensie(file->relname, ".tga")
-				||	BLI_testextensie(file->relname, ".rgb")
-				||	BLI_testextensie(file->relname, ".rgba")
-				||	BLI_testextensie(file->relname, ".bmp")
-				||	BLI_testextensie(file->relname, ".png")
-				||	BLI_testextensie(file->relname, ".iff")
-				||	BLI_testextensie(file->relname, ".lbm")
-				||	BLI_testextensie(file->relname, ".gif")
-				||	BLI_testextensie(file->relname, ".psd")
-				||	BLI_testextensie(file->relname, ".tif")
-				||	BLI_testextensie(file->relname, ".tiff")
-				||	BLI_testextensie(file->relname, ".tx")
-				||	BLI_testextensie(file->relname, ".pct")
-				||	BLI_testextensie(file->relname, ".pict")
-				||	BLI_testextensie(file->relname, ".pntg") //macpaint
-				||	BLI_testextensie(file->relname, ".qtif")
-				||	BLI_testextensie(file->relname, ".sgi")
-#ifdef WITH_HDR
-				||	BLI_testextensie(file->relname, ".hdr")
-#endif
-#ifdef WITH_CINEON
-				||	BLI_testextensie(file->relname, ".cin")
-				||	BLI_testextensie(file->relname, ".dpx")
-#endif
-#ifdef WITH_DDS
-				||	BLI_testextensie(file->relname, ".dds")
-#endif
-#ifdef WITH_OPENEXR
-				||	BLI_testextensie(file->relname, ".exr")
-#endif
-				) {
+		} else if(BLI_testextensie_array(file->relname, imb_ext_image)
+					|| (has_quicktime && BLI_testextensie_array(file->relname, imb_ext_image_qt))) {
 				file->flags |= IMAGEFILE;			
-			}
-			else if(BLI_testextensie(file->relname, ".avi")
-				||	BLI_testextensie(file->relname, ".flc")
-				||	BLI_testextensie(file->relname, ".mov")
-				||	BLI_testextensie(file->relname, ".movie")
-				||	BLI_testextensie(file->relname, ".mp4")
-				||	BLI_testextensie(file->relname, ".m4v")
-				||	BLI_testextensie(file->relname, ".m2v")
-				||	BLI_testextensie(file->relname, ".m2t")
-				||	BLI_testextensie(file->relname, ".mts")
-				||	BLI_testextensie(file->relname, ".mv")
-				||	BLI_testextensie(file->relname, ".avs")
-				||	BLI_testextensie(file->relname, ".wmv")
-				||	BLI_testextensie(file->relname, ".ogv")
-				||	BLI_testextensie(file->relname, ".dv")
-				||	BLI_testextensie(file->relname, ".mpeg")
-				||	BLI_testextensie(file->relname, ".mpg")
-				||	BLI_testextensie(file->relname, ".mpg2")
-				||	BLI_testextensie(file->relname, ".vob")
-				||	BLI_testextensie(file->relname, ".mkv")
-				||	BLI_testextensie(file->relname, ".flv")
-				||	BLI_testextensie(file->relname, ".divx")
-				||	BLI_testextensie(file->relname, ".xvid")) {
-				file->flags |= MOVIEFILE;			
-			}
-			else if(BLI_testextensie(file->relname, ".wav")
-				||	BLI_testextensie(file->relname, ".ogg")
-				||	BLI_testextensie(file->relname, ".oga")
-				||	BLI_testextensie(file->relname, ".mp3")
-				||	BLI_testextensie(file->relname, ".mp2")
-				||	BLI_testextensie(file->relname, ".ac3")
-				||	BLI_testextensie(file->relname, ".aac")
-				||	BLI_testextensie(file->relname, ".flac")
-				||	BLI_testextensie(file->relname, ".wma")
-				||	BLI_testextensie(file->relname, ".eac3")) {
-				file->flags |= SOUNDFILE;
-			}
-		} else { // no quicktime
-			if(BLI_testextensie(file->relname, ".int")
-				||	BLI_testextensie(file->relname, ".inta")
-				||	BLI_testextensie(file->relname, ".jpg")
-				||  BLI_testextensie(file->relname, ".jpeg")
-#ifdef WITH_OPENJPEG
-				||  BLI_testextensie(file->relname, ".jp2")
-#endif
-				||	BLI_testextensie(file->relname, ".tga")
-				||	BLI_testextensie(file->relname, ".rgb")
-				||	BLI_testextensie(file->relname, ".rgba")
-				||	BLI_testextensie(file->relname, ".bmp")
-				||	BLI_testextensie(file->relname, ".png")
-				||	BLI_testextensie(file->relname, ".iff")
-				||	BLI_testextensie(file->relname, ".tif")
-				||	BLI_testextensie(file->relname, ".tiff")
-				||	BLI_testextensie(file->relname, ".tx")
-#ifdef WITH_HDR
-				||	BLI_testextensie(file->relname, ".hdr")
-#endif
-#ifdef WITH_CINEON				
-				||	BLI_testextensie(file->relname, ".cin")
-				||	BLI_testextensie(file->relname, ".dpx")
-#endif
-#ifdef WITH_DDS
-				||	BLI_testextensie(file->relname, ".dds")
-#endif
-#ifdef WITH_OPENEXR
-				||	BLI_testextensie(file->relname, ".exr")
-#endif
-				||	BLI_testextensie(file->relname, ".lbm")
-				||	BLI_testextensie(file->relname, ".sgi")) {
-				file->flags |= IMAGEFILE;			
-			}
-			else if(BLI_testextensie(file->relname, ".avi")
-				||	BLI_testextensie(file->relname, ".flc")
-				||	BLI_testextensie(file->relname, ".mov")
-				||	BLI_testextensie(file->relname, ".movie")
-				||	BLI_testextensie(file->relname, ".mp4")
-				||	BLI_testextensie(file->relname, ".m4v")
-				||	BLI_testextensie(file->relname, ".m2v")
-				||	BLI_testextensie(file->relname, ".m2t")
-				||	BLI_testextensie(file->relname, ".mts")
-				||	BLI_testextensie(file->relname, ".mv")
-				||	BLI_testextensie(file->relname, ".avs")
-				||	BLI_testextensie(file->relname, ".wmv")
-				||	BLI_testextensie(file->relname, ".ogv")
-				||	BLI_testextensie(file->relname, ".dv")
-				||	BLI_testextensie(file->relname, ".mpeg")
-				||	BLI_testextensie(file->relname, ".mpg")
-				||	BLI_testextensie(file->relname, ".mpg2")
-				||	BLI_testextensie(file->relname, ".vob")
-				||	BLI_testextensie(file->relname, ".mkv")
-				||	BLI_testextensie(file->relname, ".flv")
-				||	BLI_testextensie(file->relname, ".divx")
-				||	BLI_testextensie(file->relname, ".xvid")) {
-				file->flags |= MOVIEFILE;			
-			}
-			else if(BLI_testextensie(file->relname, ".wav")
-				||	BLI_testextensie(file->relname, ".ogg")
-				||	BLI_testextensie(file->relname, ".oga")
-				||	BLI_testextensie(file->relname, ".mp3")
-				||	BLI_testextensie(file->relname, ".mp2")
-				||	BLI_testextensie(file->relname, ".ac3")
-				||	BLI_testextensie(file->relname, ".aac")
-				||	BLI_testextensie(file->relname, ".flac")
-				||	BLI_testextensie(file->relname, ".wma")
-				||	BLI_testextensie(file->relname, ".eac3")) {
-				file->flags |= SOUNDFILE;
-			}
+		} else if(BLI_testextensie_array(file->relname, imb_ext_movie)) {
+			file->flags |= MOVIEFILE;			
+		} else if(BLI_testextensie_array(file->relname, imb_ext_audio)) {
+			file->flags |= SOUNDFILE;
 		}
 	}
 }

Modified: trunk/blender/source/blender/imbuf/IMB_imbuf_types.h
===================================================================
--- trunk/blender/source/blender/imbuf/IMB_imbuf_types.h	2010-08-03 11:25:34 UTC (rev 31006)
+++ trunk/blender/source/blender/imbuf/IMB_imbuf_types.h	2010-08-03 12:34:42 UTC (rev 31007)
@@ -222,5 +222,9 @@
 #define IB_PROFILE_SRGB			2
 #define IB_PROFILE_CUSTOM		3
 
+extern const char *imb_ext_image[];
+extern const char *imb_ext_image_qt[];
+extern const char *imb_ext_movie[];
+extern const char *imb_ext_audio[];
+
 #endif
-

Modified: trunk/blender/source/blender/imbuf/intern/util.c
===================================================================
--- trunk/blender/source/blender/imbuf/intern/util.c	2010-08-03 11:25:34 UTC (rev 31006)
+++ trunk/blender/source/blender/imbuf/intern/util.c	2010-08-03 12:34:42 UTC (rev 31007)
@@ -68,6 +68,80 @@
 
 #define UTIL_DEBUG 0
 
+const char *imb_ext_image[] = {
+	".png"
+	".tga",
+	".bmp",
+	".jpg", ".jpeg",
+	".sgi", ".rgb", ".rgba",
+#ifdef WITH_TIFF
+	".tif", ".tiff", ".tx",
+#endif
+#ifdef WITH_BF_OPENJPEG
+	".jp2",
+#endif
+#ifdef WITH_HDR
+	".hdr",
+#endif
+#ifdef WITH_DDS
+	".dds",
+#endif
+#ifdef WITH_CINEON
+	".dpx",
+	".cin",
+#endif
+#ifdef WITH_BF_OPENEXR
+	".exr",
+#endif
+	NULL};
+
+const char *imb_ext_image_qt[] = {
+	".gif",
+	".psd",
+	".pct", ".pict",
+	".pntg",
+	".qtif",
+	NULL};
+
+const char *imb_ext_movie[] = {
+	".avi",
+	".flc",
+	".mov",
+	".movie",
+	".mp4",
+	".m4v",
+	".m2v",
+	".m2t",
+	".mts",
+	".mv",
+	".avs",
+	".wmv",
+	".ogv",
+	".dv",
+	".mpeg",
+	".mpg",
+	".mpg2",
+	".vob",
+	".mkv",
+	".flv",
+	".divx",
+	".xvid",
+	NULL};
+
+/* sort of wrong being here... */
+const char *imb_ext_audio[] = {
+	".wav",
+	".ogg",
+	".oga",
+	".mp3",
+	".mp2",
+	".ac3",
+	".aac",
+	".flac",
+	".wma",
+	".eac3",
+	NULL};
+
 static int IMB_ispic_name(char *name)
 {
 	ImFileType *type;
@@ -105,90 +179,16 @@
 int IMB_ispic(char *filename)
 {
 	if(U.uiflag & USER_FILTERFILEEXTS) {
-		if (BLI_testextensie(filename, ".tif")
-				||	BLI_testextensie(filename, ".tiff")
-				||	BLI_testextensie(filename, ".tx")) {

@@ Diff output truncated at 10240 characters. @@




More information about the Bf-blender-cvs mailing list