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

Comparing ray/src/util/rmtxop.c (file contents):
Revision 2.13 by greg, Mon Aug 12 01:20:26 2019 UTC vs.
Revision 2.32 by greg, Tue Dec 19 16:09:20 2023 UTC

# Line 5 | Line 5 | static const char RCSid[] = "$Id$";
5   * General component matrix operations.
6   */
7  
8 – #include <stdio.h>
9 – #include <stdlib.h>
8   #include <errno.h>
9   #include "rtio.h"
10   #include "resolu.h"
11   #include "rmatrix.h"
12   #include "platform.h"
13  
14 < #define MAXCOMP         16              /* #components we support */
14 > #define MAXCOMP         MAXCSAMP        /* #components we support */
15  
16 < static const char       stdin_name[] = "<stdin>";
19 <
20 < /* unary matrix operation(s) */
16 > /* Unary matrix operation(s) */
17   typedef struct {
22 –        double          sca[MAXCOMP];           /* scalar coefficients */
18          double          cmat[MAXCOMP*MAXCOMP];  /* component transformation */
19 <        short           nsf;                    /* number of scalars */
19 >        double          sca[MAXCOMP];           /* scalar coefficients */
20 >        const char      *csym;                  /* symbolic coefs or file */
21          short           clen;                   /* number of coefficients */
22 +        short           nsf;                    /* number of scalars */
23          short           transpose;              /* do transpose? */
24   } RUNARYOP;
25  
26 < /* matrix input source and requested operation(s) */
26 > /* Matrix input source and requested operation(s) */
27   typedef struct {
28          const char      *inspec;                /* input specification */
29 +        RMPref          rmp;                    /* matrix preference */
30          RUNARYOP        preop;                  /* unary operation(s) */
31          RMATRIX         *mtx;                   /* original matrix if loaded */
32          int             binop;                  /* binary op with next (or 0) */
# Line 40 | Line 38 | int    verbose = 0;                    /* verbose reporting? */
38   static int
39   loadmatrix(ROPMAT *rop)
40   {
41 <        if (rop->mtx != NULL)
41 >        if (rop->mtx != NULL)           /* already loaded? */
42                  return(0);
43  
44 <        rop->mtx = rmx_load(rop->inspec == stdin_name ?
45 <                                (const char *)NULL : rop->inspec);
46 <        if (rop->mtx == NULL) {
47 <                fputs(rop->inspec, stderr);
48 <                fputs(": cannot load matrix\n", stderr);
44 >        rop->mtx = rmx_load(rop->inspec, rop->rmp);
45 >
46 >        return(!rop->mtx ? -1 : 1);
47 > }
48 >
49 > static int      checksymbolic(ROPMAT *rop);
50 >
51 > /* Check/set transform based on a reference input file */
52 > static int
53 > checkreffile(ROPMAT *rop)
54 > {
55 >        static const char       *curRF = NULL;
56 >        static RMATRIX          refm;
57 >        const int               nc = rop->mtx->ncomp;
58 >        int                     i;
59 >
60 >        if (!curRF || strcmp(rop->preop.csym, curRF)) {
61 >                FILE    *fp = fopen(rop->preop.csym, "rb");
62 >                if (!rmx_load_header(&refm, fp)) {
63 >                        fprintf(stderr, "%s: cannot read info header\n",
64 >                                        rop->preop.csym);
65 >                        curRF = NULL;
66 >                        if (fp) fclose(fp);
67 >                        return(-1);
68 >                }
69 >                fclose(fp);
70 >                curRF = rop->preop.csym;
71 >        }
72 >        if (refm.ncomp == 3) {
73 >                rop->preop.csym = (refm.dtype == DTxyze) ? "XYZ" : "RGB";
74 >                return(checksymbolic(rop));
75 >        }
76 >        if (refm.ncomp == 2) {
77 >                fprintf(stderr, "%s: cannot convert to 2 components\n",
78 >                                curRF);
79                  return(-1);
80          }
81 <        return(1);
81 >        if (refm.ncomp == 1) {
82 >                rop->preop.csym = "Y";          /* XXX big assumption */
83 >                return(checksymbolic(rop));
84 >        }
85 >        if (refm.ncomp == nc &&
86 >                        !memcmp(refm.wlpart, rop->mtx->wlpart, sizeof(refm.wlpart)))
87 >                return(0);                      /* nothing to do */
88 >
89 >        if ((nc <= 3) | (nc > MAXCSAMP) | (refm.ncomp > MAXCSAMP)) {
90 >                fprintf(stderr, "%s: cannot resample from %d to %d components\n",
91 >                                curRF, nc, refm.ncomp);
92 >                return(-1);
93 >        }
94 >        rop->preop.clen = refm.ncomp * nc;      /* compute spec to ref */
95 >
96 >        for (i = 0; i < nc; i++) {
97 >                SCOLOR  scstim, scresp;
98 >                int     j;
99 >                memset(scstim, 0, sizeof(COLORV)*nc);
100 >                scstim[i] = 1.f;
101 >                convertscolor(scresp, refm.ncomp, refm.wlpart[0], refm.wlpart[3],
102 >                                scstim, nc, rop->mtx->wlpart[0], rop->mtx->wlpart[3]);
103 >                for (j = refm.ncomp; j-- > 0; )
104 >                        rop->preop.cmat[j*nc + i] = scresp[j];
105 >        }
106 >        memcpy(rop->mtx->wlpart, refm.wlpart, sizeof(rop->mtx->wlpart));
107 >        return(0);
108   }
109  
110 + /* Compute conversion row from spectrum to one channel of RGB */
111 + static void
112 + rgbrow(ROPMAT *rop, int r, int p)
113 + {
114 +        const int       nc = rop->mtx->ncomp;
115 +        const float *   wlp = rop->mtx->wlpart;
116 +        int             i;
117 +
118 +        for (i = nc; i--; ) {
119 +                int     nmEnd = wlp[0] + (wlp[3] - wlp[0])*i/nc;
120 +                int     nmStart = wlp[0] + (wlp[3] - wlp[0])*(i+1)/nc;
121 +                COLOR   crgb;
122 +                spec_rgb(crgb, nmStart, nmEnd);
123 +                rop->preop.cmat[r*nc+i] = crgb[p];
124 +        }
125 + }
126 +
127 + /* Compute conversion row from spectrum to one channel of XYZ */
128 + static void
129 + xyzrow(ROPMAT *rop, int r, int p)
130 + {
131 +        const int       nc = rop->mtx->ncomp;
132 +        const float *   wlp = rop->mtx->wlpart;
133 +        int             i;
134 +
135 +        for (i = nc; i--; ) {
136 +                int     nmEnd = wlp[0] + (wlp[3] - wlp[0])*i/nc;
137 +                int     nmStart = wlp[0] + (wlp[3] - wlp[0])*(i+1)/nc;
138 +                COLOR   cxyz;
139 +                spec_cie(cxyz, nmStart, nmEnd);
140 +                rop->preop.cmat[r*nc+i] = cxyz[p];
141 +        }
142 + }
143 +
144 + /* Use the spectral sensitivity function to compute matrix coefficients */
145 + static void
146 + sensrow(ROPMAT *rop, int r, double (*sf)(SCOLOR sc, int ncs, const float wlpt[4]))
147 + {
148 +        const int       nc = rop->mtx->ncomp;
149 +        int             i;
150 +
151 +        for (i = nc; i--; ) {
152 +                SCOLOR  sclr;
153 +                memset(sclr, 0, sizeof(COLORV)*nc);
154 +                sclr[i] = 1.f;
155 +                rop->preop.cmat[r*nc+i] = (*sf)(sclr, nc, rop->mtx->wlpart);
156 +        }
157 + }
158 +
159 + /* Check/set symbolic transform */
160 + static int
161 + checksymbolic(ROPMAT *rop)
162 + {
163 +        const int       nc = rop->mtx->ncomp;
164 +        const int       dt = rop->mtx->dtype;
165 +        double          cf = 1;
166 +        int             i, j;
167 +                                        /* check suffix => reference file */
168 +        if (strchr(rop->preop.csym, '.') > rop->preop.csym)
169 +                return(checkreffile(rop));
170 +
171 +        if (nc < 3) {
172 +                fprintf(stderr, "%s: -c '%s' requires at least 3 components\n",
173 +                                rop->inspec, rop->preop.csym);
174 +                return(-1);
175 +        }
176 +        rop->preop.clen = strlen(rop->preop.csym) * nc;
177 +        if (rop->preop.clen > MAXCOMP*MAXCOMP) {
178 +                fprintf(stderr, "%s: -c '%s' results in too many components\n",
179 +                                rop->inspec, rop->preop.csym);
180 +                return(-1);
181 +        }
182 +        for (j = 0; rop->preop.csym[j]; j++) {
183 +                int     comp = 0;
184 +                switch (rop->preop.csym[j]) {
185 +                case 'B':
186 +                case 'b':
187 +                        ++comp;
188 +                        /* fall through */
189 +                case 'G':
190 +                case 'g':
191 +                        ++comp;
192 +                        /* fall through */
193 +                case 'R':
194 +                case 'r':
195 +                        if (rop->preop.csym[j] <= 'Z')
196 +                                cf = 1./WHTEFFICACY;
197 +                        if (dt == DTxyze) {
198 +                                for (i = 3; i--; )
199 +                                        rop->preop.cmat[j*nc+i] = cf*xyz2rgbmat[comp][i];
200 +                        } else if (nc == 3)
201 +                                rop->preop.cmat[j*nc+comp] = 1.;
202 +                        else
203 +                                rgbrow(rop, j, comp);
204 +                        break;
205 +                case 'Z':
206 +                case 'z':
207 +                        ++comp;
208 +                        /* fall through */
209 +                case 'Y':
210 +                case 'y':
211 +                        ++comp;
212 +                        /* fall through */
213 +                case 'X':
214 +                case 'x':
215 +                        if ((rop->preop.csym[j] <= 'Z') & (dt != DTxyze))
216 +                                cf = WHTEFFICACY;
217 +                        if (dt == DTxyze) {
218 +                                rop->preop.cmat[j*nc+comp] = 1.;
219 +                        } else if (nc == 3) {
220 +                                for (i = 3; i--; )
221 +                                        rop->preop.cmat[j*nc+i] =
222 +                                                        rgb2xyzmat[comp][i];
223 +                        } else if (comp == CIEY)
224 +                                sensrow(rop, j, scolor2photopic);
225 +                        else
226 +                                xyzrow(rop, j, comp);
227 +
228 +                        for (i = nc*(cf != 1); i--; )
229 +                                rop->preop.cmat[j*nc+i] *= cf;
230 +                        break;
231 +                case 'S':               /* scotopic (il)luminance */
232 +                        cf = WHTSCOTOPIC;
233 +                        /* fall through */
234 +                case 's':
235 +                        sensrow(rop, j, scolor2scotopic);
236 +                        for (i = nc*(cf != 1); i--; )
237 +                                rop->preop.cmat[j*nc+i] *= cf;
238 +                        break;
239 +                case 'M':               /* melanopic (il)luminance */
240 +                        cf = WHTMELANOPIC;
241 +                        /* fall through */
242 +                case 'm':
243 +                        sensrow(rop, j, scolor2melanopic);
244 +                        for (i = nc*(cf != 1); i--; )
245 +                                rop->preop.cmat[j*nc+i] *= cf;
246 +                        break;
247 +                case 'A':               /* average component */
248 +                case 'a':
249 +                        for (i = nc; i--; )
250 +                                rop->preop.cmat[j*nc+i] = 1./(double)nc;
251 +                        break;
252 +                default:
253 +                        fprintf(stderr, "%s: -c '%c' unsupported\n",
254 +                                rop->inspec, rop->preop.csym[j]);
255 +                        return(-1);
256 +                }
257 +        }
258 +                                        /* return recommended output type */
259 +        if (!strcasecmp(rop->preop.csym, "XYZ")) {
260 +                if (dt <= DTspec)
261 +                        return(DTxyze);
262 +        } else if (!strcasecmp(rop->preop.csym, "RGB")) {
263 +                if (dt <= DTspec)
264 +                        return(DTrgbe);
265 +        } else if (dt == DTspec)
266 +                return(DTfloat);        /* probably not actual spectrum */
267 +        return(0);
268 + }
269 +
270   /* Get matrix and perform unary operations */
271   static RMATRIX *
272   loadop(ROPMAT *rop)
273   {
274 +        int     outtype = 0;
275          RMATRIX *mres;
276 <        int     i;
276 >        int     i, j;
277  
278          if (loadmatrix(rop) < 0)                /* make sure we're loaded */
279                  return(NULL);
280  
281 <        if (rop->preop.nsf > 0) {               /* apply scalar(s) */
282 <                if (rop->preop.clen > 0) {
283 <                        fputs("Options -s and -c are exclusive\n", stderr);
281 >        if (rop->preop.csym &&                  /* symbolic transform? */
282 >                        (outtype = checksymbolic(rop)) < 0)
283 >                goto failure;
284 >        if (rop->preop.clen > 0) {              /* apply component transform? */
285 >                if (rop->preop.clen % rop->mtx->ncomp) {
286 >                        fprintf(stderr, "%s: -c must have N x %d coefficients\n",
287 >                                        rop->inspec, rop->mtx->ncomp);
288                          goto failure;
289                  }
290 +                if (rop->preop.nsf > 0) {       /* scale transform, first */
291 +                        if (rop->preop.nsf == 1) {
292 +                                for (i = rop->preop.clen; i--; )
293 +                                        rop->preop.cmat[i] *= rop->preop.sca[0];
294 +                        } else if (rop->preop.nsf*rop->mtx->ncomp != rop->preop.clen) {
295 +                                fprintf(stderr, "%s: -s must have one or %d factors\n",
296 +                                                rop->inspec,
297 +                                                rop->preop.clen/rop->mtx->ncomp);
298 +                                goto failure;
299 +                        } else {
300 +                                for (i = rop->preop.nsf; i--; )
301 +                                        for (j = rop->mtx->ncomp; j--; )
302 +                                                rop->preop.cmat[i*rop->mtx->ncomp+j]
303 +                                                                *= rop->preop.sca[i];
304 +                        }
305 +                }
306 +                mres = rmx_transform(rop->mtx, rop->preop.clen/rop->mtx->ncomp,
307 +                                        rop->preop.cmat);
308 +                if (mres == NULL) {
309 +                        fprintf(stderr, "%s: matrix transform failed\n",
310 +                                                rop->inspec);
311 +                        goto failure;
312 +                }
313 +                if (verbose)
314 +                        fprintf(stderr, "%s: applied %d x %d transform%s\n",
315 +                                        rop->inspec, mres->ncomp,
316 +                                        rop->mtx->ncomp,
317 +                                        rop->preop.nsf ? " (* scalar)" : "");
318 +                rop->preop.nsf = 0;             /* now folded in */
319 +                if ((mres->ncomp > 3) & (mres->dtype <= DTspec))
320 +                        outtype = DTfloat;      /* probably not actual spectrum */
321 +                rmx_free(rop->mtx);
322 +                rop->mtx = mres;
323 +        }
324 +        if (rop->preop.nsf > 0) {               /* apply scalar(s)? */
325                  if (rop->preop.nsf == 1) {
326                          for (i = rop->mtx->ncomp; --i; )
327                                  rop->preop.sca[i] = rop->preop.sca[0];
# Line 89 | Line 343 | loadop(ROPMAT *rop)
343                          fputs(" )\n", stderr);
344                  }
345          }
346 <        if (rop->preop.clen > 0) {      /* apply transform */
93 <                if (rop->preop.clen % rop->mtx->ncomp) {
94 <                        fprintf(stderr, "%s: -c must have N x %d coefficients\n",
95 <                                        rop->inspec, rop->mtx->ncomp);
96 <                        goto failure;
97 <                }
98 <                mres = rmx_transform(rop->mtx, rop->preop.clen/rop->mtx->ncomp,
99 <                                        rop->preop.cmat);
100 <                if (mres == NULL) {
101 <                        fprintf(stderr, "%s: matrix transform failed\n",
102 <                                                rop->inspec);
103 <                        goto failure;
104 <                }
105 <                if (verbose)
106 <                        fprintf(stderr, "%s: applied %d x %d transform\n",
107 <                                        rop->inspec, mres->ncomp,
108 <                                        rop->mtx->ncomp);
109 <                rmx_free(rop->mtx);
110 <                rop->mtx = mres;
111 <        }
112 <        if (rop->preop.transpose) {     /* transpose matrix? */
346 >        if (rop->preop.transpose) {             /* transpose matrix? */
347                  mres = rmx_transpose(rop->mtx);
348                  if (mres == NULL) {
349                          fputs(rop->inspec, stderr);
# Line 125 | Line 359 | loadop(ROPMAT *rop)
359          }
360          mres = rop->mtx;
361          rop->mtx = NULL;
362 +        if (outtype)
363 +                mres->dtype = outtype;
364          return(mres);
365   failure:
366          rmx_free(rop->mtx);
# Line 140 | Line 376 | binaryop(const char *inspec, RMATRIX *mleft, int op, R
376  
377          if ((mleft == NULL) | (mright == NULL))
378                  return(NULL);
143 –
379          switch (op) {
380          case '.':                       /* concatenate */
381 <                mres = rmx_multiply(mleft, mright);
381 >                if (mleft->ncomp != mright->ncomp) {
382 >                        fputs(inspec, stderr);
383 >                        fputs(": # components do not match\n", stderr);
384 >                } else if (mleft->ncols != mright->nrows) {
385 >                        fputs(inspec, stderr);
386 >                        fputs(": mismatched dimensions\n",
387 >                                        stderr);
388 >                } else
389 >                        mres = rmx_multiply(mleft, mright);
390                  rmx_free(mleft);
391                  rmx_free(mright);
392                  if (mres == NULL) {
393                          fputs(inspec, stderr);
394 <                        if (mleft->ncols != mright->nrows)
152 <                                fputs(": mismatched dimensions for multiply\n",
153 <                                                stderr);
154 <                        else
155 <                                fputs(": concatenation failed\n", stderr);
394 >                        fputs(": concatenation failed\n", stderr);
395                          return(NULL);
396                  }
397                  if (verbose) {
# Line 229 | Line 468 | op_right2left(ROPMAT *mop)
468          int     rpos = 0;
469                                          /* find end of list */
470          while (mop[rpos].binop)
471 <                rpos++;
471 >                if (mop[rpos++].binop != '.') {
472 >                        fputs(
473 >                "Right-to-left evaluation only for matrix multiplication!\n",
474 >                                        stderr);
475 >                        return(NULL);
476 >                }
477          mright = loadop(mop+rpos);
478          while (rpos-- > 0) {
479                  if (mright == NULL)
480                          break;
481 <                mright = binaryop(mop[rpos].inspec,
481 >                mright = binaryop(mop[rpos+1].inspec,
482                                  loadop(mop+rpos), mop[rpos].binop, mright);
483          }
484          return(mright);
# Line 284 | Line 528 | get_factors(double da[], int n, char *av[])
528   }
529  
530   static ROPMAT *
531 < grow_moparray(ROPMAT *mop, int n2alloc)
531 > resize_moparr(ROPMAT *mop, int n2alloc)
532   {
533          int     nmats = 0;
534 +        int     i;
535  
536          while (mop[nmats++].binop)
537                  ;
538 +        for (i = nmats; i > n2alloc; i--)
539 +                rmx_free(mop[i].mtx);
540          mop = (ROPMAT *)realloc(mop, n2alloc*sizeof(ROPMAT));
541          if (mop == NULL) {
542 <                fputs("Out of memory in grow_moparray()\n", stderr);
542 >                fputs("Out of memory in resize_moparr()\n", stderr);
543                  exit(1);
544          }
545          if (n2alloc > nmats)
# Line 304 | Line 551 | grow_moparray(ROPMAT *mop, int n2alloc)
551   int
552   main(int argc, char *argv[])
553   {
554 <        int     outfmt = DTfromHeader;
555 <        int     nall = 2;
556 <        ROPMAT  *mop = (ROPMAT *)calloc(nall, sizeof(ROPMAT));
557 <        int     nmats = 0;
558 <        RMATRIX *mres = NULL;
559 <        int     stdin_used = 0;
560 <        int     i;
554 >        int             outfmt = DTfromHeader;
555 >        const char      *defCsym = NULL;
556 >        int             nall = 2;
557 >        ROPMAT          *mop = (ROPMAT *)calloc(nall, sizeof(ROPMAT));
558 >        int             nmats = 0;
559 >        RMATRIX         *mres = NULL;
560 >        int             stdin_used = 0;
561 >        int             i;
562                                          /* get options and arguments */
563          for (i = 1; i < argc; i++) {
316 –                if (nmats >= nall)
317 –                        mop = grow_moparray(mop, nall += 2);
564                  if (argv[i][0] && !argv[i][1] &&
565                                  strchr(".+*/", argv[i][0]) != NULL) {
566 <                        if (mop[nmats].inspec == NULL || mop[nmats].binop) {
567 <                                fprintf(stderr, "%s: missing matrix argument\n",
568 <                                                argv[0]);
566 >                        if (!nmats || mop[nmats-1].binop) {
567 >                                fprintf(stderr,
568 >                        "%s: missing matrix argument before '%c' operation\n",
569 >                                                argv[0], argv[i][0]);
570                                  return(1);
571                          }
572 <                        mop[nmats++].binop = argv[i][0];
572 >                        mop[nmats-1].binop = argv[i][0];
573                  } else if (argv[i][0] != '-' || !argv[i][1]) {
574                          if (argv[i][0] == '-') {
575                                  if (stdin_used++) {
# Line 334 | Line 581 | main(int argc, char *argv[])
581                                  mop[nmats].inspec = stdin_name;
582                          } else
583                                  mop[nmats].inspec = argv[i];
584 +                        if (!mop[nmats].preop.csym)
585 +                                mop[nmats].preop.csym = defCsym;
586                          if (nmats > 0 && !mop[nmats-1].binop)
587                                  mop[nmats-1].binop = '.';
588                          nmats++;
# Line 341 | Line 590 | main(int argc, char *argv[])
590                          int     n = argc-1 - i;
591                          switch (argv[i][1]) {   /* get option */
592                          case 'v':
593 <                                verbose = !verbose;
593 >                                verbose++;
594                                  break;
595                          case 'f':
596                                  switch (argv[i][2]) {
# Line 369 | Line 618 | main(int argc, char *argv[])
618                                  i += mop[nmats].preop.nsf =
619                                          get_factors(mop[nmats].preop.sca,
620                                                          n, argv+i+1);
621 +                                if (mop[nmats].preop.nsf <= 0) {
622 +                                        fprintf(stderr, "%s: -s missing arguments\n",
623 +                                                        argv[0]);
624 +                                        goto userr;
625 +                                }
626                                  break;
627 +                        case 'C':
628 +                                if (!n || isflt(argv[i+1]))
629 +                                        goto userr;
630 +                                defCsym = mop[nmats].preop.csym = argv[++i];
631 +                                mop[nmats].preop.clen = 0;
632 +                                break;
633                          case 'c':
634 +                                if (n && !isflt(argv[i+1])) {
635 +                                        mop[nmats].preop.csym = argv[++i];
636 +                                        mop[nmats].preop.clen = 0;
637 +                                        break;
638 +                                }
639                                  if (n > MAXCOMP*MAXCOMP) n = MAXCOMP*MAXCOMP;
640                                  i += mop[nmats].preop.clen =
641                                          get_factors(mop[nmats].preop.cmat,
642                                                          n, argv+i+1);
643 +                                if (mop[nmats].preop.clen <= 0) {
644 +                                        fprintf(stderr, "%s: -c missing arguments\n",
645 +                                                        argv[0]);
646 +                                        goto userr;
647 +                                }
648 +                                mop[nmats].preop.csym = NULL;
649                                  break;
650 +                        case 'r':
651 +                                if (argv[i][2] == 'f')
652 +                                        mop[nmats].rmp = RMPreflF;
653 +                                else if (argv[i][2] == 'b')
654 +                                        mop[nmats].rmp = RMPreflB;
655 +                                else
656 +                                        goto userr;
657 +                                break;
658                          default:
659                                  fprintf(stderr, "%s: unknown operation '%s'\n",
660                                                  argv[0], argv[i]);
661                                  goto userr;
662                          }
663                  }
664 +                if (nmats >= nall)
665 +                        mop = resize_moparr(mop, nall += 2);
666          }
667          if (mop[0].inspec == NULL)      /* nothing to do? */
668                  goto userr;
669 +        if (mop[nmats-1].binop) {
670 +                fprintf(stderr,
671 +                        "%s: missing matrix argument after '%c' operation\n",
672 +                                argv[0], mop[nmats-1].binop);
673 +                return(1);
674 +        }
675                                          /* favor quicker concatenation */
676 <        mres = prefer_right2left(mop) ? op_right2left(mop) : op_left2right(mop);
677 <        if (!mres)
676 >        mop[nmats].mtx = prefer_right2left(mop) ? op_right2left(mop)
677 >                                                : op_left2right(mop);
678 >        if (mop[nmats].mtx == NULL)
679                  return(1);
680 <                                        /* write result to stdout */
681 <        if (outfmt == DTfromHeader)
682 <                outfmt = mres->dtype;
683 <        if (outfmt != DTascii)
396 <                SET_FILE_BINARY(stdout);
397 <        newheader("RADIANCE", stdout);
398 <        printargs(argc, argv, stdout);
399 <        if (!rmx_write(mres, outfmt, stdout)) {
400 <                fprintf(stderr, "%s: error writing result matrix\n", argv[0]);
680 >                                        /* apply trailing unary operations */
681 >        mop[nmats].inspec = "trailing_ops";
682 >        mres = loadop(mop+nmats);
683 >        if (mres == NULL)
684                  return(1);
685 +        if (outfmt == DTfromHeader)     /* check data type */
686 +                outfmt = mres->dtype;
687 +        if (outfmt == DTrgbe) {
688 +                if (mres->ncomp > 3)
689 +                        outfmt = DTspec;
690 +                else if (mres->dtype == DTxyze)
691 +                        outfmt = DTxyze;
692          }
693 <        /* rmx_free(mres); free(mop); */
694 <        return(0);
693 >        newheader("RADIANCE", stdout);  /* write result to stdout */
694 >        printargs(argc, argv, stdout);
695 >        return(rmx_write(mres, outfmt, stdout) ? 0 : 1);
696   userr:
697          fprintf(stderr,
698 <        "Usage: %s [-v][-f[adfc][-t][-s sf .. | -c ce ..] m1 [.+*/] .. > mres\n",
698 >        "Usage: %s [-v][-f{adfc}][-t][-s sf .. | -c ce ..][-rf|-rb] m1 [.+*/] .. > mres\n",
699                          argv[0]);
700          return(1);
701   }

Diff Legend

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