[ Return to Bugs & Features | Post Text | Post File | Prev | Next ]

STR #2633

Trouble Report Files:

	Name/Time/Date	Filename/Size
	#1  mingodad 02:23 May 17, 2011	save-image.patch 6k

Trouble Report Comments:

	Name/Time/Date	Text
	#1  mingodad 02:23 May 17, 2011	Here is a patch to fltk 1.3 that add a static method for Fl_JPEG_Image and Fl_PNG_Image to encode from Fl_Image to JPEG and PNG format on memory.
	#2  ianmacarthur 02:51 May 17, 2011	I moving this to 1.4 temporarily so that it will not block the release of 1.3.0. Also, somne comments: - The PNG encoder needs to handle image depths of 2 also (i.e. greyscale with alpha) which it currently does not. - The JPEG encoder needs to process images with alpha too, not just discard them. - The name of the method, "encode" may not be appropriate - the fltk-style is more to call it "store" or "save" or something I guess. We should also make it explicit that this creates an in-memory version of the compressed image (which can then be written to file if needs be, of course.)
	#3  matt 03:00 May 17, 2011	I suggest that we add code to save to a file as well. I'd call them "compress_to_file(const char *filename,...)" and "compress_to_memory(void*,...)" or "save_to_file" and "save_to_memory".
	#4  ianmacarthur 03:08 May 17, 2011	Seems reasonable. The "save to file" versions would just wrap the in-memory versions? Or be standalone functions?
	#5  mike 13:17 May 17, 2011	Might be useful to include a background color for saving to JPEG - then the colors in the image can be composited with the background color...
	#6  ianmacarthur 13:28 May 17, 2011	Yes - seems like a good idea, if we are going to catch the alpha, we need *something* to blend it with. I guess we'd default to white (or black?) if no colour were supplied?
	#7  mike 13:52 May 17, 2011	Probably default to white...

[ Return to Bugs & Features | Post Text | Post File ]