[Bf-blender-cvs] [7050257] master: Cycles: Remove TODO, it is possible there'll be more intersections recorded

Sergey Sharybin noreply at git.blender.org
Tue Dec 1 23:16:23 CET 2015


Commit: 70502578b1d2849a3f8477dd574ffd98328c3a50
Author: Sergey Sharybin
Date:   Wed Dec 2 02:39:57 2015 +0500
Branches: master
https://developer.blender.org/rB70502578b1d2849a3f8477dd574ffd98328c3a50

Cycles: Remove TODO, it is possible there'll be more intersections recorded

It's just only few of them will be stored in the intersection array, nothing
wrong with that what's so ever.

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

M	intern/cycles/kernel/kernel_subsurface.h

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

diff --git a/intern/cycles/kernel/kernel_subsurface.h b/intern/cycles/kernel/kernel_subsurface.h
index b992856..e2d195a 100644
--- a/intern/cycles/kernel/kernel_subsurface.h
+++ b/intern/cycles/kernel/kernel_subsurface.h
@@ -288,9 +288,6 @@ ccl_device int subsurface_scatter_multi_intersect(
 	                           sd->object,
 	                           lcg_state,
 	                           BSSRDF_MAX_HITS);
-	/* TODO(sergey): Investigate whether scene_intersect_subsurface() could
-	 * indeed return more than BSSRDF_MAX_HITS hits.
-	 */
 	int num_eval_hits = min(ss_isect->num_hits, BSSRDF_MAX_HITS);
 
 	for(int hit = 0; hit < num_eval_hits; hit++) {




More information about the Bf-blender-cvs mailing list