|
![](/i/fill.gif) |
Sorry for this being so long in advance...
I'm working on porting over MegaPOV to OpenVMS to go along with my
POVRay v3.1g for OpenVMS and have run into a few things that I wanted to
get some input on (not knowing all the POV code and not all that great
of a programmer myself)
I'll start with the SMALL question/problem for now. Most of the
problems aren't really problems, just compiler warnings that I would
like to make go away if possible.
(I'm using the POVRay v3.1g Unix souce code with the MegaPov 0.4 patch
file.)
In "TOKENIZE.C" there is the following section of code...
/*****************************************************************************
* Not-quite-so-Static functions
******************************************************************************/
/** poviso: July 15 '96 R.S. removed 'static' for f_expr.c **/
#ifdef POVISO
#ifdef FastMacroPatch
void Echo_ungetc (int c);
#else
int Echo_ungetc (int c);
#endif
int Echo_getc (void);
int Skip_Spaces (void);
int Parse_C_Comments (void);
void Begin_String (void);
void Stuff_Character (int c);
void End_String (void);
int Read_Float (void);
void Parse_String_Literal (void);
void Read_Symbol (void);
SYM_ENTRY *Find_Symbol (int Index, char *s);
void Skip_Tokens (COND_TYPE cond);
int get_hash_value (char *s);
void init_reserved_words_hash_table (void);
void Write_Token (TOKEN Token_Id);
void Destroy_Table (int index);
void init_sym_tables (void);
void Add_Sym_Table (char *s);
void Remove_Symbol (int Index, char *Name);
POV_MACRO *Parse_Macro(void);
void Invoke_Macro(void);
void Return_From_Macro(void);
void Add_Entry (int Index,SYM_ENTRY *Table_Entry);
void Parse_Initalizer (int Sub, int Base, POV_ARRAY *a);
void Parse_Fopen(void);
void Parse_Fclose(void);
void Parse_Read(void);
void Parse_Write(void);
static int Parse_Comma_RParen(void);
int Parse_Read_Value(DATA_FILE *User_File,int Previous,int
*NumberPtr,void **Da
void Check_Macro_Vers(void);
DBL Parse_Cond_Param(void);
void Parse_Cond_Param2(DBL *V1,DBL *V2);
void Inc_CS_Index(void);
#else
#ifdef FastMacroPatch
static void Echo_ungetc (int c);
#else
static int Echo_ungetc (int c);
#endif
static int Echo_getc (void);
static int Skip_Spaces (void);
static int Parse_C_Comments (void);
static void Begin_String (void);
static void Stuff_Character (int c);
static void End_String (void);
static int Read_Float (void);
static void Parse_String_Literal (void);
static void Read_Symbol (void);
static SYM_ENTRY *Find_Symbol (int Index, char *s);
static void Skip_Tokens (COND_TYPE cond);
static int get_hash_value (char *s);
static void Write_Token (TOKEN Token_Id);
static void Destroy_Table (int index);
static void init_sym_tables (void);
static void Add_Sym_Table (char *s);
static void Remove_Symbol (int Index, char *Name);
static POV_MACRO *Parse_Macro(void);
static void Invoke_Macro(void);
static void Return_From_Macro(void);
static void Add_Entry (int Index,SYM_ENTRY *Table_Entry);
static void Parse_Initalizer (int Sub, int Base, POV_ARRAY *a);
static void Parse_Fopen(void);
static void Parse_Fclose(void);
static void Parse_Read(void);
static void Parse_Write(void);
static int Parse_Comma_RParen(void);static int
Parse_Read_Value(DATA_FILE *User_File,int Previous,int *NumberPtr,vo
static void Check_Macro_Vers(void);
static DBL Parse_Cond_Param(void);
static void Parse_Cond_Param2(DBL *V1,DBL *V2);
static void Inc_CS_Index(void);
#endif
Now, if I'm reading this right, depending on if POVISO is defined the
following...
Skip_Spaces Parse_C_Comments
Begin_String Stuff_Character
End_String Read_Float
Parse_String_Literal Read_Symbol
Echo_getc Skip_Tokens
get_hash_value init_sym_tables
Add_Sym_Table Destroy_Table
Add_Entry Find_Symbol
Remove_Symbol Check_Macro_Vers
Parse_Macro Invoke Macro
Return_From_Macro Parse_Initalizer
Parse_Fopen Parse_Fclose
Parse_Read Parse_Read_Value
Parse_Write Parse_Cond_Param
Parse_Cond_Param2 Inc_CS_Index
can either be "static" or not.
Now, the problem is that when it comes to the actual functions, they are
still defined as "static".
So the function prototype can be either "static" or not but the function
it's self is "static"?
(That is, if I'm reading all this correct).
Now, my compiler will still compile it, but with warnings.
So I'm wondering if the functions should also have a test for POVISO so
the function can be defined as "static" or not?
Example...
/*****************************************************************************
*
* FUNCTION
*
* INPUT
*
* OUTPUT
*
* RETURNS
*
* AUTHOR
*
* DESCRIPTION
*
* Skip over spaces in the input file.
*
* CHANGES
*
******************************************************************************/
#if defined(POVISO)
int Skip_Spaces ()
#else
static int Skip_Spaces ()
#endif
And if I'm write, I know other parts of the code that could use this
cleaning up.
So I'm open to suggestions and clarifications on this matter as I would
like to finish my port (and get on to the next problem).
Thanks.
(Sorry for the long post.)
--
+--------------------------+----------------------------------------+
| Robert Alan Byer | "I don't want to take over the world, |
| bye### [at] mail ourservers net | just my own little part of it." |
| Phone: (317)357-2724 | http://www.ourservers.net/~byer |
+--------------------------+----------------------------------------+
| Send an E-mail request to obtain my PGP key. ICQ #65926579 |
+-------------------------------------------------------------------+
Post a reply to this message
|
![](/i/fill.gif) |