Fix for wrong normals of a 2D extruded curve's fills
ClosedPublic
Actions

Authored by Tamito Kajiyama (kjym3) on May 16 2014, 7:28 AM.

Details

Reviewers

Commits

rBSf8554ed61a42: Fix T39669: Freestyle: Curve with extrude>0 causes warnings in console.
rBf8554ed61a42: Fix T39669: Freestyle: Curve with extrude>0 causes warnings in console.

Summary

This patch is intended to fix wrong normals of filled faces at both ends of a 2D extruded curve. The problem is detailed in the comment #19 of T39669.

Diff Detail

Event Timeline

This diff causes a simple extruded curve to flip in the viewport - when you enable backface culling.

curve_extrude.blend483 KBDownload

Campbell Barton (campbellbarton) requested changes to this revision.May 16 2014, 9:44 AM

Another attempt to fix the bug. I believe this patch is going to be a proper fix: there was an inconsistency in the use of vertex indices between BKE_mesh_nurbs_displist_to_mdata() and init_render_curve() in the case of DispList::type equal to DL_INDEX3.

Campbell Barton (campbellbarton) accepted this revision.May 17 2014, 1:13 AM

Closed by commit rBf8554ed61a42 (authored by @Tamito Kajiyama (kjym3)).