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

Comparing ray/src/rt/RpictSimulManager.cpp (file contents):
Revision 2.1 by greg, Wed Aug 14 20:05:23 2024 UTC vs.
Revision 2.11 by greg, Tue Aug 27 18:50:01 2024 UTC

# Line 9 | Line 9 | static const char RCSid[] = "$Id$";
9   *  Created by Greg Ward on 07/11/2024.
10   */
11  
12 + #define DEBUG   1       // XXX temporary!
13 +
14   #include <ctype.h>
15   #include "platform.h"
16   #include "RpictSimulManager.h"
# Line 50 | Line 52 | PixelAccess::SetPixel(int x, int y, const RAY *rp)
52                  scolor_out(col, primp, rp->rcol);
53                  return SetPixel(x, y, col, zv);
54          default:
55 <                error(INTERNAL, "missing color space type in SetPixel()");
55 >                error(INTERNAL, "botched color space type in SetPixel()");
56          }
57          return false;
58   }
# Line 79 | Line 81 | PixelAccess::SetColorSpace(RenderDataType cs, RGBPRIMP
81          case RDTrgb:
82                  primp = pr ? pr : stdprims;
83                  break;
84 <        default:                        // RDTscolr | RDTscolor
84 >        case RDTscolr:
85 >        case RDTscolor:
86                  primp = NULL;
87                  break;
88 +        default:
89 +                error(INTERNAL, "botched color space type in SetColorSpace()");
90          }
91          dtyp = RDTnewCT(dtyp, cs);
87 –        if (primp)
88 –                xyz2myrgbmat[0][0] = 0;
92          return true;
93   }
94  
# Line 138 | Line 141 | int
141   RpictSimulManager::RtCall(RAY *r, void *cd)
142   {
143          RpictSimulManager *     rsp = (RpictSimulManager *)cd;
144 <        const int               ty = r->rno / rsp->TWidth();
145 <        const int               tx = r->rno - (RNUMBER)ty*rsp->TWidth();
144 >        const int               ty = (r->rno-1) / rsp->TWidth();
145 >        const int               tx = r->rno-1 - (RNUMBER)ty*rsp->TWidth();
146  
147          if (ty >= rsp->THeight()) {
148                  error(INTERNAL, "bad pixel calculation position in RtCall()");
# Line 195 | Line 198 | RpictSimulManager::SetTile(const int ti[2])
198   bool
199   RpictSimulManager::ComputePixel(int x, int y)
200   {
201 <        static const SCOLOR     scBlack = {0};
201 >        DCHECK(doneMap.OffBitMap(x,y),
202 >                        CONSISTENCY, "illegal pixel index in ComputPixel()");
203          int     i;
204          FVECT   rodir[2];
205          double  hpos = (x+pixjitter())/TWidth();
206          double  vpos = (y+pixjitter())/THeight();
207          double  dlim = viewray(rodir[0], rodir[1], &tvw, hpos, vpos);
208          if (dlim < -FTINY) {    // off view?
209 <                pacc.SetPixel(x, y, scBlack);
209 >                pacc.SetPixel(x, y, scblack);
210                  doneMap.Set(x, y);
211                  return true;
212          }
# Line 227 | Line 231 | RpictSimulManager::ComputePixel(int x, int y)
231          for (i = (dlim > FTINY)*3; i--; )
232                  rodir[1][i] *= dlim;
233  
234 <        return EnqueueRay(rodir[0], rodir[1], (RNUMBER)y*TWidth()+x);
234 >        return EnqueueRay(rodir[0], rodir[1], (RNUMBER)y*TWidth()+x+1);
235   }
236  
237   // Check if neighbor differences are below pixel sampling threshold
238   bool
239 < RpictSimulManager::BelowSampThresh(int x, int y, const int noff[4][2]) const
239 > RpictSimulManager::BelowSampThresh(const int x, const int y, const int noff[4][2]) const
240   {
241          SCOLOR  pval[4];
242          float   dist[4];
243          int     i, j;
244  
245          for (i = 4; i--; ) {            // get pixels from tile store
246 <                int     px = x + noff[i][0];
247 <                int     py = y + noff[i][1];
246 >                const int       px = x + noff[i][0];
247 >                const int       py = y + noff[i][1];
248                  if (!doneMap.Check(px, py) ||
249                                  !pacc.GetPixel(px, py, pval[i], &dist[i]))
250                          return false;
251          }
252 <        const bool      spectr = (pacc.NC() > 3);
253 <        for (i = 4; --i; )              // do pairwise comparisons
254 <            for (j = i; j--; ) {
255 <                if (pacc.DepthType() &&
252 <                                (dist[i] - dist[j] > maxdiff*dist[j]) |
252 >                                        // do pairwise comparisons
253 >        for (i = (pacc.DepthType() != RDTnone)*4; --i > 0; )
254 >            for (j = i; j--; )
255 >                if ((dist[i] - dist[j] > maxdiff*dist[j]) |
256                                  (dist[j] - dist[i] > maxdiff*dist[i]))
257                          return false;
258 <                if (spectr ? sbigsdiff(pval[i], pval[j], maxdiff) :
259 <                                bigdiff(pval[i], pval[j], maxdiff))
258 >        if (pacc.NC() > 3) {
259 >            for (i = 4; --i; )
260 >                for (j = i; j--; )
261 >                    if (sbigsdiff(pval[i], pval[j], maxdiff))
262                          return false;
263 <            }
264 <        return true;                    // linear interpolation OK
263 >        } else {
264 >            for (i = 4; --i; )
265 >                for (j = i; j--; )
266 >                    if (bigdiff(pval[i], pval[j], maxdiff))
267 >                        return false;
268 >        }
269 >        return true;
270   }
271  
272   // Fill an interior square patch with interpolated values
273   void
274 < RpictSimulManager::FillSquare(int x, int y, const int noff[4][2])
274 > RpictSimulManager::FillSquare(const int x, const int y, const int noff[4][2])
275   {
276          SCOLOR  pval[4];
277          float   dist[4];
278          int     i, j;
279                                          // assumes 4 corners are valid!
280 <        for (i = 4; i--; )
280 >        for (i = 4; i--; ) {
281 >                DCHECK(!doneMap.Check(x+noff[i][0], y+noff[i][1]),
282 >                        CONSISTENCY, "inclusion of bad pixel in FillSquare()");
283                  pacc.GetPixel(x+noff[i][0], y+noff[i][1], pval[i], &dist[i]);
284 <
284 >        }
285          i = abs(noff[1][0]-noff[0][0]);
286 <        j = abs(noff[1][1]-noff[0][1]);
287 <        const int       slen =  i > j ? i : j;
276 <        const double    sf = 1./slen;
286 >        j = abs(noff[1][1]-noff[0][1]); // i==j for diamond fill
287 >        const int       slen =  (i > j) ? i : j;
288          const bool      spectr = (pacc.NC() > 3);
289 <        for (i = 0; i <= slen; i++) {   // bilinear interpolant
290 <            const double        c1 = i*sf;
291 <            for (j = 0; j <= slen; j++) {
292 <                const double    c2 = j*sf;
289 >        for (i = slen+1 + (i==j)*slen; i--; ) {
290 >            const double        c1 = (i>slen ? i-slen-.5 : (double)i)/slen;
291 >            for (j = slen + (i<=slen); j--; ) {
292 >                const double    c2 = (j + (i>slen)*.5)/slen;
293                  const int       px = int(x + (1.-c1)*(1.-c2)*noff[0][0] +
294                                                  c1*(1.-c2)*noff[1][0] +
295                                                  (1.-c1)*c2*noff[2][0] +
# Line 328 | Line 339 | RpictSimulManager::FillSquare(int x, int y, const int
339  
340   // helper function to set up quincunx sampling
341   static void
342 < SetQuincunx(ABitMap2 *bmp2, int noff[4][2], const int spc, const bool odd)
342 > SetQuincunx(ABitMap2 *bmp2, int noff[4][2], const int spc, bool odd, int x0, int y)
343   {
344 <        for (int y = 0; y < bmp2->Height(); y += spc>>1)
334 <            for (int x = odd*(spc>>1); x < bmp2->Width(); x += spc)
335 <                bmp2->Set(x, y);
336 <                                        // order neighbors CCW
337 <        if (odd) {
344 >        if (odd) {                      // order neighbors CCW
345                  noff[0][0] = spc>>1; noff[0][1] = 0;
346                  noff[1][0] = 0; noff[1][1] = spc>>1;
347                  noff[2][0] = -(spc>>1); noff[2][1] = 0;
# Line 345 | Line 352 | SetQuincunx(ABitMap2 *bmp2, int noff[4][2], const int
352                  noff[2][0] = -(spc>>1); noff[2][1] = -(spc>>1);
353                  noff[3][0] = spc>>1; noff[3][1] = -(spc>>1);
354          }
355 +        int     nsteps;                 // non-negative range
356 +        if (x0 < -(spc>>1)) {
357 +                nsteps = (spc-1 - x0 - (spc>>1))/spc;
358 +                x0 += nsteps*spc;
359 +        }
360 +        if (y < 0) {                    // get past y==0
361 +                nsteps = ((spc>>1)-1 - y)/(spc>>1);
362 +                y += nsteps*(spc>>1);
363 +                odd ^= nsteps&1;
364 +        }
365 +        while (y < bmp2->Height()) {
366 +            for (int x = x0 + odd*(spc>>1); x < bmp2->Width(); x += spc)
367 +                bmp2->Set(x, y);
368 +            y += spc>>1;
369 +            odd = !odd;
370 +        }
371   }
372  
373   // Render (or finish rendering) current tile
374   bool
375 < RpictSimulManager::RenderRect()
375 > RpictSimulManager::RenderRect(const int x0, const int y0)
376   {
377          if (!tvw.type || !Ready()) {
378                  error(INTERNAL, "need octree and view for RenderRect()");
# Line 359 | Line 382 | RpictSimulManager::RenderRect()
382          int             sp2 = ceil(log2((TWidth()>THeight() ? TWidth() : THeight()) - 1.));
383          int             layer = 0;
384          int             x, y;
362 – fprintf(stderr, "Rendering %dx%d tile with psample=%d, maxdiff=%.3f ...\n",
363 – TWidth(), THeight(), psample, maxdiff);
385          while (sp2 > 0) {
386                  ABitMap2        sampMap(TWidth(), THeight());
387                  int             noff[4][2];
388                  if ((prCB != NULL) & (barPix == NULL))
389                          (*prCB)(100.*doneMap.SumTotal()/doneMap.Width()/doneMap.Height());
390 <                SetQuincunx(&sampMap, noff, 1<<sp2, layer&1);
390 >                SetQuincunx(&sampMap, noff, 1<<sp2, layer&1, x0, y0);
391                  sampMap -= doneSamples; // avoid resampling pixels
392                  // Are we into adaptive sampling realm?
393                  if (noff[0][0]*noff[0][0] + noff[0][1]*noff[0][1] < psample*psample) {
394 <                        if (FlushQueue() < 0)   // need results to check threshold
394 >                        if (FlushQueue() < 0)   // need results to check thresholds
395                                  return false;
396                          ABitMap2        fillMap = sampMap;
397                          for (x = y = 0; sampMap.Find(&x, &y); x++)
# Line 378 | Line 399 | TWidth(), THeight(), psample, maxdiff);
399                                          sampMap.Reset(x, y);
400                                          // spread sampling to neighbors...
401                          const ABitMap2  origSampMap = sampMap;
402 <                        for (int yoff = -(1<<(sp2-1));
382 <                                        yoff <= 1<<(sp2-1); yoff += 1<<sp2)
383 <                            for (int xoff = -(1<<(sp2-1));
384 <                                        xoff <= 1<<(sp2-1); xoff += 1<<sp2) {
402 >                        for (x = 4; x--; ) {
403                                  ABitMap2        stamp = origSampMap;
404 <                                stamp.Shift(xoff, yoff);
404 >                                stamp.Shift(noff[x][0] + noff[(x+1)&3][0],
405 >                                                noff[x][1] + noff[(x+1)&3][1]);
406                                  sampMap |= stamp;
407                          }               // ...but don't resample what's done
408                          sampMap -= doneSamples;
409                                          // interpolate smooth regions
410                          fillMap -= sampMap;
411 +                        fillMap -= doneSamples;
412                          for (x = y = 0; fillMap.Find(&x, &y); x++)
413                                  FillSquare(x, y, noff);
414                          doneSamples |= doneMap;
# Line 397 | Line 417 | TWidth(), THeight(), psample, maxdiff);
417                          if (!ComputePixel(x, y))
418                                  return false;
419                  doneSamples |= sampMap; // samples now done or at least queued
400 – fprintf(stderr, "Sampled %ld pixels at (sp2,layer)=(%d,%d)\n",
401 – (long)sampMap.SumTotal(), sp2, layer);
402 – fprintf(stderr, "\t%ld pixels (%.3f%%) completed (+%ld in process)\n",
403 – (long)doneMap.SumTotal(), 100.*doneMap.SumTotal()/doneMap.Width()/doneMap.Height(),
404 – (long)(doneSamples.SumTotal()-doneMap.SumTotal()));
420                  sp2 -= layer++ & 1;     // next denser sampling
421          }
422          if (FlushQueue() < 0)           // make sure we got everyone
423                  return false;
424          x = y = 0;
425          if (doneMap.Find(&x, &y, false)) {
426 <                sprintf(errmsg, "missed %ld tile pixels, e.g. (%d,%d)",
427 <                                (long)doneMap.Width()*doneMap.Height() -
428 <                                        doneMap.SumTotal(), x, y);
426 >                sprintf(errmsg, "missed %.4f%% of pixels in rectangle\n",
427 >                                100. - 100.*doneMap.SumTotal() /
428 >                                        doneMap.Width() / doneMap.Height());
429                  error(WARNING, errmsg);
430          }
431          if ((prCB != NULL) & (barPix == NULL))
# Line 436 | Line 451 | RpictSimulManager::RenderTile(COLORV *rp, int ystride,
451                  pacc.SetColorSpace(RDTxyz);
452          else if (prims)
453                  pacc.SetColorSpace(RDTrgb, prims);
454 <                
454 >
455          return SetTile(tile) && RenderRect();
456   }
457  
# Line 474 | Line 489 | RpictSimulManager::RenderTile(COLRV *bp, int ystride,
489          return SetTile(tile) && RenderRect();
490   }
491  
492 + // Back to float color with 16-bit depth
493 + bool
494 + RpictSimulManager::RenderTile(COLORV *rp, int ystride, short *dp, const int *tile)
495 + {
496 +        if (!rp | (GetWidth() <= 0) | (GetHeight() <= 0) | !vw.type)
497 +                return false;
498 +        if (!ystride)                   // contiguous rows?
499 +                ystride = TWidth();
500 +        pacc.Init(rp, ystride, dp);
501 +        if (prims == xyzprims)
502 +                pacc.SetColorSpace(RDTxyz);
503 +        else if (prims)
504 +                pacc.SetColorSpace(RDTrgb, prims);
505 +
506 +        return SetTile(tile) && RenderRect();
507 + }
508 +
509   // Allocate a new render bar
510   void
511   RpictSimulManager::NewBar(int ht)
# Line 504 | Line 536 | RpictSimulManager::NewBar(int ht)
536  
537   // Shift render bar area the specified amount down the frame
538   bool
539 < RpictSimulManager::LowerBar(int v)
539 > RpictSimulManager::LowerBar(int v, int ytop)
540   {
509 –        if (v <= 0) return !v;
541          if (!barPix | !barDepth | (v > THeight()) | !tvw.type)
542                  return false;
543 <        tvw.voff -= double(v)/GetHeight();
544 <        ptvw.voff -= double(v)/GetHeight();
543 >        if (v <= 0) return !v;
544 >        if ((ytop -= v) <= 0)
545 >                return true;
546 >        tvw.voff -= double(v)/THeight();
547 >        ptvw.voff -= double(v)/THeight();
548          if (v == THeight()) {
549                  doneMap.ClearBitMap();
550                  return true;
# Line 521 | Line 555 | RpictSimulManager::LowerBar(int v)
555                          sizeof(COLORV)*NC*TWidth()*(THeight()-v));
556          memmove(barDepth, barDepth + TWidth()*v,
557                          sizeof(float)*TWidth()*(THeight()-v));
558 +        if (ytop < THeight())                   // mark what we won't do as finished
559 +                doneMap.ClearRect(0, 0, TWidth(), THeight()-ytop, true);
560          return true;
561   }
562  
# Line 543 | Line 579 | RpictSimulManager::RenderBelow(int ytop, const int vst
579                          cropview(&ptvw, 0., double(ytop-THeight())/GetHeight(),
580                                                  1., double(ytop)/GetHeight()))
581                  ptvw.type = 0;
582 <                                                // set up spectral sampling
583 <        if (setspectrsamp(CNDX, WLPART) <= 0) {
582 >                                                // update spectral sampling
583 >        int     rv = setspectrsamp(CNDX, WLPART);
584 >        if (rv < 0) {
585                  error(USER, "unsupported spectral sampling");
586                  return false;
587          }
588 +        if (!rv & (RDTcolorT(dt) != RDTscolor) & (RDTcolorT(dt) != RDTscolr))
589 +                error(WARNING, "spectral range incompatible with color output");
590          COLORV **       parr = NULL;            // set up tiny source drawing
591          float **        zarr = NULL;
592          if (!ptvw.type && directvis && dblur <= FTINY) {
# Line 562 | Line 601 | RpictSimulManager::RenderBelow(int ytop, const int vst
601          }
602          int             lastOut = ytop;         // render down frame
603          while (ytop > 0) {
565 –                if (ytop < THeight())           // mark what we won't do as finished
566 –                        doneMap.ClearRect(0, 0, TWidth(), THeight()-ytop, true);
604                  if (prCB)
605                          (*prCB)(100.*(GetHeight()-ytop)/GetHeight());
606 <                if (!RenderRect())              // render this bar
606 >                if (!RenderRect(0, THeight()-ytop))     // render this bar
607                          return false;
608                  int     nlines = lastOut - ytop + THeight();
609                  if (nlines > ytop)
# Line 605 | Line 642 | RpictSimulManager::RenderBelow(int ytop, const int vst
642                                          error(SYSTEM, "cannot write SCOLOR output");
643                                  break;
644                          default:
645 <                                error(INTERNAL, "missing output color type in RenderBelow()");
645 >                                error(INTERNAL, "botched output color type in RenderBelow()");
646                                  break;
647                          }
648                          bpos += pacc.NC()*TWidth();
# Line 614 | Line 651 | RpictSimulManager::RenderBelow(int ytop, const int vst
651                  if (fflush(pfp) == EOF || (dfp && fflush(dfp) == EOF))
652                          error(SYSTEM, "output write error");
653                                                  // advance down the frame
654 <                if (lastOut > 0 && !LowerBar(vstep))
654 >                if (lastOut > 0 && !LowerBar(vstep, ytop))
655                          return false;
656                  ytop -= vstep;
657          }
# Line 625 | Line 662 | RpictSimulManager::RenderBelow(int ytop, const int vst
662          return true;
663   }
664  
665 < /*
629 < * Render and write a frame to the named file
630 < * Include any header lines set prior to call
631 < * Picture file must not already exist
632 < * Picture to stdout if pfname==NULL
633 < * Depth written to a command if dfname[0]=='!'
634 < */
665 > // Open new output picture file (and optional depth file)
666   RenderDataType
667 < RpictSimulManager::RenderFrame(const char *pfname, RenderDataType dt, const char *dfname)
667 > RpictSimulManager::NewOutput(FILE *pdfp[2], const char *pfname,
668 >                                RenderDataType dt, const char *dfname)
669   {
670 <        int     fd = 1;
639 <        FILE *  pfp = NULL;
640 <        FILE *  dfp = NULL;
641 <
670 >        pdfp[0] = pdfp[1] = NULL;
671          if (!RDTcolorT(dt))
672 <                error(INTERNAL, "missing pixel output type in RenderFrame()");
672 >                error(INTERNAL, "missing color output type in NewOutput()");
673          if (NCSAMP == 3) {
674                  if (RDTcolorT(dt) == RDTscolr)
675                          dt = RDTnewCT(dt, prims==xyzprims ? RDTxyze : RDTrgbe);
# Line 648 | Line 677 | RpictSimulManager::RenderFrame(const char *pfname, Ren
677                          dt = RDTnewCT(dt, prims==xyzprims ? RDTxyz : RDTrgb);
678          }
679          if (!RDTdepthT(dt) ^ !dfname)
680 <                error(INTERNAL, "depth output requires file name and type in RenderFrame()");
680 >                error(INTERNAL, "depth output requires file name and type in NewOutput()");
681 >        int     fd = 1;
682          if (pfname) {                           // open picture output file
683                  if (pfname[0] == '!') {
684                          error(INTERNAL, "writing picture to a command not supported");
# Line 661 | Line 691 | RpictSimulManager::RenderFrame(const char *pfname, Ren
691                          sprintf(errmsg, "cannot open picture file '%s'", pfname);
692                          error(SYSTEM, errmsg);
693                  }
694 <                return RDTnone;                 // expected in parallel sequence
694 >                return RDTnone;                 // may be expected in sequence run
695          }
696          if (fd == 1)
697 <                pfp = stdout;
698 <        else if (!(pfp = fdopen(fd, "w")))
697 >                pdfp[0] = stdout;
698 >        else if (!(pdfp[0] = fdopen(fd, "w")))
699                  error(SYSTEM, "failure calling fdopen()");
700 <        SET_FILE_BINARY(pfp);                   // write picture header
701 <        if ((pfp != stdout) | (frameNo <= 1)) {
702 <                newheader("RADIANCE", pfp);
703 <                fputs(GetHeader(), pfp);
700 >        SET_FILE_BINARY(pdfp[0]);               // write picture header
701 >        if ((pdfp[0] != stdout) | (frameNo <= 1)) {
702 >                newheader("RADIANCE", pdfp[0]);
703 >                fputs(GetHeader(), pdfp[0]);
704          }
705 <        fputs(VIEWSTR, pfp); fprintview(&vw, pfp); fputc('\n', pfp);
705 >        fputs(VIEWSTR, pdfp[0]); fprintview(&vw, pdfp[0]); fputc('\n', pdfp[0]);
706          if (frameNo > 0)
707 <                fprintf(pfp, "FRAME=%d\n", frameNo);
707 >                fprintf(pdfp[0], "FRAME=%d\n", frameNo);
708          double  pasp = viewaspect(&vw) * GetWidth() / GetHeight();
709 <        if (!FABSEQ(pasp, 1.0))
710 <                fputaspect(pasp, pfp);
711 <        fputnow(pfp);
709 >        if ((0.99 > pasp) | (pasp > 1.01))
710 >                fputaspect(pasp, pdfp[0]);
711 >        fputnow(pdfp[0]);
712          switch (RDTcolorT(dt)) {                // set primaries and picture format
713          case RDTrgbe:
714                  if (!prims | (prims == xyzprims)) prims = stdprims;
715 <                fputprims(prims, pfp);
716 <                fputformat(COLRFMT, pfp);
715 >                fputprims(prims, pdfp[0]);
716 >                fputformat(COLRFMT, pdfp[0]);
717                  break;
718          case RDTxyze:
719                  prims = xyzprims;
720 <                fputformat(CIEFMT, pfp);
720 >                fputformat(CIEFMT, pdfp[0]);
721                  break;
722          case RDTscolr:
723                  prims = NULL;
724 <                fputwlsplit(WLPART, pfp);
725 <                fputncomp(NCSAMP, pfp);
726 <                fputformat(SPECFMT, pfp);
724 >                fputwlsplit(WLPART, pdfp[0]);
725 >                fputncomp(NCSAMP, pdfp[0]);
726 >                fputformat(SPECFMT, pdfp[0]);
727                  break;
728          case RDTrgb:
729                  if (!prims | (prims == xyzprims)) prims = stdprims;
730 <                fputprims(prims, pfp);
731 <                fputncomp(3, pfp);
732 <                fputendian(pfp);
733 <                fputformat("float", pfp);
730 >                fputprims(prims, pdfp[0]);
731 >                fputncomp(3, pdfp[0]);
732 >                fputendian(pdfp[0]);
733 >                fputformat("float", pdfp[0]);
734                  break;
735          case RDTxyz:
736                  prims = xyzprims;
737 <                fputprims(prims, pfp);
738 <                fputncomp(3, pfp);
739 <                fputendian(pfp);
740 <                fputformat("float", pfp);
737 >                fputprims(prims, pdfp[0]);
738 >                fputncomp(3, pdfp[0]);
739 >                fputendian(pdfp[0]);
740 >                fputformat("float", pdfp[0]);
741                  break;
742          case RDTscolor:
743                  prims = NULL;
744 <                fputwlsplit(WLPART, pfp);
745 <                fputncomp(NCSAMP, pfp);
746 <                fputendian(pfp);
747 <                fputformat("float", pfp);
744 >                fputwlsplit(WLPART, pdfp[0]);
745 >                fputncomp(NCSAMP, pdfp[0]);
746 >                fputendian(pdfp[0]);
747 >                fputformat("float", pdfp[0]);
748                  break;
749 <        default:;
749 >        default:;       // pro forma - caught this above
750          }
751 <        fputc('\n', pfp);                       // end picture header
752 <        fprtresolu(GetWidth(), GetHeight(), pfp);
753 <        if (dfname) {
751 >        fputc('\n', pdfp[0]);                   // flush picture header
752 >        if (fflush(pdfp[0]) == EOF) {
753 >                sprintf(errmsg, "cannot write header to picture '%s'", pfname);
754 >                error(SYSTEM, errmsg);
755 >                fclose(pdfp[0]);
756 >                pdfp[0] = NULL;
757 >                return RDTnone;
758 >        }
759 >        if (dfname) {                           // open depth output
760                  if (dfname[0] == '!')
761 <                        dfp = popen(dfname+1, "w");
761 >                        pdfp[1] = popen(dfname+1, "w");
762                  else
763 <                        dfp = fopen(dfname, "w");
764 <                if (!dfp) {
763 >                        pdfp[1] = fopen(dfname, "w");
764 >                if (!pdfp[1]) {
765                          sprintf(errmsg, "cannot open depth output '%s'", dfname);
766                          error(SYSTEM, errmsg);
767 +                        fclose(pdfp[0]);
768 +                        pdfp[0] = NULL;
769                          return RDTnone;
770                  }
771 <                SET_FILE_BINARY(dfp);
771 >                SET_FILE_BINARY(pdfp[1]);
772          }
773          if (RDTdepthT(dt) == RDTdshort) {       // write header for 16-bit depth?
774 <                newheader("RADIANCE", dfp);
775 <                fputs(GetHeader(), dfp);
776 <                fputs(VIEWSTR, dfp); fprintview(&vw, dfp); fputc('\n', dfp);
777 <                fputs(DEPTHSTR, dfp); fputs(dunit, dfp); fputc('\n', dfp);
778 <                fputformat(DEPTH16FMT, dfp);
779 <                fputc('\n', dfp);               // end-of-info
780 <                fprtresolu(GetWidth(), GetHeight(), dfp);
774 >                newheader("RADIANCE", pdfp[1]);
775 >                fputs(GetHeader(), pdfp[1]);
776 >                fputs(VIEWSTR, pdfp[1]); fprintview(&vw, pdfp[1]); fputc('\n', pdfp[1]);
777 >                fputs(DEPTHSTR, pdfp[1]); fputs(dunit, pdfp[1]); fputc('\n', pdfp[1]);
778 >                fputformat(DEPTH16FMT, pdfp[1]);
779 >                fputc('\n', pdfp[1]);           // end-of-info
780 >                if (fflush(pdfp[1]) == EOF) {
781 >                        sprintf(errmsg, "cannot write header to '%s'", dfname);
782 >                        error(SYSTEM, errmsg);
783 >                        fclose(pdfp[0]); fclose(pdfp[1]);
784 >                        pdfp[0] = pdfp[1] = NULL;
785 >                        return RDTnone;
786 >                }
787          }
788 <        const int       bheight = (psample > 1) ? int(2*psample+.99) : 4;
789 <        const int       vstep =  bheight >> (psample > 1);
788 >        return dt;                              // ready to roll
789 > }
790  
791 + /*
792 + * Render and write a frame to the named file
793 + * Include any header lines set prior to call
794 + * Picture file must not exist
795 + * Write pixels to stdout if !pfname
796 + * Write depth to a command if dfname[0]=='!'
797 + */
798 + RenderDataType
799 + RpictSimulManager::RenderFrame(const char *pfname, RenderDataType dt, const char *dfname)
800 + {
801 +        FILE    *pdfp[2];
802 +                                                // prepare output file(s)
803 +        dt = NewOutput(pdfp, pfname, dt, dfname);
804 +        if (dt == RDTnone)
805 +                return RDTnone;
806 +                                                // add resolution string(s)
807 +        fprtresolu(GetWidth(), GetHeight(), pdfp[0]);
808 +        if (RDTdepthT(dt) == RDTdshort)
809 +                fprtresolu(GetWidth(), GetHeight(), pdfp[1]);
810 +
811 +        const int       bheight = (psample > 1) ? int(8*psample+.99) : 16;
812 +        const int       vstep = bheight >> (psample > 1);
813 +
814          NewBar(bheight);                        // render frame if we can
815 <        if (!RenderBelow(GetHeight(), vstep, pfp, dt, dfp)) {
816 <                fclose(pfp);
817 <                if (dfp) (dfname[0] == '!') ? pclose(dfp) : fclose(dfp);
815 >        if (!RenderBelow(GetHeight(), vstep, pdfp[0], dt, pdfp[1])) {
816 >                fclose(pdfp[0]);
817 >                if (pdfp[1]) (dfname[0] == '!') ? pclose(pdfp[1]) : fclose(pdfp[1]);
818                  Cleanup();
819                  return RDTnone;
820          }
821          NewBar();                               // clean up and return
822 <        if (pfp != stdout)
823 <                fclose(pfp);
824 <        if (dfp) {
822 >        if (pdfp[0] != stdout)
823 >                fclose(pdfp[0]);
824 >        if (pdfp[1]) {
825                  if (dfname[0] == '!') {
826 <                        int     status = pclose(dfp);
826 >                        int     status = pclose(pdfp[1]);
827                          if (status) {
828                                  sprintf(errmsg, "depth output (%s) error status: %d",
829                                                  dfname, status);
# Line 764 | Line 831 | RpictSimulManager::RenderFrame(const char *pfname, Ren
831                                  return RDTnone;
832                          }
833                  } else
834 <                        fclose(dfp);
834 >                        fclose(pdfp[1]);
835          }
836          return dt;
837   }
838  
839   // passed struct for header line callback
840 < struct HeaderInfo {
840 > static struct HeaderInfo {
841          char            fmt[MAXFMTLEN];
842          char            depth_unit[32];
843          int             ncomp;
# Line 791 | Line 858 | struct HeaderInfo {
858                                  gotview = false;
859                                  endianMatch = true;
860                          }
861 < };
861 > }       hinfo;          // XXX single copy to hold custom primitives
862  
863   // helper function checks header line and records req. info.
864   static int
# Line 833 | Line 900 | head_check(char *s, void *p)
900          return 0;
901   }
902  
903 < // Resume partially finished rendering
837 < // Picture file must exist
903 > // Reopen output file(s), leaving pointers at end of (each) header
904   RenderDataType
905 < RpictSimulManager::ResumeFrame(const char *pfname, const char *dfname)
905 > RpictSimulManager::ReopenOutput(FILE *pdfp[2], const char *pfname, const char *dfname)
906   {
907 <        if (!pfname || pfname[0] == '!')
842 <                return RDTnone;
907 >        extern const char       HDRSTR[];
908  
909 +        if (!pfname || pfname[0] == '!') {
910 +                pdfp[0] = pdfp[1] = NULL;
911 +                return RDTnone;
912 +        }
913          RenderDataType  dt = RDTnone;
914 <        FILE *          dfp = NULL;
915 <        FILE *          pfp = fopen(pfname, "r+");
916 <        if (!pfp) {
914 >        pdfp[1] = NULL;
915 >        pdfp[0] = fopen(pfname, "r+");
916 >        if (!pdfp[0]) {
917                  sprintf(errmsg, "cannot reopen output picture '%s'", pfname);
918                  error(SYSTEM, errmsg);
919                  return RDTnone;
920          }
921 <        SET_FILE_BINARY(pfp);
922 <        HeaderInfo      hinfo;          // read header information & dimensions
923 <        RESOLU          res;
924 <        if (getheader(pfp, head_check, &hinfo) < 0) {
856 <                fclose(pfp);
921 >        SET_FILE_BINARY(pdfp[0]);       // read header information
922 >        if (getheader(pdfp[0], head_check, &hinfo) < 0) {
923 >                fclose(pdfp[0]);
924 >                pdfp[0] = NULL;
925                  return RDTnone;
926          }
859 –        if (!fgetsresolu(&res, pfp) || res.rt != PIXSTANDARD) {
860 –                sprintf(errmsg, "missing/bad resolution for '%s'", pfname);
861 –                error(USER, errmsg);
862 –                fclose(pfp);
863 –                return RDTnone;
864 –        }
927          if (!hinfo.gotview) {
928                  sprintf(errmsg, "missing view for '%s'", pfname);
929                  error(USER, errmsg);
930 <                fclose(pfp);
930 >                fclose(pdfp[0]);
931 >                pdfp[0] = NULL;
932                  return RDTnone;
933          }
934          if (hinfo.ncomp < 3) {
935                  sprintf(errmsg, "bad # components (%d) in '%s'", hinfo.ncomp, pfname);
936                  error(USER, errmsg);
937 <                fclose(pfp);
937 >                fclose(pdfp[0]);
938 >                pdfp[0] = NULL;
939                  return RDTnone;
940          }
941 <        int     bytesPer = 0;           // complicated part to set rendering/output space
941 >                                        // set rendering/output space
942          if (!strcmp(hinfo.fmt, COLRFMT)) {
943 <                prims = hinfo.prims;
943 >                prims = hinfo.prims;    // XXX static array
944                  int     n = 8*hinfo.gotprims;
945                  while (n--)
946                          if (!FABSEQ(hinfo.prims[0][n], stdprims[0][n]))
# Line 892 | Line 956 | RpictSimulManager::ResumeFrame(const char *pfname, con
956                          sprintf(errmsg, "incompatible sample count (%d) in '%s'",
957                                          hinfo.ncomp, pfname);
958                          error(USER, errmsg);
959 <                        fclose(pfp);
959 >                        fclose(pdfp[0]);
960 >                        pdfp[0] = NULL;
961                          return RDTnone;
962                  }
963 <                NCSAMP = hinfo.ncomp;   // overrides global setting
963 >                NCSAMP = hinfo.ncomp;           // overrides global setting
964                  prims = NULL;
965                  dt = RDTnewCT(dt, RDTscolr);
901 –                bytesPer = hinfo.ncomp + 1;     // XXX assumes no compression
966          } else if (!strcmp(hinfo.fmt, "float")) {
967                  if (!hinfo.endianMatch) {
968                          sprintf(errmsg, "incompatible byte ordering in '%s'", pfname);
969                          error(USER, errmsg);
970 <                        fclose(pfp);
970 >                        fclose(pdfp[0]);
971 >                        pdfp[0] = NULL;
972                          return RDTnone;
973                  }
974                  if (hinfo.ncomp == 3) {
975 <                        prims = hinfo.prims;            // custom primaries?
975 >                        prims = hinfo.prims;    // custom primaries?
976                          int     n = 8*hinfo.gotprims;
977                          while (n--)
978                                  if (!FABSEQ(hinfo.prims[0][n], stdprims[0][n]))
979                                          break;
980 <                        if (n < 0)                      // standard primaries?
980 >                        if (n < 0)              // standard primaries?
981                                  prims = stdprims;
982                          else if (hinfo.gotprims) {      // or check if XYZ
983                                  for (n = 8; n--; )
# Line 930 | Line 995 | RpictSimulManager::ResumeFrame(const char *pfname, con
995                          prims = NULL;
996                          dt = RDTnewCT(dt, RDTscolor);
997                  }
933 –                bytesPer = sizeof(float)*hinfo.ncomp;
998          } else {
999                  sprintf(errmsg, "unknown format (%s) for '%s'", hinfo.fmt, pfname);
1000                  error(USER, errmsg);
1001 <                fclose(pfp);
1001 >                fclose(pdfp[0]);
1002 >                pdfp[0] = NULL;
1003                  return RDTnone;
1004          }
1005 +        if (!dfname)                            // no depth file?
1006 +                return dt;
1007 +
1008 +        if (dfname[0] == '!') {
1009 +                error(USER, "depth data cannot be reloaded from command");
1010 +                fclose(pdfp[0]);
1011 +                pdfp[0] = NULL;
1012 +                return RDTnone;
1013 +        }
1014 +        pdfp[1] = fopen(dfname, "r+");
1015 +        if (!pdfp[1]) {
1016 +                sprintf(errmsg, "cannot reopen depth file '%s'", dfname);
1017 +                error(SYSTEM, errmsg);
1018 +                fclose(pdfp[0]);
1019 +                pdfp[0] = NULL;
1020 +                return RDTnone;
1021 +        }
1022 +        SET_FILE_BINARY(pdfp[1]);
1023 +        int     n, len = strlen(HDRSTR);
1024 +        char    buf[32];                // sniff for 16-bit header
1025 +        if (getbinary(buf, 1, len+1, pdfp[1]) < len+1) {
1026 +                sprintf(errmsg, "empty depth file '%s'", dfname);
1027 +                error(SYSTEM, errmsg);
1028 +                fclose(pdfp[0]); fclose(pdfp[1]);
1029 +                pdfp[0] = pdfp[1] = NULL;
1030 +                return RDTnone;
1031 +        }
1032 +        for (n = 0; n < len; n++)
1033 +                if (buf[n] != HDRSTR[n])
1034 +                        break;          // not a Radiance header
1035 +        rewind(pdfp[1]);
1036 +        if ((n < len) | !isprint(buf[len]))
1037 +                return RDTnewDT(dt, RDTdfloat);
1038 +
1039 +        HeaderInfo      dinfo;          // thinking it's 16-bit encoded
1040 +        if (getheader(pdfp[1], head_check, &dinfo) < 0)
1041 +                sprintf(errmsg, "bad header in encoded depth file '%s'",
1042 +                                dfname);
1043 +        else if (strcmp(dinfo.fmt, DEPTH16FMT))
1044 +                sprintf(errmsg, "wrong format (%s) for depth file '%s'",
1045 +                                dinfo.fmt, dfname);
1046 +        else if (!SetReferenceDepth(dinfo.depth_unit))
1047 +                sprintf(errmsg, "bad/missing reference depth (%s) in '%s'",
1048 +                                dinfo.depth_unit, dfname);
1049 +        else
1050 +                errmsg[0] = '\0';
1051 +
1052 +        if (errmsg[0]) {
1053 +                error(USER, errmsg);
1054 +                fclose(pdfp[1]); fclose(pdfp[0]);
1055 +                pdfp[0] = pdfp[1] = NULL;
1056 +                return RDTnone;
1057 +        }
1058 +        return RDTnewDT(dt, RDTdshort);
1059 + }
1060 +
1061 + // Resume partially finished rendering
1062 + // Picture file must exist
1063 + RenderDataType
1064 + RpictSimulManager::ResumeFrame(const char *pfname, const char *dfname)
1065 + {
1066 +        FILE            *pdfp[2];
1067 +
1068 +        RenderDataType  dt = ReopenOutput(pdfp, pfname, dfname);
1069 +        if (dt == RDTnone)
1070 +                return RDTnone;
1071 +
1072 +        int     bytesPer = 0;           // figure out how far we got...
1073 +        switch (RDTcolorT(dt)) {
1074 +        case RDTrgbe:
1075 +        case RDTxyze:
1076 +                break;
1077 +        case RDTscolr:
1078 +                bytesPer = hinfo.ncomp + 1;     // XXX assumes no compression
1079 +                break;
1080 +        case RDTrgb:
1081 +        case RDTxyz:
1082 +                bytesPer = sizeof(float)*3;
1083 +                break;
1084 +        case RDTscolor:
1085 +                bytesPer = sizeof(float)*hinfo.ncomp;
1086 +                break;
1087 +        default:
1088 +                sprintf(errmsg, "unknown format (%s) for '%s'", hinfo.fmt, pfname);
1089 +                error(USER, errmsg);
1090 +                fclose(pdfp[0]);
1091 +                if (pdfp[1]) fclose(pdfp[1]);
1092 +                return RDTnone;
1093 +        }
1094 +        RESOLU  res;
1095 +        if (!fgetsresolu(&res, pdfp[0]) || res.rt != PIXSTANDARD) {
1096 +                sprintf(errmsg, "missing/bad resolution for '%s'", pfname);
1097 +                error(USER, errmsg);
1098 +                fclose(pdfp[0]);
1099 +                if (pdfp[1]) fclose(pdfp[1]);
1100 +                return RDTnone;
1101 +        }
1102          vw.type = 0;                            // set up new (unreferenced) frame
1103          frameNo = 0;
1104          int     hvdim[2] = {res.xr, res.yr};
1105          double  noAdj = 0;
1106          if (!NewFrame(hinfo.vw, hvdim, &noAdj) ||
1107                          (hvdim[0] != res.xr) | (hvdim[1] != res.yr)) {
1108 <                fclose(pfp);
1108 >                error(CONSISTENCY, "unexpected resolution change in ResumeFrame()");
1109 >                fclose(pdfp[0]);
1110 >                if (pdfp[1]) fclose(pdfp[1]);
1111                  return RDTnone;
1112          }
1113 <        long    dataStart = ftell(pfp);         // picture starting point
1113 >        long    dataStart = ftell(pdfp[0]);     // picture starting point
1114          if (dataStart < 0) {
1115                  sprintf(errmsg, "cannot seek on '%s'", pfname);
1116                  error(SYSTEM, errmsg);
1117 <                fclose(pfp);
1117 >                fclose(pdfp[0]);
1118 >                if (pdfp[1]) fclose(pdfp[1]);
1119                  return RDTnone;
1120          }
1121          long    doneScans = 0;
1122          if (bytesPer) {                         // fixed-width records?
1123 <                fseek(pfp, 0, SEEK_END);
1124 <                long    dataEnd = ftell(pfp);
1123 >                fseek(pdfp[0], 0, SEEK_END);
1124 >                long    dataEnd = ftell(pdfp[0]);
1125                  doneScans = (dataEnd - dataStart)/(bytesPer*GetWidth());
1126                  if (dataEnd-dataStart > bytesPer*GetWidth()*doneScans)
1127 <                        fseek(pfp, dataStart + bytesPer*GetWidth()*doneScans, SEEK_SET);
1127 >                        fseek(pdfp[0], dataStart + bytesPer*GetWidth()*doneScans, SEEK_SET);
1128          } else {                                // else get compressed scanlines
1129                  COLR *  scan = (COLR *)tempbuffer(sizeof(COLR)*GetWidth());
1130 <                while (freadcolrs(scan, GetWidth(), pfp) >= 0)
1130 >                while (freadcolrs(scan, GetWidth(), pdfp[0]) >= 0)
1131                          ++doneScans;
1132 <                if (!feof(pfp)) {
1132 >                if (!feof(pdfp[0])) {
1133                          sprintf(errmsg, "error reading compressed scanline from '%s'", pfname);
1134                          error(USER, errmsg);
1135 <                        fclose(pfp);
1135 >                        fclose(pdfp[0]);
1136 >                        if (pdfp[1]) fclose(pdfp[1]);
1137                          return RDTnone;
1138                  }
1139          }
1140          if (doneScans >= GetHeight()) {         // nothing left to do?
1141                  sprintf(errmsg, "output file '%s' is already complete", pfname);
1142                  error(WARNING, errmsg);
1143 <                fclose(pfp);
1143 >                fclose(pdfp[0]);
1144 >                if (pdfp[1]) fclose(pdfp[1]);
1145                  return dt;
1146          }
1147          if (!doneScans) {
1148                  sprintf(errmsg, "restarting empty frame '%s'", pfname);
1149                  error(WARNING, errmsg);
1150          }
1151 <        if (dfname) {                           // append depth file, too?
1152 <                if (dfname[0] == '!') {
1153 <                        error(USER, "depth data cannot be reloaded from command");
1154 <                        fclose(pfp);
1151 >        long    toSkip = 0;
1152 >        switch (RDTdepthT(dt)) {                // append depth file, too?
1153 >        case RDTdfloat:
1154 >                toSkip = sizeof(float)*GetWidth()*doneScans;
1155 >                break;
1156 >        case RDTdshort:
1157 >                if (!fgetsresolu(&res, pdfp[1]) || (res.rt != PIXSTANDARD) |
1158 >                                (res.xr != GetWidth()) | (res.yr != GetHeight())) {
1159 >                        sprintf(errmsg, "missing/bad resolution for '%s'", dfname);
1160 >                        error(USER, errmsg);
1161 >                        fclose(pdfp[0]); fclose(pdfp[0]);
1162                          return RDTnone;
1163                  }
1164 <                dfp = fopen(dfname, "a");
1165 <                if (!dfp) {
1166 <                        sprintf(errmsg, "cannot reopen depth file '%s'", dfname);
1167 <                        error(SYSTEM, errmsg);
1168 <                        fclose(pfp);
1169 <                        return RDTnone;
1170 <                }
1171 <                SET_FILE_BINARY(dfp);
1172 <                const long      dflen = ftell(dfp);
999 <                if (dflen != sizeof(float)*GetWidth()*doneScans) {
1000 <                        fclose(dfp);
1001 <                        dfp = fopen(dfname, "r+");
1002 <                        if (!dfp) return RDTnone;       // WTH?
1003 <                        SET_FILE_BINARY(dfp);
1004 <                }
1005 <                if (dflen < sizeof(float)*GetWidth()*doneScans) {
1006 <                        HeaderInfo      dinfo;
1007 <                        if (getheader(dfp, head_check, &dinfo) < 0)
1008 <                                sprintf(errmsg, "bad header in encoded depth file '%s'",
1009 <                                                dfname);
1010 <                        else if (strcmp(dinfo.fmt, DEPTH16FMT))
1011 <                                sprintf(errmsg, "wrong format (%s) for depth file '%s'",
1012 <                                                dinfo.fmt, dfname);
1013 <                        else if (!SetReferenceDepth(dinfo.depth_unit))
1014 <                                sprintf(errmsg, "bad/missing reference depth (%s) in '%s'",
1015 <                                                dinfo.depth_unit, dfname);
1016 <                        else if (!fscnresolu(hvdim, hvdim+1, dfp) ||
1017 <                                        (hvdim[0] != GetWidth()) | (hvdim[1] != GetHeight()))
1018 <                                sprintf(errmsg, "bad/mismatched resolution in '%s'",
1019 <                                                dfname);
1020 <                        else
1021 <                                errmsg[0] = '\0';
1022 <
1023 <                        if (errmsg[0]) {
1024 <                                error(USER, errmsg);
1025 <                                fclose(dfp);
1026 <                                fclose(pfp);
1027 <                                return RDTnone;
1028 <                        }
1029 <                        const long      dStart = ftell(dfp);
1030 <                        if (dflen-dStart < 2*GetWidth()*doneScans) {
1031 <                                sprintf(errmsg, "missing %ld depths in '%s'",
1032 <                                        (long)GetWidth()*doneScans - (dflen-dStart)/2,
1033 <                                        dfname);
1034 <                                error(WARNING, errmsg);
1035 <                        }
1036 <                        fseek(dfp, dStart + 2*GetWidth()*doneScans, SEEK_SET);
1037 <                        dt = RDTnewDT(dt, RDTdshort);
1038 <                } else {
1039 <                        if (dflen > sizeof(float)*GetWidth()*doneScans)
1040 <                                fseek(dfp, sizeof(float)*GetWidth()*doneScans, SEEK_SET);
1041 <                        dt = RDTnewDT(dt, RDTdfloat);
1042 <                }
1164 >                toSkip = 2L*GetWidth()*doneScans;
1165 >                break;
1166 >        default:;
1167 >        }                                       // fseek() needed for output
1168 >        if (pdfp[1] && fseek(pdfp[1], toSkip, SEEK_CUR) < 0) {
1169 >                sprintf(errmsg, "cannot seek on depth file '%s'", dfname);
1170 >                error(SYSTEM, errmsg);
1171 >                fclose(pdfp[0]); fclose(pdfp[1]);
1172 >                return RDTnone;
1173          }
1174 <        int     bheight = (psample > 1) ? int(2*psample+.99) : 4;
1174 >        int     bheight = (psample > 1) ? int(8*psample+.99) : 16;
1175          if (bheight > GetHeight()-doneScans)
1176                  bheight = GetHeight()-doneScans;
1177          int     vstep =  bheight >> (psample > 1);
1178          vstep += !vstep;
1179  
1180          NewBar(bheight);                        // render remainder if we can
1181 <        if (!RenderBelow(GetHeight()-doneScans, vstep, pfp, dt, dfp)) {
1182 <                fclose(pfp);
1183 <                if (dfp) fclose(dfp);
1181 >        if (!RenderBelow(GetHeight()-doneScans, vstep, pdfp[0], dt, pdfp[1])) {
1182 >                fclose(pdfp[0]);
1183 >                if (pdfp[1]) fclose(pdfp[1]);
1184                  Cleanup();
1185                  return RDTnone;
1186          }
1187          NewBar();                               // close up and return success
1188 <        fclose(pfp);
1189 <        if (dfp) fclose(dfp);
1188 >        fclose(pdfp[0]);
1189 >        if (pdfp[1]) fclose(pdfp[1]);
1190          return dt;
1191   }

Diff Legend

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