Ooops, left some cruft in a comment in the header when committing the
last patch. Cleaned up. git-svn-id: https://svn.code.sf.net/p/freeglut/code/trunk@411 7f0cb862-5218-0410-a997-914c9d46530a
This commit is contained in:
parent
a7d846c36b
commit
0687f2f596
@ -385,17 +385,9 @@ struct tagSFG_WindowState
|
|||||||
* but this can cause warnings because the FETCH_WCB() macro type-
|
* but this can cause warnings because the FETCH_WCB() macro type-
|
||||||
* casts its result, and a type-cast value shouldn't be an lvalue.
|
* casts its result, and a type-cast value shouldn't be an lvalue.
|
||||||
*
|
*
|
||||||
* XXX Note that there is no type-checking to make sure that {func} is
|
* The {if( FETCH_WCB( ... ) != func )} test is to do type-checking
|
||||||
* XXX a suitable type. We could add a safety-check of the form:
|
* and for no other reason. Since it's hidden in the macro, the
|
||||||
* XXX
|
* ugliness is felt to be rather benign.
|
||||||
* XXX if( FETCH_WCB( ... ) != func )
|
|
||||||
* XXX ...
|
|
||||||
* XXX
|
|
||||||
* XXX ...is this desired?
|
|
||||||
*
|
|
||||||
* XXX The {if( FETCH_WCB( ... ) != func )} test is to do type-checking
|
|
||||||
* XXX and for no other reason. Since it's hidden in the macro, the
|
|
||||||
* XXX ugliness is felt to be rather benign.
|
|
||||||
*/
|
*/
|
||||||
#define SET_WCB(window,cbname,func) \
|
#define SET_WCB(window,cbname,func) \
|
||||||
do \
|
do \
|
||||||
|
Reference in New Issue
Block a user