Bug 61003

Summary:	gluSurface with Nurbs spits out C code "arc_ccw_turn..."
Product:	Mesa	Reporter:	Blaž Hrastnik <speed.the.bboy>
Component:	GLU	Assignee:	Matt Turner <mattst88>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	medium	CC:	sndirsch, speed.the.bboy
Version:	unspecified
Hardware:	x86-64 (AMD64)
OS:	Linux (All)
Whiteboard:
i915 platform:		i915 features:
Attachments:	patch

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 1 Blaž Hrastnik 2013-02-17 22:54:05 UTC

Apparently, this is an old issue:

http://www.opengl.org/discussion_boards/archive/index.php/t-152297.html

http://marc.info/?l=mesa3d-users&m=117832104821813

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.