X-Git-Url: http://git.hungrycats.org/cgi-bin/gitweb.cgi?a=blobdiff_plain;f=utils%2Fcolors.c;h=e06359c72db811738959df9bc1f733e359485a4d;hb=50be9bb40dc60130c99ffa568e6677779904ff70;hp=b568a24dd11d9379b69034746bc1b855d3bc76ee;hpb=ce3185de9d9705e259f2b60dd4b5509007fa17d4;p=xscreensaver diff --git a/utils/colors.c b/utils/colors.c index b568a24d..e06359c7 100644 --- a/utils/colors.c +++ b/utils/colors.c @@ -1,4 +1,4 @@ -/* xscreensaver, Copyright (c) 1997 Jamie Zawinski +/* xscreensaver, Copyright (c) 1997, 2002 Jamie Zawinski * * Permission to use, copy, modify, distribute, and sell this software and its * documentation for any purpose is hereby granted without fee, provided that @@ -73,6 +73,25 @@ allocate_writable_colors (Display *dpy, Colormap cmap, } +static void +complain (int wanted_colors, int got_colors, + Bool wanted_writable, Bool got_writable) +{ + if (got_colors > wanted_colors - 10) + /* don't bother complaining if we're within ten pixels. */ + return; + + if (wanted_writable && !got_writable) + fprintf (stderr, + "%s: wanted %d writable colors; got %d read-only colors.\n", + progname, wanted_colors, got_colors); + else + fprintf (stderr, "%s: wanted %d%s colors; got %d.\n", + progname, wanted_colors, (got_writable ? " writable" : ""), + got_colors); +} + + void make_color_ramp (Display *dpy, Colormap cmap, @@ -83,11 +102,19 @@ make_color_ramp (Display *dpy, Colormap cmap, Bool allocate_p, Bool writable_p) { + Bool verbose_p = True; /* argh. */ int i; - int ncolors = *ncolorsP; + int total_ncolors = *ncolorsP; + int ncolors, wanted; + Bool wanted_writable = (allocate_p && writable_p); double dh, ds, dv; /* deltas */ + wanted = total_ncolors; + if (closed_p) + wanted = (wanted / 2) + 1; + AGAIN: + ncolors = total_ncolors; memset (colors, 0, (*ncolorsP) * sizeof(*colors)); @@ -154,22 +181,30 @@ make_color_ramp (Display *dpy, Colormap cmap, } } - return; + goto WARN; FAIL: /* we weren't able to allocate all the colors we wanted; decrease the requested number and try again. */ - ncolors = (ncolors > 170 ? ncolors - 20 : - ncolors > 100 ? ncolors - 10 : - ncolors > 75 ? ncolors - 5 : - ncolors > 25 ? ncolors - 3 : - ncolors > 10 ? ncolors - 2 : - ncolors > 2 ? ncolors - 1 : - 0); - *ncolorsP = ncolors; - if (ncolors > 0) + total_ncolors = (total_ncolors > 170 ? total_ncolors - 20 : + total_ncolors > 100 ? total_ncolors - 10 : + total_ncolors > 75 ? total_ncolors - 5 : + total_ncolors > 25 ? total_ncolors - 3 : + total_ncolors > 10 ? total_ncolors - 2 : + total_ncolors > 2 ? total_ncolors - 1 : + 0); + *ncolorsP = total_ncolors; + ncolors = total_ncolors; + if (total_ncolors > 0) goto AGAIN; + + WARN: + + if (verbose_p && + /* don't warn if we got 0 writable colors -- probably TrueColor. */ + (ncolors != 0 || !wanted_writable)) + complain (wanted, ncolors, wanted_writable, wanted_writable && writable_p); } @@ -301,16 +336,11 @@ make_color_path (Display *dpy, Colormap cmap, k = 0; for (i = 0; i < npoints; i++) { - int distance, direction; - distance = h[(i+1) % npoints] - h[i]; - direction = (distance >= 0 ? -1 : 1); - - if (distance > 180) - distance = 180 - (distance - 180); - else if (distance < -180) - distance = -(180 - ((-distance) - 180)); - else - direction = -direction; + int distance = h[(i+1) % npoints] - h[i]; + int direction = (distance >= 0 ? -1 : 1); + + if (distance <= 180 && distance >= -180) + direction = -direction; #ifdef DEBUG fprintf (stderr, "point %d: %3d %.2f %.2f\n", @@ -428,23 +458,6 @@ make_color_loop (Display *dpy, Colormap cmap, } -static void -complain (int wanted_colors, int got_colors, - Bool wanted_writable, Bool got_writable) -{ - if (wanted_writable && !got_writable) - fprintf(stderr, - "%s: wanted %d writable colors; got %d read-only colors.\n", - progname, wanted_colors, got_colors); - - else if (wanted_colors > (got_colors + 10)) - /* don't bother complaining if we're within ten pixels. */ - fprintf(stderr, "%s: wanted %d%s colors; got %d.\n", - progname, wanted_colors, (got_writable ? " writable" : ""), - got_colors); -} - - void make_smooth_colormap (Display *dpy, Visual *visual, Colormap cmap, XColor *colors, int *ncolorsP, @@ -461,7 +474,8 @@ make_smooth_colormap (Display *dpy, Visual *visual, Colormap cmap, double v[MAXPOINTS]; double total_s = 0; double total_v = 0; - Screen *screen = DefaultScreenOfDisplay(dpy); /* #### WRONG! */ + Screen *screen = (dpy ? DefaultScreenOfDisplay(dpy) : 0); /* #### WRONG! */ + int loop = 0; if (*ncolorsP <= 0) return; @@ -477,6 +491,7 @@ make_smooth_colormap (Display *dpy, Visual *visual, Colormap cmap, for (i = 0; i < npoints; i++) { REPICK_THIS_COLOR: + if (++loop > 10000) abort(); h[i] = random() % 360; s[i] = frand(1.0); v[i] = frand(0.8) + 0.2; @@ -521,7 +536,7 @@ make_smooth_colormap (Display *dpy, Visual *visual, Colormap cmap, allocate_p, (writable_pP && *writable_pP)); /* If we tried for writable cells and got none, try for non-writable. */ - if (allocate_p && *ncolorsP == 0 && *writable_pP) + if (allocate_p && *ncolorsP == 0 && writable_pP && *writable_pP) { *writable_pP = False; goto RETRY_NON_WRITABLE; @@ -544,7 +559,7 @@ make_uniform_colormap (Display *dpy, Visual *visual, Colormap cmap, { int ncolors = *ncolorsP; Bool wanted_writable = (allocate_p && writable_pP && *writable_pP); - Screen *screen = DefaultScreenOfDisplay(dpy); /* #### WRONG! */ + Screen *screen = (dpy ? DefaultScreenOfDisplay(dpy) : 0); /* #### WRONG! */ double S = ((double) (random() % 34) + 66) / 100.0; /* range 66%-100% */ double V = ((double) (random() % 34) + 66) / 100.0; /* range 66%-100% */ @@ -560,7 +575,8 @@ make_uniform_colormap (Display *dpy, Visual *visual, Colormap cmap, 0, S, V, 359, S, V, colors, &ncolors, - False, True, wanted_writable); + False, allocate_p, + (writable_pP && *writable_pP)); /* If we tried for writable cells and got none, try for non-writable. */ if (allocate_p && *ncolorsP == 0 && writable_pP && *writable_pP) @@ -589,7 +605,7 @@ make_random_colormap (Display *dpy, Visual *visual, Colormap cmap, Bool wanted_writable = (allocate_p && writable_pP && *writable_pP); int ncolors = *ncolorsP; int i; - Screen *screen = DefaultScreenOfDisplay(dpy); /* #### WRONG! */ + Screen *screen = (dpy ? DefaultScreenOfDisplay(dpy) : 0); /* #### WRONG! */ if (*ncolorsP <= 0) return;