Hello!

Appended are a few very minor patches:

gl/rasterization.lisp:
- in gl:tex-parameter's :texture-border-color use "elt" rather than
"aref", to be consistent with other calls in the file
- in gl:fog allow :fog-color -- I hope this is done in a manner
consistent with cl-opengl style?

glu/glu.lisp:
- change names of some gluPerspective parameters to match their standard names

Thanks for the very useful cl-opengl code!  I hope these are worth applying!

Best,

Dan
diff -rN -u old-cl-opengl/gl/rasterization.lisp new-cl-opengl/gl/rasterization
.lisp                                                                        
--- old-cl-opengl/gl/rasterization.lisp 2008-06-13 15:05:20.000000000 -0400
+++ new-cl-opengl/gl/rasterization.lisp 2008-06-13 15:05:20.000000000 -0400
@@ -265,7 +265,7 @@
     (:texture-border-color
      (with-foreign-object (array '%gl:float 4)
        (dotimes (i 4)
-         (setf (mem-aref array '%gl:float i) (float (aref param i))))
+         (setf (mem-aref array '%gl:float i) (float (elt param i))))
        (%gl:tex-parameter-fv target pname array)))
     ((:texture-priority :texture-min-lod :texture-max-lod)
      (%gl:tex-parameter-f target pname (float param)))
@@ -376,6 +376,11 @@
 
 (defun fog (pname param)
   (ecase pname
+    (:fog-color
+     (with-foreign-object (c '%gl:float 4)
+       (dotimes (i 4)
+         (setf (mem-aref c '%gl:float i) (float (elt param i))))
+       (%gl:fog-fv pname c)))
     (:fog-mode
      (%gl:fog-i pname (foreign-enum-value '%gl:enum param)))
     (:fog-coord-src
diff -rN -u old-cl-opengl/glu/glu.lisp new-cl-opengl/glu/glu.lisp
--- old-cl-opengl/glu/glu.lisp  2008-06-13 15:05:20.000000000 -0400
+++ new-cl-opengl/glu/glu.lisp  2008-06-13 15:05:20.000000000 -0400
@@ -123,9 +123,9 @@
 
 (defcfun ("gluPerspective" perspective) :void
   (fov-y  %gl:double)
-  (right  %gl:double)
-  (bottom %gl:double)
-  (top    %gl:double))
+  (aspect %gl:double)
+  (zNear  %gl:double)
+  (zFar   %gl:double))
 
 (defcfun ("gluLookAt" look-at) :void
   (eye-x %gl:double)

_______________________________________________
cl-opengl-devel mailing list
cl-opengl-devel@common-lisp.net
http://common-lisp.net/cgi-bin/mailman/listinfo/cl-opengl-devel

Reply via email to