Bug 61003 - gluSurface with Nurbs spits out C code "arc_ccw_turn..."
Summary: gluSurface with Nurbs spits out C code "arc_ccw_turn..."
Status: RESOLVED FIXED
Alias: None
Product: Mesa
Classification: Unclassified
Component: GLU (show other bugs)
Version: unspecified
Hardware: x86-64 (AMD64) Linux (All)
: medium normal
Assignee: Matt Turner
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-02-17 13:10 UTC by Blaž Hrastnik
Modified: 2013-10-15 06:41 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments
patch (684 bytes, patch)
2013-02-18 17:53 UTC, Matt Turner
Details | Splinter Review

Description Blaž Hrastnik 2013-02-17 13:10:55 UTC
I've ran a simple test program (http://pastie.org/6202841) and when calling

   gluBeginSurface(theNurb);
   gluNurbsSurface(theNurb, ...);
   gluEndSurface(theNurb);

program spits something that looks like a part of the internal code to stdout:

arc_ccw_turn, p = 0
case b
arc_ccw_turn, p = 0
case d
arc_ccw_turn, p = 0
case a
arc_ccw_turn, p = 0
case c


I am using ArchLinux x64, with 9.0.2 release of mesa, gl, glu, glut. My graphics card is Radeon HD 5850 (Evergreen series).
Comment 2 Brian Paul 2013-02-18 17:38:21 UTC
Matt, when glu is configured without --enable-debug we should probably have -DNDEBUG in our CFLAGS.  Can you look into that?
Comment 3 Matt Turner 2013-02-18 17:53:17 UTC
Created attachment 75063 [details] [review]
patch

How about this patch?
Comment 4 Blaž Hrastnik 2013-02-21 05:58:01 UTC
Was this merged yet?
Comment 5 Matt Turner 2013-02-21 07:21:11 UTC
(In reply to comment #4)
> Was this merged yet?

Did you test it yet?
Comment 6 Blaž Hrastnik 2013-02-21 14:00:18 UTC
I thought you guys tested the patch when you wrote it... I'll go on and try the patch.
Comment 7 Blaž Hrastnik 2013-02-21 14:15:51 UTC
The patch does work.
Comment 8 Matt Turner 2013-02-22 04:48:15 UTC
Pushed. Thanks for testing!
Comment 9 Stefan Dirsch 2013-10-14 19:30:27 UTC
Hmm. I don't see this patch having been pushed to Mesa master.
Comment 10 Stefan Dirsch 2013-10-14 19:35:57 UTC
So has it been resolved differently meanwhile?
Comment 11 Matt Turner 2013-10-14 20:47:31 UTC
GLU lives in a separate repository since August 2012: http://cgit.freedesktop.org/mesa/glu/

The commit that fixed this bug is the latest commit: http://cgit.freedesktop.org/mesa/glu/commit/?id=069211570c32a8d5848e127975a2943e9d8fc6b9
Comment 12 Stefan Dirsch 2013-10-15 06:41:02 UTC
OMG. You're right. I've been stupid. SUSE even splitted off a glu package for that. I just forgot about it and didn't update this package since glu release 9.0.
Thanks for providing this information.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.