AuroraMiddleware/OpenSubdiv
1
0
Fork 0
mirror of https://github.com/PixarAnimationStudios/OpenSubdiv synced 2024-12-17 22:21:07 +00:00
Code Issues Releases Wiki Activity

Fixed bug in face-varying patches of uniformly refined Far::PatchTable:

Browse Source 
- added missing initialization of face-varying PatchParams
This commit is contained in:
barry 2017-09-13 10:50:58 -07:00
parent 161556b9eb
commit d376944dd1
1 changed files with 10 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
opensubdiv/far/patchTableFactory.cpp
View File

@ -988,6 +988,7 @@ PatchTableFactory::createUniform(TopologyRefiner const & refiner, Options option
Index * iptr = &table->_patchVerts[0];
PatchParam * pptr = &table->_paramTable[0];
Index ** fptr = 0;
PatchParam ** fpptr = 0;
// we always skip level=0 vertices (control cages)
Index levelVertOffset = refiner.GetLevel(0).GetNumVertices();
@ -999,8 +1000,10 @@ PatchTableFactory::createUniform(TopologyRefiner const & refiner, Options option
memset(levelFVarVertOffsets, 0, context.fvarChannelIndices.size()*sizeof(Index));
fptr = (Index **)alloca(context.fvarChannelIndices.size()*sizeof(Index *));
fpptr = (PatchParam **)alloca(context.fvarChannelIndices.size()*sizeof(PatchParam *));
for (int fvc=0; fvc<(int)context.fvarChannelIndices.size(); ++fvc) {
fptr[fvc] = table->getFVarValues(fvc).begin();
fpptr[fvc] = table->getFVarPatchParams(fvc).begin();
}
}
@ -1021,7 +1024,8 @@ PatchTableFactory::createUniform(TopologyRefiner const & refiner, Options option
*iptr++ = levelVertOffset + fverts[vert];
}
*pptr++ = computePatchParam(context, level, face, /*boundary*/0, /*transition*/0);
PatchParam pparam = computePatchParam(context, level, face, /*boundary*/0, /*transition*/0);
*pptr++ = pparam;
if (context.RequiresFVarPatches()) {
for (int fvc=0; fvc<(int)context.fvarChannelIndices.size(); ++fvc) {
@ -1033,6 +1037,7 @@ PatchTableFactory::createUniform(TopologyRefiner const & refiner, Options option
fptr[fvc][vert] = levelFVarVertOffsets[fvc] + fvalues[vert];
}
fptr[fvc]+=fvalues.size();
*fpptr[fvc]++ = pparam;
}
}
@ -1043,8 +1048,7 @@ PatchTableFactory::createUniform(TopologyRefiner const & refiner, Options option
*iptr = *(iptr - 3); // copy v2 index
++iptr;
*pptr = *(pptr - 1); // copy first patch param
++pptr;
*pptr++ = pparam;
if (context.RequiresFVarPatches()) {
for (int fvc=0; fvc<(int)context.fvarChannelIndices.size(); ++fvc) {
@ -1052,6 +1056,8 @@ PatchTableFactory::createUniform(TopologyRefiner const & refiner, Options option
++fptr[fvc];
*fptr[fvc] = *(fptr[fvc]-3); // copy fv2 index
++fptr[fvc];
*fpptr[fvc]++ = pparam;
}
}
}