Thanks, James. I've just checked the two files in.<br><br>Jane<br><br><div class="gmail_quote">On Mon, Jun 6, 2011 at 2:08 PM, James Porter <span dir="ltr"><<a href="mailto:jvporter@wisc.edu">jvporter@wisc.edu</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">So I finally figured out (part of?) what's going on with the distcheck<br>
errors. There are some missing files that never got added to this<br>
commit: Line_1.brep and Line_2.brep.<br>
<br>
There may be other errors of course, so adding those might not fix<br>
everything...<br>
<div><div></div><div class="h5"><br>
On Thu, 2011-05-26 at 13:29 -0500, <a href="mailto:jiangtao_ma@yahoo.com">jiangtao_ma@yahoo.com</a> wrote:<br>
> Author: janehu<br>
> Date: 2011-05-26 13:29:34 -0500 (Thu, 26 May 2011)<br>
> New Revision: 4884<br>
><br>
> Modified:<br>
> cgm/trunk/geom/OCC/OCCModifyEngine.cpp<br>
> cgm/trunk/geom/OCC/OCCSurface.cpp<br>
> cgm/trunk/test/Makefile.am<br>
> cgm/trunk/test/modify.cpp<br>
> Log:<br>
> Made changes regarding webcut using sweep curve method that is a possible bug that Yu Wenhui asked. Passed make distcheck. Testcase is in modify.cpp<br>
><br>
> Modified: cgm/trunk/geom/OCC/OCCModifyEngine.cpp<br>
> ===================================================================<br>
> --- cgm/trunk/geom/OCC/OCCModifyEngine.cpp 2011-05-26 13:49:07 UTC (rev 4883)<br>
> +++ cgm/trunk/geom/OCC/OCCModifyEngine.cpp 2011-05-26 18:29:34 UTC (rev 4884)<br>
> @@ -4442,19 +4442,17 @@<br>
> }<br>
> }<br>
> }<br>
> - else //sheet body<br>
> - {<br>
> - TopoDS_Face* topoface = occ_surface->get_TopoDS_Face();<br>
> - TopAbs_Orientation ori = topoface->Orientation();<br>
> - topoface->Orientation(ori == TopAbs_FORWARD ? TopAbs_REVERSED :<br>
> + }<br>
> + if(!occ_shell || occ_shell->is_sheet()) //sheet body<br>
> + {<br>
> + TopoDS_Face* topoface = occ_surface->get_TopoDS_Face();<br>
> + TopAbs_Orientation ori = topoface->Orientation();<br>
> + topoface->Orientation(ori == TopAbs_FORWARD ? TopAbs_REVERSED :<br>
> TopAbs_FORWARD);<br>
> - occ_surface->set_TopoDS_Face(*topoface);<br>
> - surface_list.append(occ_surface);<br>
> - }<br>
> + occ_surface->set_TopoDS_Face(*topoface);<br>
> + surface_list.append(occ_surface);<br>
> PRINT_INFO( "Modified volume\n" );<br>
> }<br>
> - else<br>
> - PRINT_WARNING( "Volume was not modified\n" );<br>
> }<br>
> face_list = surface_list;<br>
> return CUBIT_SUCCESS;<br>
> @@ -4639,22 +4637,12 @@<br>
> CubitStatus OCCModifyEngine::get_sweepable_toposhape(OCCCurve*& curve,<br>
> TopoDS_Shape& toposhape)const<br>
> {<br>
> - DLIList<OCCLoop*> loops;<br>
> - loops = curve->loops();<br>
> - if( loops.size()) //not a free curve<br>
> - {<br>
> - //copy the curve<br>
> - Curve* c_curve = make_Curve(curve);<br>
> - if(c_curve)<br>
> - curve = CAST_TO(c_curve, OCCCurve);<br>
> - else<br>
> - {<br>
> - PRINT_ERROR("Can't copy the curve for sweep.\n");<br>
> - return CUBIT_FAILURE;<br>
> - }<br>
> - }<br>
> TopoDS_Edge *edge = curve->get_TopoDS_Edge( );<br>
<br>
<br>
</div></div></blockquote></div><br>