[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [51076] trunk/blender: Fixed compilation error on widows

Sergey Sharybin sergey.vfx at gmail.com
Fri Oct 5 12:05:26 CEST 2012


Revision: 51076
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=51076
Author:   nazgul
Date:     2012-10-05 10:05:26 +0000 (Fri, 05 Oct 2012)
Log Message:
-----------
Fixed compilation error on widows

Was caused by mixing up own C-API typedefs with OCIO's

Modified Paths:
--------------
    trunk/blender/intern/opencolorio/CMakeLists.txt
    trunk/blender/intern/opencolorio/fallback_impl.cc
    trunk/blender/intern/opencolorio/ocio_capi.cc
    trunk/blender/intern/opencolorio/ocio_capi.h
    trunk/blender/intern/opencolorio/ocio_impl.cc
    trunk/blender/intern/opencolorio/ocio_impl.h
    trunk/blender/source/blender/imbuf/intern/IMB_colormanagement_intern.h
    trunk/blender/source/blender/imbuf/intern/colormanagement.c

Modified: trunk/blender/intern/opencolorio/CMakeLists.txt
===================================================================
--- trunk/blender/intern/opencolorio/CMakeLists.txt	2012-10-05 07:31:30 UTC (rev 51075)
+++ trunk/blender/intern/opencolorio/CMakeLists.txt	2012-10-05 10:05:26 UTC (rev 51076)
@@ -62,4 +62,3 @@
 
 
 blender_add_lib(bf_intern_opencolorio "${SRC}" "${INC}" "${INC_SYS}")
-

Modified: trunk/blender/intern/opencolorio/fallback_impl.cc
===================================================================
--- trunk/blender/intern/opencolorio/fallback_impl.cc	2012-10-05 07:31:30 UTC (rev 51075)
+++ trunk/blender/intern/opencolorio/fallback_impl.cc	2012-10-05 10:05:26 UTC (rev 51076)
@@ -28,20 +28,18 @@
 #include "MEM_guardedalloc.h"
 #include "BLI_math_color.h"
 
-namespace OCIO_NAMESPACE {};
-
 #include "ocio_impl.h"
 
-#define CONFIG_DEFAULT           ((ConstConfigRcPtr*)1)
+#define CONFIG_DEFAULT           ((OCIO_ConstConfigRcPtr*)1)
 
-#define PROCESSOR_LINEAR_TO_SRGB ((ConstProcessorRcPtr*)1)
-#define PROCESSOR_SRGB_TO_LINEAR ((ConstProcessorRcPtr*)2)
-#define PROCESSOR_UNKNOWN        ((ConstProcessorRcPtr*)3)
+#define PROCESSOR_LINEAR_TO_SRGB ((OCIO_ConstProcessorRcPtr*)1)
+#define PROCESSOR_SRGB_TO_LINEAR ((OCIO_ConstProcessorRcPtr*)2)
+#define PROCESSOR_UNKNOWN        ((OCIO_ConstProcessorRcPtr*)3)
 
-#define COLORSPACE_LINEAR        ((ConstColorSpaceRcPtr*)1)
-#define COLORSPACE_SRGB          ((ConstColorSpaceRcPtr*)2)
+#define COLORSPACE_LINEAR        ((OCIO_ConstColorSpaceRcPtr*)1)
+#define COLORSPACE_SRGB          ((OCIO_ConstColorSpaceRcPtr*)2)
 
-typedef struct PackedImageDescription {
+typedef struct OCIO_PackedImageDescription {
 	float *data;
 	long width;
 	long height;
@@ -49,37 +47,37 @@
 	long chanStrideBytes;
 	long xStrideBytes;
 	long yStrideBytes;
-} PackedImageDescription;
+} OCIO_PackedImageDescription;
 
-ConstConfigRcPtr *FallbackImpl::getCurrentConfig(void)
+OCIO_ConstConfigRcPtr *FallbackImpl::getCurrentConfig(void)
 {
 	return CONFIG_DEFAULT;
 }
 
-void FallbackImpl::setCurrentConfig(const ConstConfigRcPtr *)
+void FallbackImpl::setCurrentConfig(const OCIO_ConstConfigRcPtr *)
 {
 }
 
-ConstConfigRcPtr *FallbackImpl::configCreateFromEnv(void)
+OCIO_ConstConfigRcPtr *FallbackImpl::configCreateFromEnv(void)
 {
 	return CONFIG_DEFAULT;
 }
 
-ConstConfigRcPtr *FallbackImpl::configCreateFromFile(const char *)
+OCIO_ConstConfigRcPtr *FallbackImpl::configCreateFromFile(const char *)
 {
 	return CONFIG_DEFAULT;
 }
 
-void FallbackImpl::configRelease(ConstConfigRcPtr *)
+void FallbackImpl::configRelease(OCIO_ConstConfigRcPtr *)
 {
 }
 
-int FallbackImpl::configGetNumColorSpaces(ConstConfigRcPtr *)
+int FallbackImpl::configGetNumColorSpaces(OCIO_ConstConfigRcPtr *)
 {
 	return 2;
 }
 
-const char *FallbackImpl::configGetColorSpaceNameByIndex(ConstConfigRcPtr *, int index)
+const char *FallbackImpl::configGetColorSpaceNameByIndex(OCIO_ConstConfigRcPtr *, int index)
 {
 	if (index == 0)
 		return "Linear";
@@ -89,7 +87,7 @@
 	return NULL;
 }
 
-ConstColorSpaceRcPtr *FallbackImpl::configGetColorSpace(ConstConfigRcPtr *, const char *name)
+OCIO_ConstColorSpaceRcPtr *FallbackImpl::configGetColorSpace(OCIO_ConstConfigRcPtr *, const char *name)
 {
 	if (strcmp(name, "scene_linear") == 0)
 		return COLORSPACE_LINEAR;
@@ -111,9 +109,9 @@
 	return NULL;
 }
 
-int FallbackImpl::configGetIndexForColorSpace(ConstConfigRcPtr *config, const char *name)
+int FallbackImpl::configGetIndexForColorSpace(OCIO_ConstConfigRcPtr *config, const char *name)
 {
-	ConstColorSpaceRcPtr *cs = configGetColorSpace(config, name);
+	OCIO_ConstColorSpaceRcPtr *cs = configGetColorSpace(config, name);
 
 	if (cs == COLORSPACE_LINEAR)
 		return 0;
@@ -123,17 +121,17 @@
 	return -1;
 }
 
-const char *FallbackImpl::configGetDefaultDisplay(ConstConfigRcPtr *)
+const char *FallbackImpl::configGetDefaultDisplay(OCIO_ConstConfigRcPtr *)
 {
 	return "sRGB";
 }
 
-int FallbackImpl::configGetNumDisplays(ConstConfigRcPtr* config)
+int FallbackImpl::configGetNumDisplays(OCIO_ConstConfigRcPtr* config)
 {
 	return 1;
 }
 
-const char *FallbackImpl::configGetDisplay(ConstConfigRcPtr *, int index)
+const char *FallbackImpl::configGetDisplay(OCIO_ConstConfigRcPtr *, int index)
 {
 	if (index == 0)
 		return "sRGB";
@@ -141,17 +139,17 @@
 	return NULL;
 }
 
-const char *FallbackImpl::configGetDefaultView(ConstConfigRcPtr *, const char *)
+const char *FallbackImpl::configGetDefaultView(OCIO_ConstConfigRcPtr *, const char *)
 {
 	return "Default";
 }
 
-int FallbackImpl::configGetNumViews(ConstConfigRcPtr *, const char *)
+int FallbackImpl::configGetNumViews(OCIO_ConstConfigRcPtr *, const char *)
 {
 	return 1;
 }
 
-const char *FallbackImpl::configGetView(ConstConfigRcPtr *, const char *, int index)
+const char *FallbackImpl::configGetView(OCIO_ConstConfigRcPtr *, const char *, int index)
 {
 	if (index == 0)
 		return "Default";
@@ -159,29 +157,29 @@
 	return NULL;
 }
 
-const char *FallbackImpl::configGetDisplayColorSpaceName(ConstConfigRcPtr *, const char *, const char *)
+const char *FallbackImpl::configGetDisplayColorSpaceName(OCIO_ConstConfigRcPtr *, const char *, const char *)
 {
 	return "sRGB";
 }
 
-int FallbackImpl::colorSpaceIsInvertible(ConstColorSpaceRcPtr *cs)
+int FallbackImpl::colorSpaceIsInvertible(OCIO_ConstColorSpaceRcPtr *cs)
 {
 	return 1;
 }
 
-int FallbackImpl::colorSpaceIsData(ConstColorSpaceRcPtr *cs)
+int FallbackImpl::colorSpaceIsData(OCIO_ConstColorSpaceRcPtr *cs)
 {
 	return 0;
 }
 
-void FallbackImpl::colorSpaceRelease(ConstColorSpaceRcPtr *cs)
+void FallbackImpl::colorSpaceRelease(OCIO_ConstColorSpaceRcPtr *cs)
 {
 }
 
-ConstProcessorRcPtr *FallbackImpl::configGetProcessorWithNames(ConstConfigRcPtr *config, const char *srcName, const char *dstName)
+OCIO_ConstProcessorRcPtr *FallbackImpl::configGetProcessorWithNames(OCIO_ConstConfigRcPtr *config, const char *srcName, const char *dstName)
 {
-	ConstColorSpaceRcPtr *cs_src = configGetColorSpace(config, srcName);
-	ConstColorSpaceRcPtr *cs_dst = configGetColorSpace(config, dstName);
+	OCIO_ConstColorSpaceRcPtr *cs_src = configGetColorSpace(config, srcName);
+	OCIO_ConstColorSpaceRcPtr *cs_dst = configGetColorSpace(config, dstName);
 
 	if (cs_src == COLORSPACE_LINEAR && cs_dst == COLORSPACE_SRGB)
 		return PROCESSOR_LINEAR_TO_SRGB;
@@ -191,15 +189,15 @@
 	return 0;
 }
 
-ConstProcessorRcPtr *FallbackImpl::configGetProcessor(ConstConfigRcPtr *, ConstTransformRcPtr *tfm)
+OCIO_ConstProcessorRcPtr *FallbackImpl::configGetProcessor(OCIO_ConstConfigRcPtr *, OCIO_ConstTransformRcPtr *tfm)
 {
-	return (ConstProcessorRcPtr*)tfm;
+	return (OCIO_ConstProcessorRcPtr*)tfm;
 }
 
-void FallbackImpl::processorApply(ConstProcessorRcPtr *processor, PackedImageDesc *img)
+void FallbackImpl::processorApply(OCIO_ConstProcessorRcPtr *processor, OCIO_PackedImageDesc *img)
 {
 	/* OCIO_TODO stride not respected, channels must be 3 or 4 */
-	PackedImageDescription *desc = (PackedImageDescription*)img;
+	OCIO_PackedImageDescription *desc = (OCIO_PackedImageDescription*)img;
 	int channels = desc->numChannels;
 	float *pixels = desc->data;
 	int width = desc->width;
@@ -218,10 +216,10 @@
 	}
 }
 
-void FallbackImpl::processorApply_predivide(ConstProcessorRcPtr *processor, PackedImageDesc *img)
+void FallbackImpl::processorApply_predivide(OCIO_ConstProcessorRcPtr *processor, OCIO_PackedImageDesc *img)
 {
 	/* OCIO_TODO stride not respected, channels must be 3 or 4 */
-	PackedImageDescription *desc = (PackedImageDescription*)img;
+	OCIO_PackedImageDescription *desc = (OCIO_PackedImageDescription*)img;
 	int channels = desc->numChannels;
 	float *pixels = desc->data;
 	int width = desc->width;
@@ -240,7 +238,7 @@
 	}
 }
 
-void FallbackImpl::processorApplyRGB(ConstProcessorRcPtr *processor, float *pixel)
+void FallbackImpl::processorApplyRGB(OCIO_ConstProcessorRcPtr *processor, float *pixel)
 {
 	if (processor == PROCESSOR_LINEAR_TO_SRGB)
 		linearrgb_to_srgb_v3_v3(pixel, pixel);
@@ -248,7 +246,7 @@
 		srgb_to_linearrgb_v3_v3(pixel, pixel);
 }
 
-void FallbackImpl::processorApplyRGBA(ConstProcessorRcPtr *processor, float *pixel)
+void FallbackImpl::processorApplyRGBA(OCIO_ConstProcessorRcPtr *processor, float *pixel)
 {
 	if (processor == PROCESSOR_LINEAR_TO_SRGB)
 		linearrgb_to_srgb_v4(pixel, pixel);
@@ -256,7 +254,7 @@
 		srgb_to_linearrgb_v4(pixel, pixel);
 }
 
-void FallbackImpl::processorApplyRGBA_predivide(ConstProcessorRcPtr *processor, float *pixel)
+void FallbackImpl::processorApplyRGBA_predivide(OCIO_ConstProcessorRcPtr *processor, float *pixel)
 {
 	if (pixel[3] == 1.0f || pixel[3] == 0.0f) {
 		processorApplyRGBA(processor, pixel);
@@ -279,11 +277,11 @@
 	}
 }
 
-void FallbackImpl::processorRelease(ConstProcessorRcPtr *)
+void FallbackImpl::processorRelease(OCIO_ConstProcessorRcPtr *)
 {
 }
 
-const char *FallbackImpl::colorSpaceGetName(ConstColorSpaceRcPtr *cs)
+const char *FallbackImpl::colorSpaceGetName(OCIO_ConstColorSpaceRcPtr *cs)
 {
 	if (cs == COLORSPACE_LINEAR)
 		return "Linear";
@@ -293,49 +291,49 @@
 	return NULL;
 }
 
-const char *FallbackImpl::colorSpaceGetDescription(ConstColorSpaceRcPtr *)
+const char *FallbackImpl::colorSpaceGetDescription(OCIO_ConstColorSpaceRcPtr *)
 {
 	return "";
 }
 
-const char *FallbackImpl::colorSpaceGetFamily(ConstColorSpaceRcPtr *)
+const char *FallbackImpl::colorSpaceGetFamily(OCIO_ConstColorSpaceRcPtr *)
 {
 	return "";
 }
 
-DisplayTransformRcPtr *FallbackImpl::createDisplayTransform(void)
+OCIO_DisplayTransformRcPtr *FallbackImpl::createDisplayTransform(void)
 {
-	return (DisplayTransformRcPtr*)PROCESSOR_LINEAR_TO_SRGB;
+	return (OCIO_DisplayTransformRcPtr*)PROCESSOR_LINEAR_TO_SRGB;
 }
 
-void FallbackImpl::displayTransformSetInputColorSpaceName(DisplayTransformRcPtr *, const char *)
+void FallbackImpl::displayTransformSetInputColorSpaceName(OCIO_DisplayTransformRcPtr *, const char *)
 {
 }
 
-void FallbackImpl::displayTransformSetDisplay(DisplayTransformRcPtr *, const char *)
+void FallbackImpl::displayTransformSetDisplay(OCIO_DisplayTransformRcPtr *, const char *)
 {
 }
 
-void FallbackImpl::displayTransformSetView(DisplayTransformRcPtr *, const char *)
+void FallbackImpl::displayTransformSetView(OCIO_DisplayTransformRcPtr *, const char *)
 {
 }
 
-void FallbackImpl::displayTransformSetDisplayCC(DisplayTransformRcPtr *, ConstTransformRcPtr *)
+void FallbackImpl::displayTransformSetDisplayCC(OCIO_DisplayTransformRcPtr *, OCIO_ConstTransformRcPtr *)
 {
 }
 
-void FallbackImpl::displayTransformSetLinearCC(DisplayTransformRcPtr *, ConstTransformRcPtr *)

@@ Diff output truncated at 10240 characters. @@



More information about the Bf-blender-cvs mailing list