--- ray/src/util/ranimove.c 2003/02/25 02:47:24 3.2 +++ ray/src/util/ranimove.c 2009/12/12 19:01:00 3.13 @@ -1,5 +1,5 @@ #ifndef lint -static const char RCSid[] = "$Id: ranimove.c,v 3.2 2003/02/25 02:47:24 greg Exp $"; +static const char RCSid[] = "$Id: ranimove.c,v 3.13 2009/12/12 19:01:00 greg Exp $"; #endif /* * Radiance object animation program @@ -11,10 +11,18 @@ static const char RCSid[] = "$Id: ranimove.c,v 3.2 200 #include "copyright.h" -#include "ranimove.h" -#include +#include +#ifdef _WIN32 + #include /* struct timeval. XXX find a replacement? */ +#else + #include +#endif #include +#include +#include "paths.h" +#include "ranimove.h" + int NVARS = NV_INIT; /* total number of variables */ VARIABLE vv[] = VV_INIT; /* variable-value pairs */ @@ -54,11 +62,21 @@ int haveprio = 0; /* high-level saliency specified */ int gargc; /* global argc for printargs */ char **gargv; /* global argv for printargs */ +static void setdefaults(void); +static void setmove(struct ObjMove *om, char *ms); +static void setrendparams(char *optf, char *qval); +static void getradfile(char *rfargs); +static void animate(void); +static int countviews(void); /* XXX duplicated function */ +static char * getobjname(struct ObjMove *om); +static char * getxf(struct ObjMove *om, int n); + int -main(argc, argv) -int argc; -char *argv[]; +main( + int argc, + char *argv[] +) { int explicate = 0; char *cfname; @@ -91,7 +109,7 @@ char *argv[]; case 'f': /* frame range */ switch (sscanf(argv[++i], "%d,%d", &fbeg, &fend)) { case 2: - if ((fbeg <= 0 | fend < fbeg)) + if ((fbeg <= 0) | (fend < fbeg)) goto userr; break; case 1: @@ -135,7 +153,7 @@ char *argv[]; /* all done */ if (lorendoptf[0]) unlink(lorendoptf); - if (hirendoptf[0]) + if (hirendoptf[0] && strcmp(hirendoptf, lorendoptf)) unlink(hirendoptf); if (objtmpf[0]) unlink(objtmpf); @@ -145,12 +163,14 @@ userr: "Usage: %s [-n nprocs][-f beg,end][-t sec][-d jnd][-s][-w][-e] anim_file\n", progname); quit(1); + return 1; /* pro forma return */ } void -eputs(s) /* put string to stderr */ -register char *s; +eputs( /* put string to stderr */ + register char *s +) { static int midline = 0; @@ -169,8 +189,18 @@ register char *s; void -setdefaults() /* set default values */ +quit(ec) /* make sure exit is called */ +int ec; { + if (ray_pnprocs > 0) /* close children if any */ + ray_pclose(0); + exit(ec); +} + + +static void +setdefaults(void) /* set default values */ +{ int nviews; int decades; char buf[256]; @@ -241,14 +271,15 @@ setdefaults() /* set default values */ setrendparams(lorendoptf, vval(LOWQ)); ray_save(&lorendparams); curparams = &lorendparams; - twolevels = bcmp(&lorendparams, &hirendparams, sizeof(RAYPARAMS)); + twolevels = memcmp(&lorendparams, &hirendparams, sizeof(RAYPARAMS)); } -void -setmove(om, ms) /* assign a move object from spec. */ -struct ObjMove *om; -char *ms; +static void +setmove( /* assign a move object from spec. */ + struct ObjMove *om, + char *ms +) { char parname[128]; char *cp; @@ -293,7 +324,7 @@ char *ms; if (isflt(om->prio_file)) { om->prio = atof(om->prio_file); om->prio_file[0] = '\0'; - haveprio |= (om->prio < 0.95 | om->prio > 1.05); + haveprio |= ((om->prio < 0.95) | (om->prio > 1.05)); } else haveprio = 1; } else @@ -305,10 +336,11 @@ badspec: } -void -setrendparams(optf, qval) /* set global rendering parameters */ -char *optf; -char *qval; +static void +setrendparams( /* set global rendering parameters */ + char *optf, + char *qval +) { char *argv[1024]; char **av = argv; @@ -328,8 +360,8 @@ char *qval; if (qval != NULL && qval[0] == '-') ac += wordstring(av+ac, qval); - /* start with default parameters */ - ray_defaults(NULL); + /* restore default parameters */ + ray_restore(NULL); /* set what we have */ for (i = 0; i < ac; i++) { while ((rval = expandarg(&ac, &av, i)) > 0) @@ -343,25 +375,25 @@ char *qval; continue; } rval = getrenderopt(ac-i, av+i); - if (rval >= 0) { - i += rval; - continue; + if (rval < 0) { + sprintf(errmsg, "bad render option at '%s'", av[i]); + error(USER, errmsg); } - sprintf(errmsg, "bad render option at '%s'", av[i]); - error(USER, errmsg); + i += rval; } } -void -getradfile(rfargs) /* run rad and get needed variables */ -char *rfargs; +static void +getradfile( /* run rad and get needed variables */ + char *rfargs +) { static short mvar[] = {OCONV,OCTREEF,RESOLUTION,EXPOSURE,-1}; char combuf[256]; register int i; register char *cp; - char *pippt; + char *pippt = NULL; /* create rad command */ strcpy(lorendoptf, "ranim0.opt"); sprintf(combuf, @@ -382,7 +414,7 @@ char *rfargs; pippt = NULL; } if (pippt != NULL) - strcpy(pippt, "> /dev/null"); /* nothing to match */ + strcpy(pippt, "> " NULL_DEVICE); /* nothing to match */ else { strcpy(cp, ")[ \t]*=' > ranimove.var"); cp += 11; /* point to file name */ @@ -404,8 +436,8 @@ char *rfargs; } -void -animate() /* run through animation */ +static void +animate(void) /* run through animation */ { int rpass; @@ -436,9 +468,10 @@ animate() /* run through animation */ } -VIEW * -getview(n) /* get view number n */ -int n; +extern VIEW * +getview( /* get view number n */ + int n +) { static FILE *viewfp = NULL; /* view file pointer */ static int viewnum = 0; /* current view number */ @@ -450,7 +483,7 @@ int n; fclose(viewfp); viewfp = NULL; viewnum = 0; - copystruct(&curview, &stdview); + curview = stdview; } return(NULL); } @@ -466,7 +499,7 @@ int n; perror(vval(VIEWFILE)); quit(1); } - copystruct(&curview, &stdview); + curview = stdview; viewnum = 0; } if (n < 0) { /* get next view */ @@ -486,8 +519,8 @@ int n; } -int -countviews() /* count views in view file */ +static int +countviews(void) /* count views in view file */ { int n; @@ -499,9 +532,10 @@ countviews() /* count views in view file */ } -char * -getexp(n) /* get exposure for nth frame */ -int n; +extern char * +getexp( /* get exposure for nth frame */ + int n +) { extern char *fskip(); static char expval[32]; @@ -553,12 +587,14 @@ formerr: sprintf(errmsg, "%s: exposure format error on line %d", vval(EXPOSURE), curfrm); error(USER, errmsg); + return NULL; /* pro forma return */ } -double -expspec_val(s) /* get exposure value from spec. */ -char *s; +extern double +expspec_val( /* get exposure value from spec. */ + char *s +) { double expval; @@ -566,15 +602,16 @@ char *s; return(1.0); expval = atof(s); - if ((s[0] == '+' | s[0] == '-')) + if ((s[0] == '+') | (s[0] == '-')) return(pow(2.0, expval)); return(expval); } -char * -getoctspec(n) /* get octree for the given frame */ -int n; +extern char * +getoctspec( /* get octree for the given frame */ + int n +) { static char combuf[1024]; int cfm = 0; @@ -626,9 +663,10 @@ int n; } -char * -getobjname(om) /* get fully qualified object name */ -register struct ObjMove *om; +static char * +getobjname( /* get fully qualified object name */ + register struct ObjMove *om +) { static char objName[512]; register char *cp = objName; @@ -644,10 +682,11 @@ register struct ObjMove *om; } -char * -getxf(om, n) /* get total transform for object */ -register struct ObjMove *om; -int n; +static char * +getxf( /* get total transform for object */ + register struct ObjMove *om, + int n +) { static char xfsbuf[4096]; char *xfp; @@ -755,19 +794,21 @@ int n; om->cprio = om->prio; } /* XXX bxfm relies on call order */ - if (framestep) + if (framestep) { if (invmat4(om->bxfm, om->cxfm)) multmat4(om->bxfm, om->bxfm, oxf.xfm); else setident4(om->bxfm); + } /* all done */ return(xfp); } -int -getmove(obj) /* find matching move object */ -OBJECT obj; +extern int +getmove( /* find matching move object */ + OBJECT obj +) { static int lasti; static OBJECT lasto = OVOID; @@ -785,7 +826,7 @@ OBJECT obj; objnm = obj_move[i].name; len = strlen(objnm); if (!strncmp(onm, objnm, len)) { - if ((obj_move[i].parent < 0 & onm[len] == '.')) + if ((obj_move[i].parent < 0) & (onm[len] == '.')) break; objnm = getobjname(&obj_move[i]) + len; len2 = strlen(objnm); @@ -798,9 +839,10 @@ OBJECT obj; } -double -obj_prio(obj) /* return priority for object */ -OBJECT obj; +extern double +obj_prio( /* return priority for object */ + OBJECT obj +) { int moi; @@ -810,8 +852,8 @@ OBJECT obj; } -double -getTime() /* get current time (CPU or real) */ +extern double +getTime(void) /* get current time (CPU or real) */ { struct timeval time_now; /* return CPU time if one process */