[Bf-blender-cvs] [0aa68f5] cycles_disney_brdf: Resolved inconsistencies in using tabs and spaces

Pascal Schoen noreply at git.blender.org
Mon Nov 7 12:06:25 CET 2016


Commit: 0aa68f533529c9fd197a3ab0427f9e41a15456b9
Author: Pascal Schoen
Date:   Mon Nov 7 12:04:38 2016 +0100
Branches: cycles_disney_brdf
https://developer.blender.org/rB0aa68f533529c9fd197a3ab0427f9e41a15456b9

Resolved inconsistencies in using tabs and spaces

===================================================================

M	intern/cycles/blender/blender_shader.cpp
M	intern/cycles/kernel/CMakeLists.txt
M	intern/cycles/kernel/closure/bsdf.h
M	intern/cycles/kernel/closure/bsdf_disney_diffuse.h
M	intern/cycles/kernel/closure/bsdf_disney_sheen.h
M	intern/cycles/kernel/closure/bsdf_microfacet.h
M	intern/cycles/kernel/closure/bsdf_microfacet_multi_impl.h
M	intern/cycles/kernel/closure/bsdf_util.h
M	intern/cycles/kernel/shaders/CMakeLists.txt
M	intern/cycles/kernel/shaders/node_disney_bsdf.osl
M	intern/cycles/kernel/svm/svm_closure.h
M	intern/cycles/kernel/svm/svm_types.h
M	source/blender/nodes/CMakeLists.txt
M	source/blender/nodes/shader/nodes/node_shader_bsdf_disney.c

===================================================================

diff --git a/intern/cycles/blender/blender_shader.cpp b/intern/cycles/blender/blender_shader.cpp
index fc13fae..cb50b74 100644
--- a/intern/cycles/blender/blender_shader.cpp
+++ b/intern/cycles/blender/blender_shader.cpp
@@ -529,7 +529,7 @@ static ShaderNode *add_node(Scene *scene,
 				break;
 		}
 		node = disney;
-    }
+	}
 	else if(b_node.is_a(&RNA_ShaderNodeBsdfTranslucent)) {
 		node = new TranslucentBsdfNode();
 	}
diff --git a/intern/cycles/kernel/CMakeLists.txt b/intern/cycles/kernel/CMakeLists.txt
index 6679797..73485ca 100644
--- a/intern/cycles/kernel/CMakeLists.txt
+++ b/intern/cycles/kernel/CMakeLists.txt
@@ -108,8 +108,8 @@ set(SRC_CLOSURE_HEADERS
 	closure/bssrdf.h
 	closure/emissive.h
 	closure/volume.h
-    closure/bsdf_disney_diffuse.h
-    closure/bsdf_disney_sheen.h
+	closure/bsdf_disney_diffuse.h
+	closure/bsdf_disney_sheen.h
 )
 
 set(SRC_SVM_HEADERS
diff --git a/intern/cycles/kernel/closure/bsdf.h b/intern/cycles/kernel/closure/bsdf.h
index 31bd840..b7c53df 100644
--- a/intern/cycles/kernel/closure/bsdf.h
+++ b/intern/cycles/kernel/closure/bsdf.h
@@ -93,7 +93,6 @@ ccl_device_forceinline int bsdf_sample(KernelGlobals *kg,
 			label = bsdf_microfacet_ggx_sample(kg, sc, ccl_fetch(sd, Ng), ccl_fetch(sd, I), ccl_fetch(sd, dI).dx, ccl_fetch(sd, dI).dy, randu, randv,
 				eval, omega_in, &domega_in->dx, &domega_in->dy, pdf);
 			break;
-		case CLOSURE_BSDF_MICROFACET_MULTI_GGX_REFRACTION_ID:
 		case CLOSURE_BSDF_MICROFACET_MULTI_GGX_ID:
 			label = bsdf_microfacet_multi_ggx_sample(kg, sc, ccl_fetch(sd, Ng), ccl_fetch(sd, I), ccl_fetch(sd, dI).dx, ccl_fetch(sd, dI).dy, randu, randv,
 			        eval, omega_in,  &domega_in->dx, &domega_in->dy, pdf, &ccl_fetch(sd, lcg_state));
@@ -280,7 +279,6 @@ float3 bsdf_eval(KernelGlobals *kg,
 				eval = bsdf_microfacet_ggx_eval_transmit(sc, ccl_fetch(sd, I), omega_in, pdf);
 				break;
 			case CLOSURE_BSDF_MICROFACET_MULTI_GGX_ID:
-			case CLOSURE_BSDF_MICROFACET_MULTI_GGX_REFRACTION_ID:
 				eval = bsdf_microfacet_multi_ggx_eval_transmit(sc, ccl_fetch(sd, I), omega_in, pdf, &ccl_fetch(sd, lcg_state));
 				break;
 			case CLOSURE_BSDF_MICROFACET_MULTI_GGX_GLASS_ID:
diff --git a/intern/cycles/kernel/closure/bsdf_disney_diffuse.h b/intern/cycles/kernel/closure/bsdf_disney_diffuse.h
index 4469646..ccb5966 100644
--- a/intern/cycles/kernel/closure/bsdf_disney_diffuse.h
+++ b/intern/cycles/kernel/closure/bsdf_disney_diffuse.h
@@ -37,15 +37,15 @@ ccl_device float3 calculate_disney_diffuse_brdf(const DisneyDiffuseBsdf *bsdf,
 	float NdotL = max(dot(N, L), 0.0f);
 	float NdotV = max(dot(N, V), 0.0f);
 
-    if(NdotL < 0 || NdotV < 0) {
-        *pdf = 0.0f;
-        return make_float3(0.0f, 0.0f, 0.0f);
-    }
+	if(NdotL < 0 || NdotV < 0) {
+		*pdf = 0.0f;
+		return make_float3(0.0f, 0.0f, 0.0f);
+	}
 
 	float LdotH = dot(L, H);
 
 	float FL = schlick_fresnel(NdotL), FV = schlick_fresnel(NdotV);
-    const float Fd90 = 0.5f + 2.0f * LdotH*LdotH * bsdf->roughness;
+	const float Fd90 = 0.5f + 2.0f * LdotH*LdotH * bsdf->roughness;
 	float Fd = (1.0f * (1.0f - FL) + Fd90 * FL) * (1.0f * (1.0f - FV) + Fd90 * FV);
 
 	float value = M_1_PI_F * NdotL * Fd;
@@ -69,14 +69,14 @@ ccl_device float3 bsdf_disney_diffuse_eval_reflect(const ShaderClosure *sc, cons
 	float3 L = omega_in; // incoming
 	float3 H = normalize(L + V);
 
-    if(dot(N, omega_in) > 0.0f) {
-        *pdf = fmaxf(dot(N, omega_in), 0.0f) * M_1_PI_F;
-        return calculate_disney_diffuse_brdf(bsdf, N, V, L, H, pdf);
-    }
-    else {
-        *pdf = 0.0f;
-        return make_float3(0.0f, 0.0f, 0.0f);
-    }
+	if(dot(N, omega_in) > 0.0f) {
+		*pdf = fmaxf(dot(N, omega_in), 0.0f) * M_1_PI_F;
+		return calculate_disney_diffuse_brdf(bsdf, N, V, L, H, pdf);
+	}
+	else {
+		*pdf = 0.0f;
+		return make_float3(0.0f, 0.0f, 0.0f);
+	}
 }
 
 ccl_device float3 bsdf_disney_diffuse_eval_transmit(const ShaderClosure *sc, const float3 I,
diff --git a/intern/cycles/kernel/closure/bsdf_disney_sheen.h b/intern/cycles/kernel/closure/bsdf_disney_sheen.h
index 65283a7..3fe111c 100644
--- a/intern/cycles/kernel/closure/bsdf_disney_sheen.h
+++ b/intern/cycles/kernel/closure/bsdf_disney_sheen.h
@@ -35,10 +35,10 @@ ccl_device float3 calculate_disney_sheen_brdf(const DisneySheenBsdf *bsdf,
 	float NdotL = dot(N, L);
 	float NdotV = dot(N, V);
 
-    if(NdotL < 0 || NdotV < 0) {
-        *pdf = 0.0f;
-        return make_float3(0.0f, 0.0f, 0.0f);
-    }
+	if(NdotL < 0 || NdotV < 0) {
+		*pdf = 0.0f;
+		return make_float3(0.0f, 0.0f, 0.0f);
+	}
 
 	float LdotH = dot(L, H);
 
@@ -63,14 +63,14 @@ ccl_device float3 bsdf_disney_sheen_eval_reflect(const ShaderClosure *sc, const
 	float3 L = omega_in; // incoming
 	float3 H = normalize(L + V);
 
-    if(dot(N, omega_in) > 0.0f) {
-        *pdf = fmaxf(dot(N, omega_in), 0.0f) * M_1_PI_F;
-        return calculate_disney_sheen_brdf(bsdf, N, V, L, H, pdf);
-    }
-    else {
-        *pdf = 0.0f;
-        return make_float3(0.0f, 0.0f, 0.0f);
-    }
+	if(dot(N, omega_in) > 0.0f) {
+		*pdf = fmaxf(dot(N, omega_in), 0.0f) * M_1_PI_F;
+		return calculate_disney_sheen_brdf(bsdf, N, V, L, H, pdf);
+	}
+	else {
+		*pdf = 0.0f;
+		return make_float3(0.0f, 0.0f, 0.0f);
+	}
 }
 
 ccl_device float3 bsdf_disney_sheen_eval_transmit(const ShaderClosure *sc, const float3 I,
diff --git a/intern/cycles/kernel/closure/bsdf_microfacet.h b/intern/cycles/kernel/closure/bsdf_microfacet.h
index 213135b..1e83f04 100644
--- a/intern/cycles/kernel/closure/bsdf_microfacet.h
+++ b/intern/cycles/kernel/closure/bsdf_microfacet.h
@@ -243,23 +243,23 @@ ccl_device_forceinline float3 microfacet_sample_stretched(
  * Else it is simply white
  */
 ccl_device_forceinline float3 reflection_color(const MicrofacetBsdf *bsdf, float3 L, float3 H) {
-    float3 F = make_float3(1.0f, 1.0f, 1.0f);
+	float3 F = make_float3(1.0f, 1.0f, 1.0f);
 
-    if(bsdf->extra && bsdf->extra->use_fresnel) {
+	if(bsdf->extra && bsdf->extra->use_fresnel) {
 		float F0 = fresnel_dielectric_cos(1.0f, bsdf->ior);
 
 		F = interpolate_fresnel_color(L, H, bsdf->ior, F0, bsdf->extra->cspec0);
-    }
+	}
 
-    return F;
+	return F;
 }
 
 ccl_device_forceinline float D_GTR1(float NdotH, float alpha)
 {
-    if (alpha >= 1.0f) return M_1_PI_F;
+	if (alpha >= 1.0f) return M_1_PI_F;
 	float alpha2 = alpha*alpha;
-    float t = 1.0f + (alpha2 - 1.0f) * NdotH*NdotH;
-    return (alpha2 - 1.0f) / (M_1_PI_F * logf(alpha2) * t);
+	float t = 1.0f + (alpha2 - 1.0f) * NdotH*NdotH;
+	return (alpha2 - 1.0f) / (M_1_PI_F * logf(alpha2) * t);
 }
 
 /* GGX microfacet with Smith shadow-masking from:
@@ -277,7 +277,7 @@ ccl_device_forceinline float D_GTR1(float NdotH, float alpha)
 
 ccl_device int bsdf_microfacet_ggx_setup(MicrofacetBsdf *bsdf)
 {
-    bsdf->extra = NULL;
+	bsdf->extra = NULL;
 
 	bsdf->alpha_x = saturate(bsdf->alpha_x);
 	bsdf->alpha_y = bsdf->alpha_x;
@@ -289,12 +289,12 @@ ccl_device int bsdf_microfacet_ggx_setup(MicrofacetBsdf *bsdf)
 
 ccl_device int bsdf_microfacet_ggx_fresnel_setup(MicrofacetBsdf *bsdf)
 {
-    bsdf->extra->use_fresnel = true;
-    bsdf->extra->is_disney_clearcoat = false;
+	bsdf->extra->use_fresnel = true;
+	bsdf->extra->is_disney_clearcoat = false;
 
-    bsdf->extra->cspec0.x = saturate(bsdf->extra->cspec0.x);
-    bsdf->extra->cspec0.y = saturate(bsdf->extra->cspec0.y);
-    bsdf->extra->cspec0.z = saturate(bsdf->extra->cspec0.z);
+	bsdf->extra->cspec0.x = saturate(bsdf->extra->cspec0.x);
+	bsdf->extra->cspec0.y = saturate(bsdf->extra->cspec0.y);
+	bsdf->extra->cspec0.z = saturate(bsdf->extra->cspec0.z);
 
 	bsdf->alpha_x = saturate(bsdf->alpha_x);
 	bsdf->alpha_y = bsdf->alpha_x;
@@ -306,12 +306,12 @@ ccl_device int bsdf_microfacet_ggx_fresnel_setup(MicrofacetBsdf *bsdf)
 
 ccl_device int bsdf_microfacet_ggx_clearcoat_setup(MicrofacetBsdf *bsdf)
 {
-    bsdf->extra->use_fresnel = true;
-    bsdf->extra->is_disney_clearcoat = true;
+	bsdf->extra->use_fresnel = true;
+	bsdf->extra->is_disney_clearcoat = true;
 
-    bsdf->extra->cspec0.x = saturate(bsdf->extra->cspec0.x);
-    bsdf->extra->cspec0.y = saturate(bsdf->extra->cspec0.y);
-    bsdf->extra->cspec0.z = saturate(bsdf->extra->cspec0.z);
+	bsdf->extra->cspec0.x = saturate(bsdf->extra->cspec0.x);
+	bsdf->extra->cspec0.y = saturate(bsdf->extra->cspec0.y);
+	bsdf->extra->cspec0.z = saturate(bsdf->extra->cspec0.z);
 
 	bsdf->alpha_x = saturate(bsdf->alpha_x);
 	bsdf->alpha_y = bsdf->alpha_x;
@@ -338,7 +338,7 @@ ccl_device bool bsdf_microfacet_merge(const ShaderClosure *a, const ShaderClosur
 
 ccl_device int bsdf_microfacet_ggx_aniso_setup(MicrofacetBsdf *bsdf)
 {
-    bsdf->extra = NULL;
+	bsdf->extra = NULL;
 
 	bsdf->alpha_x = saturate(bsdf->alpha_x);
 	bsdf->alpha_y = saturate(bsdf->alpha_y);
@@ -350,12 +350,12 @@ ccl_device int bsdf_microfacet_ggx_aniso_setup(MicrofacetBsdf *bsdf)
 
 ccl_device int bsdf_microfacet_ggx_aniso_fresnel_setup(MicrofacetBsdf *bsdf)
 {
-    bsdf->extra->use_fresnel = true;
-    bsdf->extra->is_disney_clearcoat = false;
+	bsdf->extra->use_fresnel = true;
+	bsdf->extra->is_disney_clearcoat = false;
 
-    bsdf->extra->cspec0.x = saturate(bsdf->extra->cspec0.x);
-    bsdf->extra->cspec0.y = saturate(bsdf->extra->cspec0.y);
-    bsdf->extra->cspec0.z = saturate(bsdf->extra->cspec0.z);
+	bsdf->extra->cspec0.x = saturate(bsdf->extra->cspec0.x);
+	bsdf->extra->cspec0.y = saturate(bsdf->extra->cspec0.y);
+	bsdf->extra->cspec0.z = saturate(bsdf->extra->cspec0.z);
 
 	bsdf->alpha_x = saturate(bsdf->alpha_x);
 	bsdf->alpha_y = saturate(bsdf->alpha_y);
@@ -367,7 +367,7 @@ ccl_device int bsdf_microfacet_ggx_aniso_fresnel_setup(MicrofacetBsdf *bsdf)
 
 ccl_device int bsdf_microfacet_ggx_refraction_setup(MicrofacetBsdf *bsdf)
 {
-    bsdf->extra = NULL;
+	bsdf->extra = NULL;
 
 	bsdf->alpha_x = saturate(bsdf->alpha_x);
 	bsdf->alpha_y = bsdf->alpha_x;
@@ -420,7 +420,7 @@ ccl_device float3 bsdf_microfacet_ggx_eval_reflect(const ShaderClosure *sc, cons
 
 				/* the alpha value for clearcoat is a fixed 0.25 => alpha2 = 0.25 * 0.25 */
 				alpha2 = 0.0625f;
-            }
+			}
 			else {
 				/* use GTR2 otherwise */
 				D = alpha2 / (M_PI_F * cosThetaM4 * (alpha2 + tanThetaM2) * (alpha2 + tanThetaM2));
@@ -472,7 +472,7 @@ ccl_device float3 bsdf_microfacet_ggx_eval_reflect(const ShaderClosure *sc,

@@ Diff output truncated at 10240 characters. @@




More information about the Bf-blender-cvs mailing list