ViewVC Help
View File | Revision Log | Show Annotations | Download File | Root Listing
root/radiance/ray/src/rt/rpict.c
(Generate patch)

Comparing ray/src/rt/rpict.c (file contents):
Revision 2.61 by schorsch, Mon Jul 21 22:30:19 2003 UTC vs.
Revision 2.101 by greg, Wed Jul 3 16:46:43 2024 UTC

# Line 1 | Line 1
1   #ifndef lint
2 < static const char RCSid[] = "$Id";
2 > static const char RCSid[] = "$Id$";
3   #endif
4   /*
5   *  rpict.c - routines and variables for picture generation.
# Line 7 | Line 7 | static const char RCSid[] = "$Id";
7  
8   #include "copyright.h"
9  
10 – #include  "platform.h"
11 – #include  "ray.h"
12 –
10   #include  <sys/types.h>
11  
12 < #ifndef NON_POSIX
13 < #ifdef BSD
14 < #include  <sys/time.h>
15 < #include  <sys/resource.h>
12 > #include  "platform.h"
13 > #ifdef NON_POSIX
14 > #ifdef MINGW
15 >  #include  <sys/time.h>
16 > #endif
17   #else
18 < #include  <sys/times.h>
19 < #include  <unistd.h>
18 > #ifdef BSD
19 >  #include  <sys/time.h>
20 >  #include  <sys/resource.h>
21 > #else
22 >  #include  <sys/times.h>
23 >  #include  <unistd.h>
24 > #endif
25   #endif
23 – #endif
26  
27   #include  <time.h>
28   #include  <signal.h>
29  
30 + #include  "ray.h"
31 + #include  "paths.h"
32 + #include  "ambient.h"
33   #include  "view.h"
34   #include  "random.h"
35   #include  "paths.h"
36 + #include  "hilbert.h"
37 + #include  "pmapbias.h"
38 + #include  "pmapdiag.h"
39  
32 –
40   #define  RFTEMPLATE     "rfXXXXXX"
41  
42   #ifndef SIGCONT
# Line 41 | Line 48 | static const char RCSid[] = "$Id";
48   CUBE  thescene;                         /* our scene */
49   OBJECT  nsceneobjs;                     /* number of objects in our scene */
50  
51 + extern RGBPRIMP  out_prims;             /* output color primitives (NULL if spectral) */
52 +
53   int  dimlist[MAXDIM];                   /* sampling dimensions */
54   int  ndims = 0;                         /* number of sampling dimensions */
55   int  samplendx;                         /* sample index number */
56  
48 – extern void  ambnotify();
57   void  (*addobjnotify[])() = {ambnotify, NULL};
58  
59   VIEW  ourview = STDVIEW;                /* view parameters */
# Line 59 | Line 67 | double dstrpix = 0.67;                 /* square pixel distribution
67  
68   double  mblur = 0.;                     /* motion blur parameter */
69  
70 + double  dblur = 0.;                     /* depth-of-field blur parameter */
71 +
72   void  (*trace)() = NULL;                /* trace call */
73  
74   int  do_irrad = 0;                      /* compute irradiance? */
75  
76 + int  rand_samp = 0;                     /* pure Monte Carlo sampling? */
77 +
78   double  dstrsrc = 0.0;                  /* square source distribution */
79   double  shadthresh = .05;               /* shadow threshold */
80   double  shadcert = .5;                  /* shadow certainty */
# Line 81 | Line 93 | double specjitter = 1.;                /* specular sampling jitter *
93  
94   int  backvis = 1;                       /* back face visibility */
95  
96 < int  maxdepth = 6;                      /* maximum recursion depth */
97 < double  minweight = 5e-3;               /* minimum ray weight */
96 > int  maxdepth = 7;                      /* maximum recursion depth */
97 > double  minweight = 1e-4;               /* minimum ray weight */
98  
99   char  *ambfile = NULL;                  /* ambient file name */
100   COLOR  ambval = BLKCOLOR;               /* ambient value */
101   int  ambvwt = 0;                        /* initial weight for ambient value */
102   double  ambacc = 0.2;                   /* ambient accuracy */
103 < int  ambres = 32;                       /* ambient resolution */
104 < int  ambdiv = 128;                      /* ambient divisions */
105 < int  ambssamp = 0;                      /* ambient super-samples */
103 > int  ambres = 64;                       /* ambient resolution */
104 > int  ambdiv = 512;                      /* ambient divisions */
105 > int  ambssamp = 128;                    /* ambient super-samples */
106   int  ambounce = 0;                      /* ambient bounces */
107 < char  *amblist[128];                    /* ambient include/exclude list */
107 > char  *amblist[AMBLLEN];                /* ambient include/exclude list */
108   int  ambincl = -1;                      /* include == 1, exclude == 0 */
109  
110   int  ralrm = 0;                         /* seconds between reports */
# Line 109 | Line 121 | int  hres, vres;                       /* resolution for this frame */
121  
122   static VIEW     lastview;               /* the previous view input */
123  
124 < extern char  *mktemp();  /* XXX should be in stdlib.h or unistd.h */
124 > static void report(int);
125 > static int nextview(FILE *fp);
126 > static void render(char *zfile, char *oldfile);
127 > static void fillscanline(COLOR *scanline, float *zline, char *sd, int xres,
128 >                int y, int xstep);
129 > static void fillscanbar(COLOR *scanbar[], float *zbar[], int xres,
130 >                int y, int ysize);
131 > static int fillsample(COLOR *colline, float *zline, int x, int y,
132 >                int xlen, int ylen, int b);
133 > static double pixvalue(COLOR  col, int  x, int  y);
134 > static int salvage(char  *oldfile);
135 > static int pixnumber(int  x, int  y, int  xres, int  yres);
136  
114 – void  report();
137  
116 – double  pixvalue();
138  
118 – #ifdef RHAS_STAT
119 – #include  <sys/types.h>
120 – #include  <sys/stat.h>
121 – int
122 – file_exists(fname)                              /* ordinary file exists? */
123 – char  *fname;
124 – {
125 –        struct stat  sbuf;
126 –        if (stat(fname, &sbuf) < 0) return(0);
127 –        return((sbuf.st_mode & S_IFREG) != 0);
128 – }
129 – #else
130 – #define  file_exists(f) (access(f,F_OK)==0)
131 – #endif
132 –
133 –
139   void
140 < quit(code)                      /* quit program */
136 < int  code;
140 > quit(int code)                  /* quit program */
141   {
142          if (code)                       /* report status */
143 <                report();
143 >                report(0);
144   #ifndef NON_POSIX
145          headclean();                    /* delete header file */
146          pfclean();                      /* clean up persist files */
# Line 146 | Line 150 | int  code;
150  
151  
152   #ifndef NON_POSIX
153 < void
154 < report()                /* report progress */
153 > static void
154 > report(int dummy)               /* report progress */
155   {
156 <        extern char  *myhostname();
157 <        double  u, s;
156 >        char                    bcStat [128];
157 >        double          u, s;
158   #ifdef BSD
159 <        struct rusage  rubuf;
159 >        struct rusage   rubuf;
160   #else
161 <        struct tms  tbuf;
162 <        double  period;
161 >        double          period = 1.0 / 60.0;
162 >        struct tms      tbuf;
163   #endif
164  
165          tlastrept = time((time_t *)NULL);
166   #ifdef BSD
167          getrusage(RUSAGE_SELF, &rubuf);
168 <        u = rubuf.ru_utime.tv_sec + rubuf.ru_utime.tv_usec/1e6;
169 <        s = rubuf.ru_stime.tv_sec + rubuf.ru_stime.tv_usec/1e6;
168 >        u = rubuf.ru_utime.tv_sec + rubuf.ru_utime.tv_usec*1e-6;
169 >        s = rubuf.ru_stime.tv_sec + rubuf.ru_stime.tv_usec*1e-6;
170          getrusage(RUSAGE_CHILDREN, &rubuf);
171 <        u += rubuf.ru_utime.tv_sec + rubuf.ru_utime.tv_usec/1e6;
172 <        s += rubuf.ru_stime.tv_sec + rubuf.ru_stime.tv_usec/1e6;
171 >        u += rubuf.ru_utime.tv_sec + rubuf.ru_utime.tv_usec*1e-6;
172 >        s += rubuf.ru_stime.tv_sec + rubuf.ru_stime.tv_usec*1e-6;
173   #else
174          times(&tbuf);
175   #ifdef _SC_CLK_TCK
176          period = 1.0 / sysconf(_SC_CLK_TCK);
173 – #else
174 –        period = 1.0 / 60.0;
177   #endif
178          u = ( tbuf.tms_utime + tbuf.tms_cutime ) * period;
179          s = ( tbuf.tms_stime + tbuf.tms_cstime ) * period;
180   #endif
181  
182 +        /* PMAP: Get photon map bias compensation statistics */
183 +        pmapBiasCompReport(bcStat);
184 +        
185          sprintf(errmsg,
186 <                "%lu rays, %4.2f%% after %.3fu %.3fs %.3fr hours on %s\n",
187 <                        nrays, pctdone, u/3600., s/3600.,
188 <                        (tlastrept-tstart)/3600., myhostname());
186 >                        "%lu rays, %s%4.2f%% after %.3fu %.3fs %.3fr hours on %s (PID %d)\n",
187 >                        nrays, bcStat, pctdone, u*(1./3600.), s*(1./3600.),
188 >                        (tlastrept-tstart)*(1./3600.), myhostname(), getpid());
189          eputs(errmsg);
190   #ifdef SIGCONT
191          signal(SIGCONT, report);
192   #endif
193   }
194   #else
195 < void
196 < report()                /* report progress */
195 > static void
196 > report(int dummy)               /* report progress */
197   {
198 +        char    bcStat [128];
199 +        
200          tlastrept = time((time_t *)NULL);
201 <        sprintf(errmsg, "%lu rays, %4.2f%% after %5.4f hours\n",
202 <                        nrays, pctdone, (tlastrept-tstart)/3600.0);
201 >
202 >        /* PMAP: Get photon map bias compensation statistics */
203 >        pmapBiasCompReport(bcStat);
204 >        
205 >        sprintf(errmsg, "%lu rays, %s%4.2f%% after %5.4f hours\n",
206 >                        nrays, bcStat, pctdone, (tlastrept-tstart)/3600.0);
207          eputs(errmsg);
208   }
209   #endif
210  
211  
212   void
213 < rpict(seq, pout, zout, prvr)                    /* generate image(s) */
214 < int  seq;
215 < char  *pout, *zout, *prvr;
213 > rpict(                  /* generate image(s) */
214 >        int  seq,
215 >        char  *pout,
216 >        char  *zout,
217 >        char  *prvr
218 > )
219   /*
220   * If seq is greater than zero, then we will render a sequence of
221   * images based on view parameter strings read from the standard input.
# Line 216 | Line 230 | char  *pout, *zout, *prvr;
230   {
231          char  fbuf[128], fbuf2[128];
232          int  npicts;
233 <        register char  *cp;
233 >        char  *cp;
234          RESOLU  rs;
235          double  pa;
236                                          /* check sampling */
# Line 232 | Line 246 | char  *pout, *zout, *prvr;
246          if (seq <= 0)
247                  seq = 0;
248          else if (prvr != NULL && isint(prvr)) {
249 <                register int  rn;               /* skip to specified view */
249 >                int  rn;                /* skip to specified view */
250                  if ((rn = atoi(prvr)) < seq)
251                          error(USER, "recover frame less than start frame");
252                  if (pout == NULL)
# Line 243 | Line 257 | char  *pout, *zout, *prvr;
257                  setview(&ourview);
258                  prvr = fbuf;                    /* mark for renaming */
259          }
260 <        if (pout != NULL & prvr != NULL) {
260 >        if ((pout != NULL) & (prvr != NULL)) {
261                  sprintf(fbuf, pout, seq);
262                  if (!strcmp(prvr, fbuf)) {      /* rename */
263                          strcpy(fbuf2, fbuf);
# Line 275 | Line 289 | char  *pout, *zout, *prvr;
289                          break;
290                  pctdone = 0.0;
291                  if (pout != NULL) {
292 +                        int     myfd;
293 +                        close(1);                       /* reassign stdout */
294                          sprintf(fbuf, pout, seq);
295 <                        if (file_exists(fbuf)) {
296 <                                if (prvr != NULL || !strcmp(fbuf, pout)) {
295 >                tryagain:
296 >                        errno = 0;                      /* exclusive open */
297 >                        if ((myfd = open(fbuf, O_WRONLY|O_CREAT|O_EXCL, 0666)) < 0) {
298 >                                if ((errno != EEXIST) | (prvr != NULL) ||
299 >                                                !strcmp(fbuf, pout)) {
300                                          sprintf(errmsg,
301 <                                                "output file \"%s\" exists",
301 >                                                "cannot open output file \"%s\"",
302                                                  fbuf);
303 <                                        error(USER, errmsg);
303 >                                        error(SYSTEM, errmsg);
304                                  }
305                                  setview(&ourview);
306                                  continue;               /* don't clobber */
307                          }
308 <                        if (freopen(fbuf, "w", stdout) == NULL) {
309 <                                sprintf(errmsg,
310 <                                        "cannot open output file \"%s\"", fbuf);
292 <                                error(SYSTEM, errmsg);
308 >                        if (myfd != 1) {
309 >                                unlink(fbuf);
310 >                                goto tryagain;          /* leave it open */
311                          }
312 <                        SET_FILE_BINARY(stdout);
312 >                        SET_FD_BINARY(1);
313                          dupheader();
314                  }
315                  hres = hresolu; vres = vresolu; pa = pixaspect;
316                  if (prvr != NULL) {
317 <                        if (viewfile(prvr, &ourview, &rs) <= 0
300 <                                        || rs.rt != PIXSTANDARD) {
317 >                        if (viewfile(prvr, &ourview, &rs) <= 0) {
318                                  sprintf(errmsg,
319                          "cannot recover view parameters from \"%s\"", prvr);
320                                  error(WARNING, errmsg);
# Line 322 | Line 339 | char  *pout, *zout, *prvr;
339                  fputs(VIEWSTR, stdout);
340                  fprintview(&ourview, stdout);
341                  putchar('\n');
342 <                if (pa < .99 || pa > 1.01)
342 >                if ((pa < .99) | (pa > 1.01))
343                          fputaspect(pa, stdout);
344                  fputnow(stdout);
345 <                fputformat(COLRFMT, stdout);
346 <                putchar('\n');
345 >                if (out_prims == xyzprims) {
346 >                        fputformat(CIEFMT, stdout);
347 >                } else {
348 >                        fputprims(out_prims, stdout);
349 >                        fputformat(COLRFMT, stdout);
350 >                }
351 >                putchar('\n');          /* close header */
352                  if (zout != NULL)
353                          sprintf(cp=fbuf, zout, seq);
354                  else
# Line 341 | Line 363 | char  *pout, *zout, *prvr;
363   }
364  
365  
366 < nextview(fp)                            /* get next view from fp */
367 < FILE  *fp;
366 > static int
367 > nextview(                               /* get next view from fp */
368 >        FILE  *fp
369 > )
370   {
371          char  linebuf[256];
372  
# Line 354 | Line 378 | FILE  *fp;
378   }      
379  
380  
381 < render(zfile, oldfile)                          /* render the scene */
382 < char  *zfile, *oldfile;
381 > static void
382 > render(                         /* render the scene */
383 >        char  *zfile,
384 >        char  *oldfile
385 > )
386   {
387 +        const int  srcdrawing =         /* manually draw tiny light sources? */
388 +                (directvis && dblur <= FTINY && (mblur <= FTINY) | !lastview.type);
389          COLOR  *scanbar[MAXDIV+1];      /* scanline arrays of pixel values */
390          float  *zbar[MAXDIV+1];         /* z values */
391          char  *sampdens;                /* previous sample density */
# Line 366 | Line 395 | char  *zfile, *oldfile;
395          int  zfd;
396          COLOR  *colptr;
397          float  *zptr;
398 <        register int  i;
398 >        int  i;
399                                          /* check for empty image */
400 <        if (hres <= 0 || vres <= 0) {
400 >        if ((hres <= 0) | (vres <= 0)) {
401                  error(WARNING, "empty output picture");
402                  fprtresolu(0, 0, stdout);
403                  return;
# Line 412 | Line 441 | char  *zfile, *oldfile;
441          i = salvage(oldfile);
442          if (i >= vres)
443                  goto alldone;
444 <        if (zfd != -1 && i > 0 &&
445 <                        lseek(zfd, (off_t)i*hres*sizeof(float), 0) < 0)
444 >        if ((zfd != -1) & (i > 0) &&
445 >                        lseek(zfd, (off_t)i*hres*sizeof(float), SEEK_SET) < 0)
446                  error(SYSTEM, "z-file seek error in render");
447          pctdone = 100.0*i/vres;
448          if (ralrm > 0)                  /* report init stats */
449 <                report();
449 >                report(0);
450   #ifdef SIGCONT
451          else
452          signal(SIGCONT, report);
453   #endif
454          ypos = vres-1 - i;                      /* initialize sampling */
455 <        if (directvis)
455 >        if (srcdrawing)
456                  init_drawsources(psample);
457          fillscanline(scanbar[0], zbar[0], sampdens, hres, ypos, hstep);
458                                                  /* compute scanlines */
# Line 444 | Line 473 | char  *zfile, *oldfile;
473                                  hres, ypos, hstep);
474                                                          /* fill bar */
475                  fillscanbar(scanbar, zbar, hres, ypos, ystep);
476 <                if (directvis)                          /* add bitty sources */
477 <                        drawsources(scanbar, zbar, 0, hres, ypos, ystep);
476 >                if (srcdrawing)                         /* add bitty sources */
477 >                        drawsources((COLORV **)scanbar, zbar, 0, hres, ypos, ystep);
478                                                          /* write it out */
479   #ifdef SIGCONT
480                  signal(SIGCONT, SIG_IGN);       /* don't interrupt writes */
# Line 463 | Line 492 | char  *zfile, *oldfile;
492                                                          /* record progress */
493                  pctdone = 100.0*(vres-1-ypos)/vres;
494                  if (ralrm > 0 && time((time_t *)NULL) >= tlastrept+ralrm)
495 <                        report();
495 >                        report(0);
496   #ifdef SIGCONT
497                  else
498                          signal(SIGCONT, report);
# Line 492 | Line 521 | alldone:
521                  free(sampdens);
522          pctdone = 100.0;
523          if (ralrm > 0)
524 <                report();
524 >                report(0);
525   #ifdef SIGCONT
526          signal(SIGCONT, SIG_DFL);
527   #endif
# Line 504 | Line 533 | memerr:
533   }
534  
535  
536 < fillscanline(scanline, zline, sd, xres, y, xstep)       /* fill scan at y */
537 < register COLOR  *scanline;
538 < register float  *zline;
539 < register char  *sd;
540 < int  xres, y, xstep;
536 > static void
537 > fillscanline(   /* fill scan at y */
538 >        COLOR   *scanline,
539 >        float   *zline,
540 >        char  *sd,
541 >        int  xres,
542 >        int  y,
543 >        int  xstep
544 > )
545   {
546          static int  nc = 0;             /* number of calls */
547          int  bl = xstep, b = xstep;
548          double  z;
549 <        register int  i;
549 >        int  i;
550  
551          z = pixvalue(scanline[0], 0, y);
552          if (zline) zline[0] = z;
# Line 539 | Line 572 | int  xres, y, xstep;
572   }
573  
574  
575 < fillscanbar(scanbar, zbar, xres, y, ysize)      /* fill interior */
576 < register COLOR  *scanbar[];
577 < register float  *zbar[];
578 < int  xres, y, ysize;
575 > static void
576 > fillscanbar(    /* fill interior */
577 >        COLOR   *scanbar[],
578 >        float   *zbar[],
579 >        int  xres,
580 >        int  y,
581 >        int  ysize
582 > )
583   {
584          COLOR  vline[MAXDIV+1];
585          float  zline[MAXDIV+1];
586          int  b = ysize;
587 <        register int  i, j;
587 >        int  i, j;
588  
589          for (i = 0; i < xres; i++) {
590                  copycolor(vline[0], scanbar[0][i]);
# Line 568 | Line 605 | int  xres, y, ysize;
605   }
606  
607  
608 < int
609 < fillsample(colline, zline, x, y, xlen, ylen, b) /* fill interior points */
610 < register COLOR  *colline;
611 < register float  *zline;
612 < int  x, y;
613 < int  xlen, ylen;
614 < int  b;
608 > static int
609 > fillsample( /* fill interior points */
610 >        COLOR   *colline,
611 >        float   *zline,
612 >        int  x,
613 >        int  y,
614 >        int  xlen,
615 >        int  ylen,
616 >        int  b
617 > )
618   {
619          double  ratio;
620          double  z;
621          COLOR  ctmp;
622          int  ncut;
623 <        register int  len;
623 >        int  len;
624  
625          if (xlen > 0)                   /* x or y length is zero */
626                  len = xlen;
# Line 621 | Line 661 | int  b;
661   }
662  
663  
664 < double
665 < pixvalue(col, x, y)             /* compute pixel value */
666 < COLOR  col;                     /* returned color */
667 < int  x, y;                      /* pixel position */
664 > static double
665 > pixvalue(               /* compute pixel value */
666 >        COLOR  col,                     /* returned color */
667 >        int  x,                 /* pixel position */
668 >        int  y
669 > )
670   {
671 +        static COLORMAT xyz2myrgbmat;
672          RAY  thisray;
673          FVECT   lorg, ldir;
674 <        double  hpos, vpos, lmax, d;
674 >        double  hpos, vpos, lmax;
675 >        int     i;
676                                                  /* compute view ray */
677 +        setcolor(col, 0.0, 0.0, 0.0);
678          hpos = (x+pixjitter())/hres;
679          vpos = (y+pixjitter())/vres;
680          if ((thisray.rmax = viewray(thisray.rorg, thisray.rdir,
681 <                                        &ourview, hpos, vpos)) < -FTINY) {
637 <                setcolor(col, 0.0, 0.0, 0.0);
681 >                                        &ourview, hpos, vpos)) < -FTINY)
682                  return(0.0);
683 <        }
684 <
641 <        samplendx = pixnumber(x,y,hres,vres);   /* set pixel index */
642 <
683 >                                                /* set pixel index */
684 >        samplendx = pixnumber(x,y,hres,vres);
685                                                  /* optional motion blur */
686          if (lastview.type && mblur > FTINY && (lmax = viewray(lorg, ldir,
687                                          &lastview, hpos, vpos)) >= -FTINY) {
688 <                register int    i;
647 <                register double  d = mblur*(.5-urand(281+samplendx));
688 >                double  d = mblur*(.5-urand(281+samplendx));
689  
690                  thisray.rmax = (1.-d)*thisray.rmax + d*lmax;
691                  for (i = 3; i--; ) {
# Line 654 | Line 695 | int  x, y;                     /* pixel position */
695                  if (normalize(thisray.rdir) == 0.0)
696                          return(0.0);
697          }
698 +                                                /* depth-of-field */
699 +        if (!jitteraperture(thisray.rorg, thisray.rdir, &ourview, dblur))
700 +                return(0.0);
701  
702 <        rayorigin(&thisray, NULL, PRIMARY, 1.0);
702 >        rayorigin(&thisray, PRIMARY, NULL, NULL);
703  
704          rayvalue(&thisray);                     /* trace ray */
705  
706 <        copycolor(col, thisray.rcol);           /* return color */
706 >        if (out_prims == stdprims) {            /* return color */
707 >                scolor_rgb(col, thisray.rcol);
708 >        } else if (out_prims == xyzprims) {
709 >                scolor_cie(col, thisray.rcol);
710 >                scalecolor(col, WHTEFFICACY);
711 >        } else if (NCSAMP > 3) {
712 >                COLOR   xyz;
713 >                if (xyz2myrgbmat[0][0] == 0)
714 >                        compxyz2rgbWBmat(xyz2myrgbmat, out_prims);
715 >                scolor_cie(xyz, thisray.rcol);
716 >                colortrans(col, xyz2myrgbmat, xyz);
717 >                clipgamut(col, xyz[CIEY], CGAMUT_LOWER, cblack, cwhite);
718 >        } else
719 >                copycolor(col, thisray.rcol);
720  
721 <        return(thisray.rt);                     /* return distance */
721 >        return(raydistance(&thisray));          /* return distance */
722   }
723  
724  
725 < int
726 < salvage(oldfile)                /* salvage scanlines from killed program */
727 < char  *oldfile;
725 > static int
726 > salvage(                /* salvage scanlines from killed program */
727 >        char  *oldfile
728 > )
729   {
730          COLR  *scanline;
731          FILE  *fp;
# Line 693 | Line 751 | char  *oldfile;
751                  goto gotzip;
752          }
753  
754 <        if (x != hres || y != vres) {
754 >        if ((x != hres) | (y != vres)) {
755                  sprintf(errmsg, "resolution mismatch in recover file \"%s\"",
756                                  oldfile);
757                  error(USER, errmsg);
# Line 721 | Line 779 | gotzip:
779   writerr:
780          sprintf(errmsg, "write error during recovery of \"%s\"", oldfile);
781          error(SYSTEM, errmsg);
782 +        return -1; /* pro forma return */
783   }
784  
785 <
786 < int
787 < pixnumber(x, y, xres, yres)             /* compute pixel index (brushed) */
788 < register int  x, y;
789 < int  xres, yres;
785 > static int
786 > pixnumber(              /* compute pixel index (screen door) */
787 >        int  x,
788 >        int  y,
789 >        int  xres,
790 >        int  yres
791 > )
792   {
793 <        x -= y;
794 <        while (x < 0)
795 <                x += xres;
796 <        return((((x>>2)*yres + y) << 2) + (x & 3));
793 >        unsigned        nbits = 0;
794 >        bitmask_t       coord[2];
795 >
796 >        if (xres < yres) xres = yres;
797 >        while (xres > 0) {
798 >                xres >>= 1;
799 >                ++nbits;
800 >        }
801 >        coord[0] = x; coord[1] = y;
802 >        return ((int)hilbert_c2i(2, nbits, coord));
803   }

Diff Legend

– Removed lines
+ Added lines
< Changed lines
> Changed lines