*/
static SFG_Font* fghFontByID( void* font )
{
- /*
- * XXX Use a macro, a table of some kind, or else split these
- * XXX statements properly. Jamming "return" on the end of an
- * XXX "if" is just bad style, IMHO.
- */
- if( font == GLUT_BITMAP_8_BY_13 ) return &fgFontFixed8x13;
- if( font == GLUT_BITMAP_9_BY_15 ) return &fgFontFixed9x15;
- if( font == GLUT_BITMAP_HELVETICA_10 ) return &fgFontHelvetica10;
- if( font == GLUT_BITMAP_HELVETICA_12 ) return &fgFontHelvetica12;
- if( font == GLUT_BITMAP_HELVETICA_18 ) return &fgFontHelvetica18;
- if( font == GLUT_BITMAP_TIMES_ROMAN_10 ) return &fgFontTimesRoman10;
- if( font == GLUT_BITMAP_TIMES_ROMAN_24 ) return &fgFontTimesRoman24;
+ if( font == GLUT_BITMAP_8_BY_13 )
+ return &fgFontFixed8x13;
+ if( font == GLUT_BITMAP_9_BY_15 )
+ return &fgFontFixed9x15;
+ if( font == GLUT_BITMAP_HELVETICA_10 )
+ return &fgFontHelvetica10;
+ if( font == GLUT_BITMAP_HELVETICA_12 )
+ return &fgFontHelvetica12;
+ if( font == GLUT_BITMAP_HELVETICA_18 )
+ return &fgFontHelvetica18;
+ if( font == GLUT_BITMAP_TIMES_ROMAN_10 )
+ return &fgFontTimesRoman10;
+ if( font == GLUT_BITMAP_TIMES_ROMAN_24 )
+ return &fgFontTimesRoman24;
+
fgError( "font 0x%08x not found", font );
return 0; /*** NOT REACHED ***/
}
*/
static SFG_StrokeFont* fghStrokeByID( void* font )
{
- /*
- * XXX Same comment as above about jamming "return" in after an
- * XXX "if".
- */
- if( font == GLUT_STROKE_ROMAN ) return &fgStrokeRoman;
- if( font == GLUT_STROKE_MONO_ROMAN ) return &fgStrokeMonoRoman;
+ if( font == GLUT_STROKE_ROMAN )
+ return &fgStrokeRoman;
+ if( font == GLUT_STROKE_MONO_ROMAN )
+ return &fgStrokeMonoRoman;
+
fgError( "stroke font 0x%08x not found", font );
return 0; /*** NOT REACHED ***/
}