I just compiled the latest sources and found a few things to mention.

1. When developing/debugging be sure to define the DEBUG symbol (with
   -DDEBUG, for example).  This detected an uninitialized entry in the
   API function table.

2. Compile with maximum warnings (gcc -ansi -pedantic -Wall).
   In src/FX/fxsetup.c I found a switch statement without a default
   case.  I added a default no-op case.  David, is that right or
   should an error be signaled?

3. Instead of printf(), use gl_problem() to print serious/fatal error
   conditions.  Call gl_warning() to report questionable runtime cond-
   tions to the user (like enabling GL_DEPTH_DEST without a depth
   buffer present).

Thanks.

-Brian
   
----------------------------------------------------------------------
Brian Paul        Avid Technology / Softimage      [EMAIL PROTECTED]

Reply via email to