Re: color box appearing for rgb plots

[Daniel J S. <dan...@ie...>]

Petr Mikulik wrote:
>>I've compiled the latest CVS version and run the demo.  I notice
>>that in the "image.dem" demo that color boxes are now appearing
>>with RGB plots by default.  That used to be disallowed.  Everyone
>>agree that color boxes shouldn't appear for RGB (i.e., non-palette)
>>plots?
> 
> 
> I though both are IMAGE and RGBIMAGE are equivalent, thus I let them
> function in the same way.
> 
> See pm3d.c, lines 768 and 794.
> If you replace  if ( ... == IMAGE || ... == RGBIMAGE)  by
> 
> #ifdef WITH_IMAGE
> 	if (this_3dplot->plot_style == IMAGE)
>     	    return;
> 	if (this_3dplot->plot_style == RGBIMAGE) {
> 	    want_palette_but_not_colorbox = TRUE;
>     	    return;
> 	}
> #endif
> 
> does it work as you expect?
> 
> 
> Yet another question:
> Does RGBIMAGE need palette of continous colors or not?
> Thus, should plot_has_palette be TRUE or FALSE?

Well, I think the above would work (sort of), but really
an RGBIMAGE does not need a palette.

However, is it possible to have both an RGBIMAGE and a
palette-based IMAGE on a plot at the same time?  If so,
it may be the case that one should set a variable like
"has_palette" to false when starting the plot and then
set that variable to TRUE when IMAGE is created, but
don't set it to FALSE when RGBIMAGE is created.

Dan