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

Comparing ray/src/util/rmatrix.c (file contents):
Revision 2.25 by greg, Mon Aug 28 15:59:46 2017 UTC vs.
Revision 2.75 by greg, Mon Dec 11 19:00:22 2023 UTC

# Line 5 | Line 5 | static const char RCSid[] = "$Id$";
5   * General matrix operations.
6   */
7  
8 – #include <stdio.h>
8   #include <stdlib.h>
10 – #include <string.h>
11 – #include <fcntl.h>
9   #include <errno.h>
10   #include "rtio.h"
11   #include "platform.h"
12   #include "resolu.h"
13   #include "paths.h"
14   #include "rmatrix.h"
15 + #if !defined(_WIN32) && !defined(_WIN64)
16 + #include <sys/mman.h>
17 + #endif
18  
19 < static char     rmx_mismatch_warn[] = "WARNING: data type mismatch\n";
19 > static const char       rmx_mismatch_warn[] = "WARNING: data type mismatch\n";
20  
21 < /* Allocate a nr x nc matrix with n components */
21 > #define array_size(rm)  (sizeof(double)*(rm)->nrows*(rm)->ncols*(rm)->ncomp)
22 > #define mapped_size(rm) ((char *)(rm)->mtx + array_size(rm) - (char *)(rm)->mapped)
23 >
24 > /* Initialize a RMATRIX struct but don't allocate array space */
25   RMATRIX *
26 < rmx_alloc(int nr, int nc, int n)
26 > rmx_new(int nr, int nc, int n)
27   {
28          RMATRIX *dnew;
29  
30 <        if ((nr <= 0) | (nc <= 0) | (n <= 0))
30 >        if (n <= 0)
31                  return(NULL);
32 <        dnew = (RMATRIX *)malloc(sizeof(RMATRIX)-sizeof(dnew->mtx) +
33 <                                        sizeof(dnew->mtx[0])*(n*nr*nc));
34 <        if (dnew == NULL)
32 >
33 >        dnew = (RMATRIX *)calloc(1, sizeof(RMATRIX));
34 >        if (!dnew)
35                  return(NULL);
36 <        dnew->nrows = nr; dnew->ncols = nc; dnew->ncomp = n;
36 >
37          dnew->dtype = DTdouble;
38 <        dnew->info = NULL;
38 >        dnew->nrows = nr;
39 >        dnew->ncols = nc;
40 >        dnew->ncomp = n;
41 >        setcolor(dnew->cexp, 1.f, 1.f, 1.f);
42 >        memcpy(dnew->wlpart, WLPART, sizeof(dnew->wlpart));
43 >
44          return(dnew);
45   }
46  
47 < /* Free a RMATRIX array */
47 > /* Prepare a RMATRIX for writing (allocate array if needed) */
48 > int
49 > rmx_prepare(RMATRIX *rm)
50 > {
51 >        if (!rm) return(0);
52 >        if (rm->mtx)
53 >                return(1);
54 >        if ((rm->nrows <= 0) | (rm->ncols <= 0) | (rm->ncomp <= 0))
55 >                return(0);
56 >        rm->mtx = (double *)malloc(array_size(rm));
57 >        return(rm->mtx != NULL);
58 > }
59 >
60 > /* Call rmx_new() and rmx_prepare() */
61 > RMATRIX *
62 > rmx_alloc(int nr, int nc, int n)
63 > {
64 >        RMATRIX *dnew = rmx_new(nr, nc, n);
65 >
66 >        if (dnew && !rmx_prepare(dnew)) {
67 >                rmx_free(dnew);
68 >                dnew = NULL;
69 >        }
70 >        return(dnew);
71 > }
72 >
73 > /* Clear state by freeing info and matrix data */
74   void
75 < rmx_free(RMATRIX *rm)
75 > rmx_reset(RMATRIX *rm)
76   {
77          if (!rm) return;
78 <        if (rm->info)
78 >        if (rm->info) {
79                  free(rm->info);
80 +                rm->info = NULL;
81 +        }
82 +        if (rm->mtx) {
83 + #ifdef MAP_FILE
84 +                if (rm->mapped) {
85 +                        munmap(rm->mapped, mapped_size(rm));
86 +                        rm->mapped = NULL;
87 +                } else
88 + #endif
89 +                        free(rm->mtx);
90 +                rm->mtx = NULL;
91 +        }
92 + }
93 +
94 + /* Free an RMATRIX struct and data */
95 + void
96 + rmx_free(RMATRIX *rm)
97 + {
98 +        if (!rm) return;
99 +        rmx_reset(rm);
100          free(rm);
101   }
102  
# Line 50 | Line 104 | rmx_free(RMATRIX *rm)
104   int
105   rmx_newtype(int dtyp1, int dtyp2)
106   {
107 <        if ((dtyp1==DTxyze) | (dtyp1==DTrgbe) |
108 <                        (dtyp2==DTxyze) | (dtyp2==DTrgbe)
107 >        if ((dtyp1==DTxyze) | (dtyp1==DTrgbe) | (dtyp1==DTspec) |
108 >                        (dtyp2==DTxyze) | (dtyp2==DTrgbe) | (dtyp2==DTspec)
109                          && dtyp1 != dtyp2)
110                  return(0);
111          if (dtyp1 < dtyp2)
# Line 63 | Line 117 | rmx_newtype(int dtyp1, int dtyp2)
117   int
118   rmx_addinfo(RMATRIX *rm, const char *info)
119   {
120 <        if (!info || !*info)
120 >        int     oldlen = 0;
121 >
122 >        if (!rm || !info || !*info)
123                  return(0);
124          if (!rm->info) {
125                  rm->info = (char *)malloc(strlen(info)+1);
126                  if (rm->info) rm->info[0] = '\0';
127 <        } else
127 >        } else {
128 >                oldlen = strlen(rm->info);
129                  rm->info = (char *)realloc(rm->info,
130 <                                strlen(rm->info)+strlen(info)+1);
130 >                                oldlen+strlen(info)+1);
131 >        }
132          if (!rm->info)
133                  return(0);
134 <        strcat(rm->info, info);
134 >        strcpy(rm->info+oldlen, info);
135          return(1);
136   }
137  
# Line 81 | Line 139 | static int
139   get_dminfo(char *s, void *p)
140   {
141          RMATRIX *ip = (RMATRIX *)p;
142 <        char    fmt[64];
142 >        char    fmt[MAXFMTLEN];
143          int     i;
144  
145 <        if (headidval(fmt, s))
145 >        if (headidval(NULL, s))
146                  return(0);
147 <        if (!strncmp(s, "NCOMP=", 6)) {
148 <                ip->ncomp = atoi(s+6);
147 >        if (isncomp(s)) {
148 >                ip->ncomp = ncompval(s);
149                  return(0);
150          }
151          if (!strncmp(s, "NROWS=", 6)) {
# Line 98 | Line 156 | get_dminfo(char *s, void *p)
156                  ip->ncols = atoi(s+6);
157                  return(0);
158          }
159 +        if ((i = isbigendian(s)) >= 0) {
160 +                ip->swapin = (nativebigendian() != i);
161 +                return(0);
162 +        }
163 +        if (isexpos(s)) {
164 +                float   f = exposval(s);
165 +                scalecolor(ip->cexp, f);
166 +                return(0);
167 +        }
168 +        if (iscolcor(s)) {
169 +                COLOR   ctmp;
170 +                colcorval(ctmp, s);
171 +                multcolor(ip->cexp, ctmp);
172 +                return(0);
173 +        }
174 +        if (iswlsplit(s)) {
175 +                wlsplitval(ip->wlpart, s);
176 +                return(0);
177 +        }
178          if (!formatval(fmt, s)) {
179                  rmx_addinfo(ip, s);
180                  return(0);
181 <        }
181 >        }                       /* else check format */
182          for (i = 1; i < DTend; i++)
183                  if (!strcmp(fmt, cm_fmt_id[i])) {
184                          ip->dtype = i;
# Line 111 | Line 188 | get_dminfo(char *s, void *p)
188   }
189  
190   static int
191 < rmx_load_ascii(RMATRIX *rm, FILE *fp)
191 > rmx_load_ascii(double *drp, const RMATRIX *rm, FILE *fp)
192   {
193 <        int     i, j, k;
193 >        int     j, k;
194  
195 <        for (i = 0; i < rm->nrows; i++)
196 <            for (j = 0; j < rm->ncols; j++)
197 <                for (k = 0; k < rm->ncomp; k++)
198 <                    if (fscanf(fp, "%lf", &rmx_lval(rm,i,j,k)) != 1)
122 <                        return(0);
195 >        for (j = 0; j < rm->ncols; j++)
196 >                for (k = rm->ncomp; k-- > 0; )
197 >                        if (fscanf(fp, "%lf", drp++) != 1)
198 >                                return(0);
199          return(1);
200   }
201  
202   static int
203 < rmx_load_float(RMATRIX *rm, FILE *fp)
203 > rmx_load_float(double *drp, const RMATRIX *rm, FILE *fp)
204   {
205 <        int     i, j, k;
205 >        int     j, k;
206          float   val[100];
207  
208          if (rm->ncomp > 100) {
209                  fputs("Unsupported # components in rmx_load_float()\n", stderr);
210                  exit(1);
211          }
212 <        for (i = 0; i < rm->nrows; i++)
137 <            for (j = 0; j < rm->ncols; j++) {
212 >        for (j = 0; j < rm->ncols; j++) {
213                  if (getbinary(val, sizeof(val[0]), rm->ncomp, fp) != rm->ncomp)
214 <                    return(0);
215 <                for (k = rm->ncomp; k--; )
216 <                     rmx_lval(rm,i,j,k) = val[k];
217 <            }
214 >                        return(0);
215 >                if (rm->swapin)
216 >                        swap32((char *)val, rm->ncomp);
217 >                for (k = 0; k < rm->ncomp; k++)
218 >                        *drp++ = val[k];
219 >        }
220          return(1);
221   }
222  
223   static int
224 < rmx_load_double(RMATRIX *rm, FILE *fp)
224 > rmx_load_double(double *drp, const RMATRIX *rm, FILE *fp)
225   {
226 <        int     i, j, k;
227 <        double  val[100];
226 >        if (getbinary(drp, sizeof(*drp)*rm->ncomp, rm->ncols, fp) != rm->ncols)
227 >                return(0);
228 >        if (rm->swapin)
229 >                swap64((char *)drp, rm->ncols*rm->ncomp);
230 >        return(1);
231 > }
232  
233 <        if (rm->ncomp > 100) {
234 <                fputs("Unsupported # components in rmx_load_double()\n", stderr);
235 <                exit(1);
233 > static int
234 > rmx_load_rgbe(double *drp, const RMATRIX *rm, FILE *fp)
235 > {
236 >        COLR    *scan;
237 >        COLOR   col;
238 >        int     j;
239 >
240 >        if (rm->ncomp != 3)
241 >                return(0);
242 >        scan = (COLR *)tempbuffer(sizeof(COLR)*rm->ncols);
243 >        if (!scan)
244 >                return(0);
245 >        if (freadcolrs(scan, rm->ncols, fp) < 0)
246 >                return(0);
247 >        for (j = 0; j < rm->ncols; j++) {
248 >                colr_color(col, scan[j]);
249 >                *drp++ = colval(col,RED);
250 >                *drp++ = colval(col,GRN);
251 >                *drp++ = colval(col,BLU);
252          }
156 –        for (i = 0; i < rm->nrows; i++)
157 –            for (j = 0; j < rm->ncols; j++) {
158 –                if (getbinary(val, sizeof(val[0]), rm->ncomp, fp) != rm->ncomp)
159 –                    return(0);
160 –                for (k = rm->ncomp; k--; )
161 –                     rmx_lval(rm,i,j,k) = val[k];
162 –            }
253          return(1);
254   }
255  
256   static int
257 < rmx_load_rgbe(RMATRIX *rm, FILE *fp)
257 > rmx_load_spec(double *drp, const RMATRIX *rm, FILE *fp)
258   {
259 <        COLOR   *scan = (COLOR *)malloc(sizeof(COLOR)*rm->ncols);
260 <        int     i, j;
259 >        uby8    *scan;
260 >        SCOLOR  scol;
261 >        int     j, k;
262  
263 <        if (scan == NULL)
263 >        if ((rm->ncomp < 3) | (rm->ncomp > MAXCSAMP))
264                  return(0);
265 <        for (i = 0; i < rm->nrows; i++) {
266 <            if (freadscan(scan, rm->ncols, fp) < 0) {
176 <                free(scan);
265 >        scan = (uby8 *)tempbuffer((rm->ncomp+1)*rm->ncols);
266 >        if (!scan)
267                  return(0);
268 <            }
269 <            for (j = rm->ncols; j--; ) {
270 <                rmx_lval(rm,i,j,0) = colval(scan[j],RED);
271 <                rmx_lval(rm,i,j,1) = colval(scan[j],GRN);
272 <                rmx_lval(rm,i,j,2) = colval(scan[j],BLU);
273 <            }
268 >        if (freadscolrs(scan, rm->ncomp, rm->ncols, fp) < 0)
269 >                return(0);
270 >        for (j = 0; j < rm->ncols; j++) {
271 >                scolr2scolor(scol, scan+j*(rm->ncomp+1), rm->ncomp);
272 >                for (k = 0; k < rm->ncomp; k++)
273 >                        *drp++ = scol[k];
274          }
185 –        free(scan);
275          return(1);
276   }
277  
278 + /* Read matrix header from input stream (cannot be XML) */
279 + int
280 + rmx_load_header(RMATRIX *rm, FILE *fp)
281 + {
282 +        if (!rm | !fp)
283 +                return(0);
284 +        rmx_reset(rm);                          /* clear state */
285 +        if (rm->nrows | rm->ncols | !rm->dtype) {
286 +                rm->nrows = rm->ncols = 0;
287 +                rm->ncomp = 3;
288 +                setcolor(rm->cexp, 1.f, 1.f, 1.f);
289 +                memcpy(rm->wlpart, WLPART, sizeof(rm->wlpart));
290 +                rm->swapin = 0;
291 +        }
292 +        rm->dtype = DTascii;                    /* assumed w/o FORMAT */
293 +        if (getheader(fp, get_dminfo, rm) < 0) {
294 +                fputs("Unrecognized matrix format\n", stderr);
295 +                return(0);
296 +        }
297 +        if ((rm->dtype == DTrgbe) | (rm->dtype == DTxyze) &&
298 +                        rm->ncomp != 3)
299 +                return(0);
300 +        if (rm->ncols <= 0 &&                   /* resolution string? */
301 +                        !fscnresolu(&rm->ncols, &rm->nrows, fp))
302 +                return(0);
303 +        if (rm->dtype == DTascii)               /* set file type (WINDOWS) */
304 +                SET_FILE_TEXT(fp);
305 +        else
306 +                SET_FILE_BINARY(fp);
307 +        return(1);
308 + }
309 +
310 + /* Load next row as double (cannot be XML) */
311 + int
312 + rmx_load_row(double *drp, const RMATRIX *rm, FILE *fp)
313 + {
314 +        switch (rm->dtype) {
315 +        case DTascii:
316 +                return(rmx_load_ascii(drp, rm, fp));
317 +        case DTfloat:
318 +                return(rmx_load_float(drp, rm, fp));
319 +        case DTdouble:
320 +                return(rmx_load_double(drp, rm, fp));
321 +        case DTrgbe:
322 +        case DTxyze:
323 +                return(rmx_load_rgbe(drp, rm, fp));
324 +        case DTspec:
325 +                return(rmx_load_spec(drp, rm, fp));
326 +        default:
327 +                fputs("Unsupported data type in rmx_load_row()\n", stderr);
328 +        }
329 +        return(0);
330 + }
331 +
332 + /* Allocate & load post-header data from stream given type set in rm->dtype */
333 + int
334 + rmx_load_data(RMATRIX *rm, FILE *fp)
335 + {
336 +        int     i;
337 + #ifdef MAP_FILE
338 +        long    pos;            /* map memory for file > 1MB if possible */
339 +        if ((rm->dtype == DTdouble) & !rm->swapin && array_size(rm) >= 1L<<20 &&
340 +                        (pos = ftell(fp)) >= 0 && !(pos % sizeof(double))) {
341 +                rm->mapped = mmap(NULL, array_size(rm)+pos, PROT_READ|PROT_WRITE,
342 +                                        MAP_PRIVATE, fileno(fp), 0);
343 +                if (rm->mapped != MAP_FAILED) {
344 +                        rm->mtx = (double *)rm->mapped + pos/sizeof(double);
345 +                        return(1);
346 +                }               /* else fall back on reading into memory */
347 +                rm->mapped = NULL;
348 +        }
349 + #endif
350 +        if (!rmx_prepare(rm)) { /* need in-core matrix array */
351 +                fprintf(stderr, "Cannot allocate %g MByte matrix array\n",
352 +                                (1./(1L<<20))*(double)array_size(rm));
353 +                return(0);
354 +        }
355 +        for (i = 0; i < rm->nrows; i++)
356 +                if (!rmx_load_row(rmx_lval(rm,i,0), rm, fp))
357 +                        return(0);
358 +        return(1);
359 + }
360 +
361   /* Load matrix from supported file type */
362   RMATRIX *
363 < rmx_load(const char *inspec)
363 > rmx_load(const char *inspec, RMPref rmp)
364   {
365 <        FILE            *fp = stdin;
194 <        RMATRIX         dinfo;
365 >        FILE            *fp;
366          RMATRIX         *dnew;
367 +        int             ok;
368  
369 <        if (inspec == NULL) {                   /* reading from stdin? */
370 <                inspec = "<stdin>";
371 <                SET_FILE_BINARY(stdin);
372 <        } else if (inspec[0] == '!') {
373 <                if ((fp = popen(inspec+1, "r")) == NULL)
374 <                        return(NULL);
375 <                SET_FILE_BINARY(stdin);
376 <        } else {
369 >        if (!inspec)
370 >                inspec = stdin_name;
371 >        else if (!*inspec)
372 >                return(NULL);
373 >        if (inspec == stdin_name)               /* reading from stdin? */
374 >                fp = stdin;
375 >        else if (inspec[0] == '!')
376 >                fp = popen(inspec+1, "r");
377 >        else {
378                  const char      *sp = inspec;   /* check suffix */
379                  while (*sp)
380                          ++sp;
381                  while (sp > inspec && sp[-1] != '.')
382                          --sp;
383                  if (!strcasecmp(sp, "XML")) {   /* assume it's a BSDF */
384 <                        CMATRIX *cm = cm_loadBTDF((char *)inspec);
385 <                        if (cm == NULL)
384 >                        CMATRIX *cm = rmp==RMPnone ? (CMATRIX *)NULL :
385 >                                        rmp==RMPtrans ? cm_loadBTDF(inspec) :
386 >                                        cm_loadBRDF(inspec, rmp==RMPreflB) ;
387 >                        if (!cm)
388                                  return(NULL);
389                          dnew = rmx_from_cmatrix(cm);
390                          cm_free(cm);
391                          dnew->dtype = DTascii;
392 <                        return(dnew);
393 <                }
394 <                                                /* else open it ourselves */
220 <                if ((fp = fopen(inspec, "rb")) == NULL)
221 <                        return(NULL);
392 >                        return(dnew);           /* return here */
393 >                }                               /* else open it ourselves */
394 >                fp = fopen(inspec, "r");
395          }
396 +        if (!fp)
397 +                return(NULL);
398   #ifdef getc_unlocked
399          flockfile(fp);
400   #endif
401 <        dinfo.nrows = dinfo.ncols = dinfo.ncomp = 0;
402 <        dinfo.dtype = DTascii;                  /* assumed w/o FORMAT */
403 <        dinfo.info = NULL;
404 <        if (getheader(fp, get_dminfo, &dinfo) < 0) {
405 <                fclose(fp);
401 >        SET_FILE_BINARY(fp);                    /* load header info */
402 >        if (!rmx_load_header(dnew = rmx_new(0,0,3), fp)) {
403 >                fprintf(stderr, "Bad header in: %s\n", inspec);
404 >                if (inspec[0] == '!') pclose(fp);
405 >                else fclose(fp);
406 >                rmx_free(dnew);
407                  return(NULL);
408          }
409 <        if ((dinfo.nrows <= 0) | (dinfo.ncols <= 0)) {
410 <                if (!fscnresolu(&dinfo.ncols, &dinfo.nrows, fp)) {
411 <                        fclose(fp);
236 <                        return(NULL);
237 <                }
238 <                if (dinfo.ncomp <= 0)
239 <                        dinfo.ncomp = 3;
240 <                else if ((dinfo.dtype == DTrgbe) | (dinfo.dtype == DTxyze) &&
241 <                                dinfo.ncomp != 3) {
242 <                        fclose(fp);
243 <                        return(NULL);
244 <                }
245 <        }
246 <        dnew = rmx_alloc(dinfo.nrows, dinfo.ncols, dinfo.ncomp);
247 <        if (dnew == NULL) {
248 <                fclose(fp);
249 <                return(NULL);
250 <        }
251 <        dnew->info = dinfo.info;
252 <        switch (dinfo.dtype) {
253 <        case DTascii:
254 <                SET_FILE_TEXT(stdin);
255 <                if (!rmx_load_ascii(dnew, fp))
256 <                        goto loaderr;
257 <                dnew->dtype = DTascii;          /* should leave double? */
258 <                break;
259 <        case DTfloat:
260 <                if (!rmx_load_float(dnew, fp))
261 <                        goto loaderr;
262 <                dnew->dtype = DTfloat;
263 <                break;
264 <        case DTdouble:
265 <                if (!rmx_load_double(dnew, fp))
266 <                        goto loaderr;
267 <                dnew->dtype = DTdouble;
268 <                break;
269 <        case DTrgbe:
270 <        case DTxyze:
271 <                if (!rmx_load_rgbe(dnew, fp))
272 <                        goto loaderr;
273 <                dnew->dtype = dinfo.dtype;
274 <                break;
275 <        default:
276 <                goto loaderr;
277 <        }
278 <        if (fp != stdin) {
409 >        ok = rmx_load_data(dnew, fp);           /* allocate & load data */
410 >
411 >        if (fp != stdin) {                      /* close input stream */
412                  if (inspec[0] == '!')
413                          pclose(fp);
414                  else
# Line 285 | Line 418 | rmx_load(const char *inspec)
418          else
419                  funlockfile(fp);
420   #endif
421 +        if (!ok) {                              /* load failure? */
422 +                fprintf(stderr, "Error loading data from: %s\n", inspec);
423 +                rmx_free(dnew);
424 +                return(NULL);
425 +        }
426 +                                                /* undo exposure? */
427 +        if ((dnew->cexp[0] != 1.f) |
428 +                        (dnew->cexp[1] != 1.f) | (dnew->cexp[2] != 1.f)) {
429 +                double  cmlt[MAXCSAMP];
430 +                int     i;
431 +                cmlt[0] = 1./dnew->cexp[0];
432 +                cmlt[1] = 1./dnew->cexp[1];
433 +                cmlt[2] = 1./dnew->cexp[2];
434 +                if (dnew->ncomp > MAXCSAMP) {
435 +                        fprintf(stderr, "Excess spectral components in: %s\n",
436 +                                        inspec);
437 +                        rmx_free(dnew);
438 +                        return(NULL);
439 +                }
440 +                for (i = dnew->ncomp; i-- > 3; )
441 +                        cmlt[i] = cmlt[1];
442 +                rmx_scale(dnew, cmlt);
443 +                setcolor(dnew->cexp, 1.f, 1.f, 1.f);
444 +        }
445          return(dnew);
289 – loaderr:                                        /* should report error? */
290 –        if (inspec[0] == '!')
291 –                pclose(fp);
292 –        else
293 –                fclose(fp);
294 –        rmx_free(dnew);
295 –        return(NULL);
446   }
447  
448   static int
449 < rmx_write_ascii(const RMATRIX *rm, FILE *fp)
449 > rmx_write_ascii(const double *dp, int nc, int len, FILE *fp)
450   {
451 <        int     i, j, k;
452 <
453 <        for (i = 0; i < rm->nrows; i++) {
454 <            for (j = 0; j < rm->ncols; j++) {
305 <                for (k = 0; k < rm->ncomp; k++)
306 <                    fprintf(fp, " %.15e", rmx_lval(rm,i,j,k));
451 >        while (len-- > 0) {
452 >                int     k = nc;
453 >                while (k-- > 0)
454 >                        fprintf(fp, " %.7e", *dp++);
455                  fputc('\t', fp);
308 –            }
309 –            fputc('\n', fp);
456          }
457 <        return(1);
457 >        return(fputc('\n', fp) != EOF);
458   }
459  
460   static int
461 < rmx_write_float(const RMATRIX *rm, FILE *fp)
461 > rmx_write_float(const double *dp, int len, FILE *fp)
462   {
463 <        int     i, j, k;
318 <        float   val[100];
463 >        float   val;
464  
465 <        if (rm->ncomp > 100) {
466 <                fputs("Unsupported # components in rmx_write_float()\n", stderr);
467 <                exit(1);
323 <        }
324 <        for (i = 0; i < rm->nrows; i++)
325 <            for (j = 0; j < rm->ncols; j++) {
326 <                for (k = rm->ncomp; k--; )
327 <                    val[k] = (float)rmx_lval(rm,i,j,k);
328 <                if (putbinary(val, sizeof(val[0]), rm->ncomp, fp) != rm->ncomp)
465 >        while (len--) {
466 >                val = *dp++;
467 >                if (putbinary(&val, sizeof(float), 1, fp) != 1)
468                          return(0);
469 <            }
469 >        }
470          return(1);
471   }
472  
473   static int
474 < rmx_write_double(const RMATRIX *rm, FILE *fp)
474 > rmx_write_rgbe(const double *dp, int nc, int len, FILE *fp)
475   {
476 <        int     i, j, k;
477 <        double  val[100];
476 >        COLR    *scan;
477 >        int     j;
478  
479 <        if (rm->ncomp > 100) {
480 <                fputs("Unsupported # components in rmx_write_double()\n", stderr);
481 <                exit(1);
479 >        if ((nc != 1) & (nc != 3)) return(0);
480 >        scan = (COLR *)tempbuffer(sizeof(COLR)*len);
481 >        if (!scan) return(0);
482 >
483 >        for (j = 0; j < len; j++, dp += nc)
484 >                if (nc == 1)
485 >                        setcolr(scan[j], dp[0], dp[0], dp[0]);
486 >                else
487 >                        setcolr(scan[j], dp[0], dp[1], dp[2]);
488 >
489 >        return(fwritecolrs(scan, len, fp) >= 0);
490 > }
491 >
492 > static int
493 > rmx_write_spec(const double *dp, int nc, int len, FILE *fp)
494 > {
495 >        uby8    *scan;
496 >        SCOLOR  scol;
497 >        int     j, k;
498 >
499 >        if (nc < 3) return(0);
500 >        scan = (uby8 *)tempbuffer((nc+1)*len);
501 >        if (!scan) return(0);
502 >        for (j = len; j--; dp += nc) {
503 >                for (k = nc; k--; )
504 >                        scol[k] = dp[k];
505 >                scolor2scolr(scan+j*(nc+1), scol, nc);
506          }
507 <        for (i = 0; i < rm->nrows; i++)
345 <            for (j = 0; j < rm->ncols; j++) {
346 <                for (k = rm->ncomp; k--; )
347 <                    val[k] = rmx_lval(rm,i,j,k);
348 <                if (putbinary(val, sizeof(val[0]), rm->ncomp, fp) != rm->ncomp)
349 <                        return(0);
350 <            }
351 <        return(1);
507 >        return(fwritescolrs(scan, nc, len, fp) >= 0);
508   }
509  
510 + /* Check if CIE XYZ primaries were specified */
511   static int
512 < rmx_write_rgbe(const RMATRIX *rm, FILE *fp)
512 > findCIEprims(const char *info)
513   {
514 <        COLOR   *scan = (COLOR *)malloc(sizeof(COLOR)*rm->ncols);
358 <        int     i, j;
514 >        RGBPRIMS        prims;
515  
516 <        if (scan == NULL)
516 >        if (!info)
517                  return(0);
518 <        for (i = 0; i < rm->nrows; i++) {
519 <            for (j = rm->ncols; j--; )
364 <                setcolor(scan[j],       rmx_lval(rm,i,j,0),
365 <                                        rmx_lval(rm,i,j,1),
366 <                                        rmx_lval(rm,i,j,2)      );
367 <            if (fwritescan(scan, rm->ncols, fp) < 0) {
368 <                free(scan);
518 >        info = strstr(info, PRIMARYSTR);
519 >        if (!info || !primsval(prims, info))
520                  return(0);
521 <            }
522 <        }
523 <        free(scan);
524 <        return(1);
521 >
522 >        return((prims[RED][CIEX] > .99) & (prims[RED][CIEY] < .01) &&
523 >                        (prims[GRN][CIEX] < .01) & (prims[GRN][CIEY] > .99) &&
524 >                        (prims[BLU][CIEX] < .01) & (prims[BLU][CIEY] < .01));
525   }
526  
527 < /* Write matrix to file type indicated by dtype */
527 > /* Finish writing header data with resolution and format, returning type used */
528   int
529 < rmx_write(const RMATRIX *rm, int dtype, FILE *fp)
529 > rmx_write_header(const RMATRIX *rm, int dtype, FILE *fp)
530   {
531 <        RMATRIX *mydm = NULL;
381 <        int     ok = 1;
382 <
383 <        if ((rm == NULL) | (fp == NULL))
531 >        if (!rm | !fp || rm->ncols <= 0)
532                  return(0);
385 – #ifdef getc_unlocked
386 –        flockfile(fp);
387 – #endif
388 –                                                /* complete header */
533          if (rm->info)
534                  fputs(rm->info, fp);
535          if (dtype == DTfromHeader)
536                  dtype = rm->dtype;
537 <        else if ((dtype == DTrgbe) & (rm->dtype == DTxyze))
537 >        else if (dtype == DTrgbe && (rm->dtype == DTxyze ||
538 >                                        findCIEprims(rm->info)))
539                  dtype = DTxyze;
540          else if ((dtype == DTxyze) & (rm->dtype == DTrgbe))
541                  dtype = DTrgbe;
542 <        if ((dtype != DTrgbe) & (dtype != DTxyze)) {
543 <                fprintf(fp, "NROWS=%d\n", rm->nrows);
542 >        if ((dtype < DTspec) & (rm->ncomp > 3))
543 >                dtype = DTspec;
544 >        else if ((dtype == DTspec) & (rm->ncomp <= 3))
545 >                return(0);
546 >
547 >        if (dtype == DTascii)                   /* set file type (WINDOWS) */
548 >                SET_FILE_TEXT(fp);
549 >        else
550 >                SET_FILE_BINARY(fp);
551 >                                                /* write exposure? */
552 >        if (rm->ncomp == 3 && (rm->cexp[RED] != rm->cexp[GRN]) |
553 >                        (rm->cexp[GRN] != rm->cexp[BLU]))
554 >                fputcolcor(rm->cexp, fp);
555 >        else if (rm->cexp[GRN] != 1.f)
556 >                fputexpos(rm->cexp[GRN], fp);
557 >                                                /* matrix size? */
558 >        if ((dtype > DTspec) | (rm->nrows <= 0)) {
559 >                if (rm->nrows > 0)
560 >                        fprintf(fp, "NROWS=%d\n", rm->nrows);
561                  fprintf(fp, "NCOLS=%d\n", rm->ncols);
400 –                fprintf(fp, "NCOMP=%d\n", rm->ncomp);
401 –        } else if (rm->ncomp != 3) {            /* wrong # components? */
402 –                double  cmtx[3];
403 –                if (rm->ncomp != 1)             /* only convert grayscale */
404 –                        return(0);
405 –                cmtx[0] = cmtx[1] = cmtx[2] = 1;
406 –                mydm = rmx_transform(rm, 3, cmtx);
407 –                if (mydm == NULL)
408 –                        return(0);
409 –                rm = mydm;
562          }
563 <        fputformat((char *)cm_fmt_id[dtype], fp);
564 <        fputc('\n', fp);
565 <        switch (dtype) {                        /* write data */
563 >        if (dtype >= DTspec) {                  /* # components & split? */
564 >                fputncomp(rm->ncomp, fp);
565 >                if (rm->ncomp > 3 &&
566 >                                memcmp(rm->wlpart, WLPART, sizeof(WLPART)))
567 >                        fputwlsplit(rm->wlpart, fp);
568 >        } else if ((rm->ncomp != 3) & (rm->ncomp != 1))
569 >                return(0);                      /* wrong # components */
570 >        if ((dtype == DTfloat) | (dtype == DTdouble))
571 >                fputendian(fp);                 /* important to record */
572 >        fputformat(cm_fmt_id[dtype], fp);
573 >        fputc('\n', fp);                        /* end of header */
574 >        if ((dtype <= DTspec) & (rm->nrows > 0))
575 >                fprtresolu(rm->ncols, rm->nrows, fp);
576 >        return(dtype);
577 > }
578 >
579 > /* Write out matrix data (usually by row) */
580 > int
581 > rmx_write_data(const double *dp, int nc, int len, int dtype, FILE *fp)
582 > {
583 >        switch (dtype) {
584          case DTascii:
585 <                ok = rmx_write_ascii(rm, fp);
416 <                break;
585 >                return(rmx_write_ascii(dp, nc, len, fp));
586          case DTfloat:
587 <                ok = rmx_write_float(rm, fp);
419 <                break;
587 >                return(rmx_write_float(dp, nc*len, fp));
588          case DTdouble:
589 <                ok = rmx_write_double(rm, fp);
422 <                break;
589 >                return(putbinary(dp, sizeof(*dp)*nc, len, fp) == len);
590          case DTrgbe:
591          case DTxyze:
592 <                fprtresolu(rm->ncols, rm->nrows, fp);
593 <                ok = rmx_write_rgbe(rm, fp);
594 <                break;
428 <        default:
429 <                return(0);
592 >                return(rmx_write_rgbe(dp, nc, len, fp));
593 >        case DTspec:
594 >                return(rmx_write_spec(dp, nc, len, fp));
595          }
596 <        ok &= (fflush(fp) == 0);
596 >        return(0);
597 > }
598 >
599 > /* Write matrix using file format indicated by dtype */
600 > int
601 > rmx_write(const RMATRIX *rm, int dtype, FILE *fp)
602 > {
603 >        int     ok = 0;
604 >        int     i;
605 >                                                /* complete header */
606 >        dtype = rmx_write_header(rm, dtype, fp);
607 >        if (dtype <= 0)
608 >                return(0);
609   #ifdef getc_unlocked
610 +        flockfile(fp);
611 + #endif
612 +        if (dtype == DTdouble)                  /* write all at once? */
613 +                ok = rmx_write_data(rm->mtx, rm->ncomp,
614 +                                rm->nrows*rm->ncols, dtype, fp);
615 +        else                                    /* else row by row */
616 +                for (i = 0; i < rm->nrows; i++) {
617 +                        ok = rmx_write_data(rmx_val(rm,i,0), rm->ncomp,
618 +                                        rm->ncols, dtype, fp);
619 +                        if (!ok) break;
620 +                }
621 +
622 +        if (ok) ok = (fflush(fp) == 0);
623 + #ifdef getc_unlocked
624          funlockfile(fp);
625   #endif
626 <        rmx_free(mydm);
626 >        if (!ok) fputs("Error writing matrix\n", stderr);
627          return(ok);
628   }
629  
# Line 443 | Line 634 | rmx_identity(const int dim, const int n)
634          RMATRIX *rid = rmx_alloc(dim, dim, n);
635          int     i, k;
636  
637 <        if (rid == NULL)
637 >        if (!rid)
638                  return(NULL);
639 <        memset(rid->mtx, 0, sizeof(rid->mtx[0])*n*dim*dim);
640 <        for (i = dim; i--; )
639 >        memset(rid->mtx, 0, array_size(rid));
640 >        for (i = dim; i--; ) {
641 >            double      *dp = rmx_lval(rid,i,i);
642              for (k = n; k--; )
643 <                rmx_lval(rid,i,i,k) = 1;
643 >                dp[k] = 1.;
644 >        }
645          return(rid);
646   }
647  
648 < /* Duplicate the given matrix */
648 > /* Duplicate the given matrix (may be unallocated) */
649   RMATRIX *
650   rmx_copy(const RMATRIX *rm)
651   {
652          RMATRIX *dnew;
653  
654 <        if (rm == NULL)
654 >        if (!rm)
655                  return(NULL);
656 <        dnew = rmx_alloc(rm->nrows, rm->ncols, rm->ncomp);
657 <        if (dnew == NULL)
656 >        dnew = rmx_new(rm->nrows, rm->ncols, rm->ncomp);
657 >        if (!dnew)
658                  return(NULL);
659 +        if (rm->mtx) {
660 +                if (!rmx_prepare(dnew)) {
661 +                        rmx_free(dnew);
662 +                        return(NULL);
663 +                }
664 +                memcpy(dnew->mtx, rm->mtx, array_size(dnew));
665 +        }
666          rmx_addinfo(dnew, rm->info);
667          dnew->dtype = rm->dtype;
668 <        memcpy(dnew->mtx, rm->mtx,
669 <                sizeof(rm->mtx[0])*rm->ncomp*rm->nrows*rm->ncols);
668 >        copycolor(dnew->cexp, rm->cexp);
669 >        memcpy(dnew->wlpart, rm->wlpart, sizeof(dnew->wlpart));
670          return(dnew);
671   }
672  
673 + /* Replace data in first matrix with data from second */
674 + int
675 + rmx_transfer_data(RMATRIX *rdst, RMATRIX *rsrc, int dometa)
676 + {
677 +        if (!rdst | !rsrc || (rdst->nrows != rsrc->nrows) |
678 +                        (rdst->ncols != rsrc->ncols) |
679 +                        (rdst->ncomp != rsrc->ncomp))
680 +                return(0);
681 +
682 +        if (dometa) {           /* transfer everything? */
683 +                rmx_reset(rdst);
684 +                *rdst = *rsrc;
685 +                rsrc->info = NULL; rsrc->mapped = NULL; rsrc->mtx = NULL;
686 +                return(1);
687 +        }
688 +        if (rdst->mapped)
689 +                return(0);      /* XXX can't handle this case */
690 +                                /* just matrix data -- leave metadata */
691 +        if (rdst->mtx) free(rdst->mtx);
692 +        rdst->mtx = rsrc->mtx;
693 +        rsrc->mtx = NULL;
694 +        return(1);
695 + }
696 +
697   /* Allocate and assign transposed matrix */
698   RMATRIX *
699   rmx_transpose(const RMATRIX *rm)
700   {
701          RMATRIX *dnew;
702 <        int     i, j, k;
702 >        int     i, j;
703  
704 <        if (rm == NULL)
704 >        if (!rm || !rm->mtx)
705                  return(0);
706 +        if ((rm->nrows == 1) | (rm->ncols == 1)) {
707 +                dnew = rmx_copy(rm);
708 +                if (!dnew)
709 +                        return(NULL);
710 +                dnew->nrows = rm->ncols;
711 +                dnew->ncols = rm->nrows;
712 +                return(dnew);
713 +        }
714          dnew = rmx_alloc(rm->ncols, rm->nrows, rm->ncomp);
715 <        if (dnew == NULL)
715 >        if (!dnew)
716                  return(NULL);
717          if (rm->info) {
718                  rmx_addinfo(dnew, rm->info);
719                  rmx_addinfo(dnew, "Transposed rows and columns\n");
720          }
721          dnew->dtype = rm->dtype;
722 <        for (i = dnew->nrows; i--; )
723 <            for (j = dnew->ncols; j--; )
724 <                for (k = dnew->ncomp; k--; )
725 <                        rmx_lval(dnew,i,j,k) = rmx_lval(rm,j,i,k);
722 >        copycolor(dnew->cexp, rm->cexp);
723 >        memcpy(dnew->wlpart, rm->wlpart, sizeof(dnew->wlpart));
724 >        for (j = dnew->ncols; j--; )
725 >            for (i = dnew->nrows; i--; )
726 >                memcpy(rmx_lval(dnew,i,j), rmx_val(rm,j,i),
727 >                                sizeof(double)*dnew->ncomp);
728          return(dnew);
729   }
730  
# Line 501 | Line 735 | rmx_multiply(const RMATRIX *m1, const RMATRIX *m2)
735          RMATRIX *mres;
736          int     i, j, k, h;
737  
738 <        if ((m1 == NULL) | (m2 == NULL) ||
738 >        if (!m1 | !m2 || !m1->mtx | !m2->mtx |
739                          (m1->ncomp != m2->ncomp) | (m1->ncols != m2->nrows))
740                  return(NULL);
741          mres = rmx_alloc(m1->nrows, m2->ncols, m1->ncomp);
742 <        if (mres == NULL)
742 >        if (!mres)
743                  return(NULL);
744          i = rmx_newtype(m1->dtype, m2->dtype);
745          if (i)
# Line 515 | Line 749 | rmx_multiply(const RMATRIX *m1, const RMATRIX *m2)
749          for (i = mres->nrows; i--; )
750              for (j = mres->ncols; j--; )
751                  for (k = mres->ncomp; k--; ) {
752 <                    long double d = 0;
752 >                    double      d = 0;
753                      for (h = m1->ncols; h--; )
754 <                        d += rmx_lval(m1,i,h,k) * rmx_lval(m2,h,j,k);
755 <                    rmx_lval(mres,i,j,k) = (double)d;
754 >                        d += rmx_val(m1,i,h)[k] * rmx_val(m2,h,j)[k];
755 >                    rmx_lval(mres,i,j)[k] = d;
756                  }
757          return(mres);
758   }
# Line 530 | Line 764 | rmx_elemult(RMATRIX *m1, const RMATRIX *m2, int divide
764          int     zeroDivides = 0;
765          int     i, j, k;
766  
767 <        if ((m1 == NULL) | (m2 == NULL) ||
768 <                        (m1->ncols != m2->ncols) | (m1->nrows != m2->nrows))
767 >        if (!m1 | !m2 || !m1->mtx | !m2->mtx |
768 >                         (m1->ncols != m2->ncols) | (m1->nrows != m2->nrows))
769                  return(0);
770          if ((m2->ncomp > 1) & (m2->ncomp != m1->ncomp))
771                  return(0);
# Line 545 | Line 779 | rmx_elemult(RMATRIX *m1, const RMATRIX *m2, int divide
779                  if (divide) {
780                      double      d;
781                      if (m2->ncomp == 1) {
782 <                        d = rmx_lval(m2,i,j,0);
782 >                        d = rmx_val(m2,i,j)[0];
783                          if (d == 0) {
784                              ++zeroDivides;
785                              for (k = m1->ncomp; k--; )
786 <                                rmx_lval(m1,i,j,k) = 0;
786 >                                rmx_lval(m1,i,j)[k] = 0;
787                          } else {
788                              d = 1./d;
789                              for (k = m1->ncomp; k--; )
790 <                                rmx_lval(m1,i,j,k) *= d;
790 >                                rmx_lval(m1,i,j)[k] *= d;
791                          }
792                      } else
793                          for (k = m1->ncomp; k--; ) {
794 <                            d = rmx_lval(m2,i,j,k);
794 >                            d = rmx_val(m2,i,j)[k];
795                              if (d == 0) {
796                                  ++zeroDivides;
797 <                                rmx_lval(m1,i,j,k) = 0;
797 >                                rmx_lval(m1,i,j)[k] = 0;
798                              } else
799 <                                rmx_lval(m1,i,j,k) /= d;
799 >                                rmx_lval(m1,i,j)[k] /= d;
800                          }
801                  } else {
802                      if (m2->ncomp == 1) {
803 <                        const double    d = rmx_lval(m2,i,j,0);
803 >                        const double    d = rmx_val(m2,i,j)[0];
804                          for (k = m1->ncomp; k--; )
805 <                            rmx_lval(m1,i,j,k) *= d;
805 >                            rmx_lval(m1,i,j)[k] *= d;
806                      } else
807                          for (k = m1->ncomp; k--; )
808 <                            rmx_lval(m1,i,j,k) *= rmx_lval(m2,i,j,k);
808 >                            rmx_lval(m1,i,j)[k] *= rmx_val(m2,i,j)[k];
809                  }
810          if (zeroDivides) {
811 <                fputs("Divide by zero in rmx_elemult()\n", stderr);
811 >                rmx_addinfo(m1, "WARNING: zero divide(s) corrupted results\n");
812                  errno = ERANGE;
813          }
814          return(1);
# Line 587 | Line 821 | rmx_sum(RMATRIX *msum, const RMATRIX *madd, const doub
821          double  *mysf = NULL;
822          int     i, j, k;
823  
824 <        if ((msum == NULL) | (madd == NULL) ||
824 >        if (!msum | !madd || !msum->mtx | !madd->mtx |
825                          (msum->nrows != madd->nrows) |
826                          (msum->ncols != madd->ncols) |
827                          (msum->ncomp != madd->ncomp))
828                  return(0);
829 <        if (sf == NULL) {
829 >        if (!sf) {
830                  mysf = (double *)malloc(sizeof(double)*msum->ncomp);
831 <                if (mysf == NULL)
831 >                if (!mysf)
832                          return(0);
833                  for (k = msum->ncomp; k--; )
834                          mysf[k] = 1;
# Line 606 | Line 840 | rmx_sum(RMATRIX *msum, const RMATRIX *madd, const doub
840          else
841                  rmx_addinfo(msum, rmx_mismatch_warn);
842          for (i = msum->nrows; i--; )
843 <            for (j = msum->ncols; j--; )
843 >            for (j = msum->ncols; j--; ) {
844 >                const double    *da = rmx_val(madd,i,j);
845 >                double          *ds = rmx_lval(msum,i,j);
846                  for (k = msum->ncomp; k--; )
847 <                     rmx_lval(msum,i,j,k) += sf[k] * rmx_lval(madd,i,j,k);
848 <
849 <        free(mysf);
847 >                     ds[k] += sf[k] * da[k];
848 >            }
849 >        if (mysf)
850 >                free(mysf);
851          return(1);
852   }
853  
# Line 620 | Line 857 | rmx_scale(RMATRIX *rm, const double sf[])
857   {
858          int     i, j, k;
859  
860 <        if ((rm == NULL) | (sf == NULL))
860 >        if (!rm | !sf || !rm->mtx)
861                  return(0);
862          for (i = rm->nrows; i--; )
863 <            for (j = rm->ncols; j--; )
863 >            for (j = rm->ncols; j--; ) {
864 >                double  *dp = rmx_lval(rm,i,j);
865                  for (k = rm->ncomp; k--; )
866 <                    rmx_lval(rm,i,j,k) *= sf[k];
867 <
866 >                    dp[k] *= sf[k];
867 >            }
868 >        if (rm->info)
869 >                rmx_addinfo(rm, "Applied scalar\n");
870 >        /* XXX: should record as exposure for COLR and SCOLR types? */
871          return(1);
872   }
873  
# Line 637 | Line 878 | rmx_transform(const RMATRIX *msrc, int n, const double
878          int     i, j, ks, kd;
879          RMATRIX *dnew;
880  
881 <        if ((msrc == NULL) | (n <= 0) | (cmat == NULL))
881 >        if (!msrc | (n <= 0) | !cmat || !msrc->mtx)
882                  return(NULL);
883          dnew = rmx_alloc(msrc->nrows, msrc->ncols, n);
884 <        if (dnew == NULL)
884 >        if (!dnew)
885                  return(NULL);
886 +        if (msrc->info) {
887 +                char    buf[128];
888 +                sprintf(buf, "Applied %dx%d component transform\n",
889 +                                dnew->ncomp, msrc->ncomp);
890 +                rmx_addinfo(dnew, msrc->info);
891 +                rmx_addinfo(dnew, buf);
892 +        }
893          dnew->dtype = msrc->dtype;
894          for (i = dnew->nrows; i--; )
895 <            for (j = dnew->ncols; j--; )
895 >            for (j = dnew->ncols; j--; ) {
896 >                const double    *ds = rmx_val(msrc,i,j);
897                  for (kd = dnew->ncomp; kd--; ) {
898                      double      d = 0;
899                      for (ks = msrc->ncomp; ks--; )
900 <                        d += cmat[kd*msrc->ncomp + ks] * rmx_lval(msrc,i,j,ks);
901 <                    rmx_lval(dnew,i,j,kd) = d;
900 >                        d += cmat[kd*msrc->ncomp + ks] * ds[ks];
901 >                    rmx_lval(dnew,i,j)[kd] = d;
902                  }
903 +            }
904          return(dnew);
905   }
906  
# Line 661 | Line 911 | rmx_from_cmatrix(const CMATRIX *cm)
911          int     i, j;
912          RMATRIX *dnew;
913  
914 <        if (cm == NULL)
914 >        if (!cm)
915                  return(NULL);
916          dnew = rmx_alloc(cm->nrows, cm->ncols, 3);
917 <        if (dnew == NULL)
917 >        if (!dnew)
918                  return(NULL);
919          dnew->dtype = DTfloat;
920          for (i = dnew->nrows; i--; )
921              for (j = dnew->ncols; j--; ) {
922                  const COLORV    *cv = cm_lval(cm,i,j);
923 <                rmx_lval(dnew,i,j,0) = cv[0];
924 <                rmx_lval(dnew,i,j,1) = cv[1];
925 <                rmx_lval(dnew,i,j,2) = cv[2];
923 >                double          *dp = rmx_lval(dnew,i,j);
924 >                dp[0] = cv[0];
925 >                dp[1] = cv[1];
926 >                dp[2] = cv[2];
927              }
928          return(dnew);
929   }
# Line 684 | Line 935 | cm_from_rmatrix(const RMATRIX *rm)
935          int     i, j;
936          CMATRIX *cnew;
937  
938 <        if (rm == NULL || rm->ncomp != 3)
938 >        if (!rm || !rm->mtx | (rm->ncomp == 2))
939                  return(NULL);
940          cnew = cm_alloc(rm->nrows, rm->ncols);
941 <        if (cnew == NULL)
941 >        if (!cnew)
942                  return(NULL);
943          for (i = cnew->nrows; i--; )
944              for (j = cnew->ncols; j--; ) {
945 <                COLORV  *cv = cm_lval(cnew,i,j);
946 <                cv[0] = (COLORV)rmx_lval(rm,i,j,0);
947 <                cv[1] = (COLORV)rmx_lval(rm,i,j,1);
948 <                cv[2] = (COLORV)rmx_lval(rm,i,j,2);
945 >                const double    *dp = rmx_val(rm,i,j);
946 >                COLORV          *cv = cm_lval(cnew,i,j);
947 >                switch (rm->ncomp) {
948 >                case 3:
949 >                    setcolor(cv, dp[0], dp[1], dp[2]);
950 >                    break;
951 >                case 1:
952 >                    setcolor(cv, dp[0], dp[0], dp[0]);
953 >                    break;
954 >                default: {
955 >                        SCOLOR  scol;
956 >                        int     k;
957 >                        for (k = rm->ncomp; k--; )
958 >                                scol[k] = dp[k];
959 >                        scolor2color(cv, scol, rm->ncomp, rm->wlpart);
960 >                    } break;
961 >                }
962              }
963          return(cnew);
964   }

Diff Legend

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