Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(786)

Unified Diff: src/liboslexec/opcolor.cpp

Issue 206045: Pre-review: run state overhaul (Closed) Base URL: http://openshadinglanguage.googlecode.com/svn/trunk/
Patch Set: Update patch set to sync with trunk Created 14 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: src/liboslexec/opcolor.cpp
===================================================================
--- src/liboslexec/opcolor.cpp (revision 583)
+++ src/liboslexec/opcolor.cpp (working copy)
@@ -187,14 +187,14 @@
} else if (! vary) {
// Result is varying, but everything else is uniform
Color3 r = to_rgb (*space, *x, *y, *z, exec);
- for (int i = beginpoint; i < endpoint; ++i)
- if (runflags[i])
- result[i] = r;
+ SHADE_LOOP_BEGIN
+ result[i] = r;
+ SHADE_LOOP_END
} else {
// Fully varying case
- for (int i = beginpoint; i < endpoint; ++i)
- if (runflags[i])
- result[i] = to_rgb (space[i], x[i], y[i], z[i], exec);
+ SHADE_LOOP_BEGIN
+ result[i] = to_rgb (space[i], x[i], y[i], z[i], exec);
+ SHADE_LOOP_END
}
}
@@ -225,7 +225,7 @@
impl = color_ctr_transform; // special case: colors different
else
impl = triple_ctr;
- impl (exec, nargs, args, runflags, beginpoint, endpoint);
+ impl (exec, nargs, args);
// Use the specialized one for next time! Never have to check the
// types or do the other sanity checks again.
// FIXME -- is this thread-safe?
@@ -246,7 +246,7 @@
DASSERT (! Result.typespec().is_closure() && ! C.typespec().is_closure());
DASSERT (Result.typespec().is_float() && C.typespec().is_triple());
- unary_op_guts<Float, Color3, Luminance> (Result, C, exec, runflags, beginpoint, endpoint);
+ unary_op_guts<Float, Color3, Luminance> (Result, C, exec);
}

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b