Revision: 56944
          
http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=56944
Author:   dingto
Date:     2013-05-21 13:15:27 +0000 (Tue, 21 May 2013)
Log Message:
-----------
Code cleanup / Cycles:
* Re-use already defined Westin closure class. 

Modified Paths:
--------------
    trunk/blender/intern/cycles/kernel/closure/bsdf.h
    trunk/blender/intern/cycles/kernel/osl/CMakeLists.txt

Removed Paths:
-------------
    trunk/blender/intern/cycles/kernel/osl/bsdf_westin.cpp

Modified: trunk/blender/intern/cycles/kernel/closure/bsdf.h
===================================================================
--- trunk/blender/intern/cycles/kernel/closure/bsdf.h   2013-05-21 12:18:35 UTC 
(rev 56943)
+++ trunk/blender/intern/cycles/kernel/closure/bsdf.h   2013-05-21 13:15:27 UTC 
(rev 56944)
@@ -96,14 +96,14 @@
                        label = bsdf_ashikhmin_velvet_sample(sc, sd->Ng, sd->I, 
sd->dI.dx, sd->dI.dy, randu, randv,
                                eval, omega_in, &domega_in->dx, &domega_in->dy, 
pdf);
                        break;
-               /*case CLOSURE_BSDF_WESTIN_BACKSCATTER_ID:
+               case CLOSURE_BSDF_WESTIN_BACKSCATTER_ID:
                        label = bsdf_westin_backscatter_sample(sc, sd->Ng, 
sd->I, sd->dI.dx, sd->dI.dy, randu, randv,
                                eval, omega_in, &domega_in->dx, &domega_in->dy, 
pdf);
                        break;
                case CLOSURE_BSDF_WESTIN_SHEEN_ID:
                        label = bsdf_westin_sheen_sample(sc, sd->Ng, sd->I, 
sd->dI.dx, sd->dI.dy, randu, randv,
                                eval, omega_in, &domega_in->dx, &domega_in->dy, 
pdf);
-                       break;*/
+                       break;
 #endif
                default:
                        label = LABEL_NONE;
@@ -165,12 +165,12 @@
                        case CLOSURE_BSDF_ASHIKHMIN_VELVET_ID:
                                eval = bsdf_ashikhmin_velvet_eval_reflect(sc, 
sd->I, omega_in, pdf);
                                break;
-                       /*case CLOSURE_BSDF_WESTIN_BACKSCATTER_ID:
+                       case CLOSURE_BSDF_WESTIN_BACKSCATTER_ID:
                                eval = bsdf_westin_backscatter_eval_reflect(sc, 
sd->I, omega_in, pdf);
                                break;
                        case CLOSURE_BSDF_WESTIN_SHEEN_ID:
                                eval = bsdf_westin_sheen_eval_reflect(sc, 
sd->I, omega_in, pdf);
-                               break;*/
+                               break;
 #endif
                        default:
                                eval = make_float3(0.0f, 0.0f, 0.0f);
@@ -214,12 +214,12 @@
                        case CLOSURE_BSDF_ASHIKHMIN_VELVET_ID:
                                eval = bsdf_ashikhmin_velvet_eval_transmit(sc, 
sd->I, omega_in, pdf);
                                break;
-                       /*case CLOSURE_BSDF_WESTIN_BACKSCATTER_ID:
+                       case CLOSURE_BSDF_WESTIN_BACKSCATTER_ID:
                                eval = 
bsdf_westin_backscatter_eval_transmit(sc, sd->I, omega_in, pdf);
                                break;
                        case CLOSURE_BSDF_WESTIN_SHEEN_ID:
                                eval = bsdf_westin_sheen_eval_transmit(sc, 
sd->I, omega_in, pdf);
-                               break;*/
+                               break;
 #endif
                        default:
                                eval = make_float3(0.0f, 0.0f, 0.0f);
@@ -281,12 +281,12 @@
                case CLOSURE_BSDF_ASHIKHMIN_VELVET_ID:
                        bsdf_ashikhmin_velvet_blur(sc, roughness);
                        break;
-               /*case CLOSURE_BSDF_WESTIN_BACKSCATTER_ID:
+               case CLOSURE_BSDF_WESTIN_BACKSCATTER_ID:
                        bsdf_westin_backscatter_blur(sc, roughness);
                        break;
                case CLOSURE_BSDF_WESTIN_SHEEN_ID:
                        bsdf_westin_sheen_blur(sc, roughness);
-                       break;*/
+                       break;
 #endif
                default:
                        break;

Modified: trunk/blender/intern/cycles/kernel/osl/CMakeLists.txt
===================================================================
--- trunk/blender/intern/cycles/kernel/osl/CMakeLists.txt       2013-05-21 
12:18:35 UTC (rev 56943)
+++ trunk/blender/intern/cycles/kernel/osl/CMakeLists.txt       2013-05-21 
13:15:27 UTC (rev 56944)
@@ -17,7 +17,6 @@
        bsdf_diffuse_ramp.cpp
        bsdf_phong_ramp.cpp
        bsdf_toon.cpp
-       bsdf_westin.cpp
        emissive.cpp
        osl_bssrdf.cpp
        osl_closures.cpp

Deleted: trunk/blender/intern/cycles/kernel/osl/bsdf_westin.cpp
===================================================================
--- trunk/blender/intern/cycles/kernel/osl/bsdf_westin.cpp      2013-05-21 
12:18:35 UTC (rev 56943)
+++ trunk/blender/intern/cycles/kernel/osl/bsdf_westin.cpp      2013-05-21 
13:15:27 UTC (rev 56944)
@@ -1,177 +0,0 @@
-/*
- * Adapted from Open Shading Language with this license:
- *
- * Copyright (c) 2009-2010 Sony Pictures Imageworks Inc., et al.
- * All Rights Reserved.
- *
- * Modifications Copyright 2011, Blender Foundation.
- * 
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions are
- * met:
- * * Redistributions of source code must retain the above copyright
- *   notice, this list of conditions and the following disclaimer.
- * * Redistributions in binary form must reproduce the above copyright
- *   notice, this list of conditions and the following disclaimer in the
- *   documentation and/or other materials provided with the distribution.
- * * Neither the name of Sony Pictures Imageworks nor the names of its
- *   contributors may be used to endorse or promote products derived from
- *   this software without specific prior written permission.
- * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
- * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
- * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
- * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
- * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
- * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
- * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
- * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
- * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#include <OpenImageIO/fmath.h>
-
-#include <OSL/genclosure.h>
-
-#include "osl_closures.h"
-
-#include "kernel_types.h"
-#include "kernel_montecarlo.h"
-#include "closure/bsdf_westin.h"
-
-CCL_NAMESPACE_BEGIN
-
-using namespace OSL;
-
-/* WESTIN BACKSCATTER */
-
-class WestinBackscatterClosure : public CBSDFClosure {
-public:
-       WestinBackscatterClosure() : CBSDFClosure(LABEL_GLOSSY) {}
-
-       size_t memsize() const { return sizeof(*this); }
-       const char *name() const { return "westin_backscatter"; }
-
-       void setup()
-       {
-               sc.prim = this;
-               m_shaderdata_flag = bsdf_westin_backscatter_setup(&sc);
-       }
-
-       bool mergeable(const ClosurePrimitive *other) const
-       {
-               return false;
-       }
-
-       void blur(float roughness)
-       {
-               bsdf_westin_backscatter_blur(&sc, roughness);
-       }
-
-       void print_on(std::ostream &out) const
-       {
-               out << name() << " ((" << sc.N[0] << ", " << sc.N[1] << ", " << 
sc.N[2] << "))";
-       }
-
-       float3 eval_reflect(const float3 &omega_out, const float3 &omega_in, 
float& pdf) const
-       {
-               return bsdf_westin_backscatter_eval_reflect(&sc, omega_out, 
omega_in, &pdf);
-       }
-
-       float3 eval_transmit(const float3 &omega_out, const float3 &omega_in, 
float& pdf) const
-       {
-               return bsdf_westin_backscatter_eval_transmit(&sc, omega_out, 
omega_in, &pdf);
-       }
-
-       int sample(const float3 &Ng,
-                  const float3 &omega_out, const float3 &domega_out_dx, const 
float3 &domega_out_dy,
-                  float randu, float randv,
-                  float3 &omega_in, float3 &domega_in_dx, float3 &domega_in_dy,
-                  float &pdf, float3 &eval) const
-       {
-               return bsdf_westin_backscatter_sample(&sc, Ng, omega_out, 
domega_out_dx, domega_out_dy,
-                       randu, randv, &eval, &omega_in, &domega_in_dx, 
&domega_in_dy, &pdf);
-       }
-};
-
-ClosureParam *closure_westin_backscatter_params()
-{
-       static ClosureParam params[] = {
-               CLOSURE_FLOAT3_PARAM(WestinBackscatterClosure, sc.N),
-               CLOSURE_FLOAT_PARAM(WestinBackscatterClosure, sc.data0),
-               CLOSURE_STRING_KEYPARAM("label"),
-           CLOSURE_FINISH_PARAM(WestinBackscatterClosure)
-       };
-       return params;
-}
-
-CLOSURE_PREPARE(closure_westin_backscatter_prepare, WestinBackscatterClosure)
-
-/* WESTIN SHEEN */
-
-class WestinSheenClosure : public CBSDFClosure {
-public:
-       WestinSheenClosure() : CBSDFClosure(LABEL_DIFFUSE) {}
-
-       size_t memsize() const { return sizeof(*this); }
-       const char *name() const { return "westin_sheen"; }
-
-       void setup()
-       {
-               sc.prim = this;
-               m_shaderdata_flag = bsdf_westin_sheen_setup(&sc);
-       }
-
-       bool mergeable(const ClosurePrimitive *other) const
-       {
-               return false;
-       }
-
-       void blur(float roughness)
-       {
-               bsdf_westin_sheen_blur(&sc, roughness);
-       }
-
-       void print_on(std::ostream &out) const
-       {
-               out << name() << " ((" << sc.N[0] << ", " << sc.N[1] << ", " << 
sc.N[2] << "))";
-       }
-
-       float3 eval_reflect(const float3 &omega_out, const float3 &omega_in, 
float& pdf) const
-       {
-               return bsdf_westin_sheen_eval_reflect(&sc, omega_out, omega_in, 
&pdf);
-       }
-
-       float3 eval_transmit(const float3 &omega_out, const float3 &omega_in, 
float& pdf) const
-       {
-               return bsdf_westin_sheen_eval_transmit(&sc, omega_out, 
omega_in, &pdf);
-       }
-
-       int sample(const float3 &Ng,
-                  const float3 &omega_out, const float3 &domega_out_dx, const 
float3 &domega_out_dy,
-                  float randu, float randv,
-                  float3 &omega_in, float3 &domega_in_dx, float3 &domega_in_dy,
-                  float &pdf, float3 &eval) const
-       {
-               return bsdf_westin_sheen_sample(&sc, Ng, omega_out, 
domega_out_dx, domega_out_dy,
-                       randu, randv, &eval, &omega_in, &domega_in_dx, 
&domega_in_dy, &pdf);
-       }
-};
-
-ClosureParam *closure_westin_sheen_params()
-{
-       static ClosureParam params[] = {
-               CLOSURE_FLOAT3_PARAM(WestinSheenClosure, sc.N),
-               CLOSURE_FLOAT_PARAM(WestinSheenClosure, sc.data0),
-               CLOSURE_STRING_KEYPARAM("label"),
-           CLOSURE_FINISH_PARAM(WestinSheenClosure)
-       };
-       return params;
-}
-
-CLOSURE_PREPARE(closure_westin_sheen_prepare, WestinSheenClosure)
-
-
-CCL_NAMESPACE_END
-

_______________________________________________
Bf-blender-cvs mailing list
[email protected]
http://lists.blender.org/mailman/listinfo/bf-blender-cvs

Reply via email to