[amsn] Fix compile against new libpng (backport http://amsn.svn.sourceforge.net/viewvc/amsn?view=revision&r

Sander Hoentjen tjikkun at fedoraproject.org
Fri Jan 27 12:57:51 UTC 2012


commit 492d513eee790fe9f40a47a018978a558964e0ac
Author: Sander Hoentjen <s.hoentjen at oxilion.nl>
Date:   Fri Jan 27 13:57:25 2012 +0100

    Fix compile against new libpng (backport http://amsn.svn.sourceforge.net/viewvc/amsn?view=revision&revision=12357)

 amsn-0.98.4-libpng15.patch |  498 ++++++++++++++++++++++++++++++++++++++++++++
 amsn.spec                  |   49 +++--
 2 files changed, 525 insertions(+), 22 deletions(-)
---
diff --git a/amsn-0.98.4-libpng15.patch b/amsn-0.98.4-libpng15.patch
new file mode 100644
index 0000000..5beae53
--- /dev/null
+++ b/amsn-0.98.4-libpng15.patch
@@ -0,0 +1,498 @@
+--- amsn/utils/TkCximage/src/CxImage/ximapng.cpp	2012/01/14 15:27:06	12356
++++ amsn/utils/TkCximage/src/CxImage/ximapng.cpp	2012/01/24 21:21:23	12357
+@@ -15,7 +15,7 @@
+ void CxImagePNG::ima_png_error(png_struct *png_ptr, char *message)
+ {
+ 	strcpy(info.szLastError,message);
+-	longjmp(png_ptr->jmpbuf, 1);
++	longjmp(png_jmpbuf(png_ptr), 1);
+ }
+ ////////////////////////////////////////////////////////////////////////////////
+ #if CXIMAGE_SUPPORT_DECODE
+@@ -59,10 +59,10 @@
+ 		cx_throw("Failed to initialize PNG info structure");
+ 	}
+ 
+-    /* Set error handling if you are using the setjmp/longjmp method (this is
+-    * the normal method of doing things with libpng).  REQUIRED unless you
+-    * set up your own error handlers in the png_create_read_struct() earlier. */
+-	if (setjmp(png_ptr->jmpbuf)) {
++	/* Set error handling if you are using the setjmp/longjmp method (this is
++	 * the normal method of doing things with libpng).  REQUIRED unless you
++	 * set up your own error handlers in the png_create_read_struct() earlier. */
++	if (setjmp(png_jmpbuf(png_ptr))) {
+ 		/* Free all of the memory associated with the png_ptr and info_ptr */
+ 		delete [] row_pointers;
+ 		png_destroy_read_struct(&png_ptr, &info_ptr, (png_infopp)NULL);
+@@ -70,22 +70,22 @@
+ 	}
+ 
+ 	// use custom I/O functions
+-    png_set_read_fn(png_ptr, hFile, /*(png_rw_ptr)*/user_read_data);
++	png_set_read_fn(png_ptr, hFile, /*(png_rw_ptr)*/user_read_data);
+ 	png_set_error_fn(png_ptr,info.szLastError,/*(png_error_ptr)*/user_error_fn,NULL);
+ 
+ 	/* read the file information */
+ 	png_read_info(png_ptr, info_ptr);
+ 
+ 	if (info.nEscape == -1){
+-		head.biWidth = info_ptr->width;
+-		head.biHeight= info_ptr->height;
++		head.biWidth = png_get_image_width(png_ptr, info_ptr);
++		head.biHeight= png_get_image_height(png_ptr, info_ptr);
+ 		info.dwType = CXIMAGE_FORMAT_PNG;
+-		longjmp(png_ptr->jmpbuf, 1);
++		longjmp(png_jmpbuf(png_ptr), 1);
+ 	}
+ 
+ 	/* calculate new number of channels */
+ 	int channels=0;
+-	switch(info_ptr->color_type){
++	switch(png_get_color_type(png_ptr, info_ptr)){
+ 	case PNG_COLOR_TYPE_GRAY:
+ 	case PNG_COLOR_TYPE_PALETTE:
+ 		channels = 1;
+@@ -101,97 +101,106 @@
+ 		break;
+ 	default:
+ 		strcpy(info.szLastError,"unknown PNG color type");
+-		longjmp(png_ptr->jmpbuf, 1);
++		longjmp(png_jmpbuf(png_ptr), 1);
+ 	}
+ 
+ 	//find the right pixel depth used for cximage
+-	int pixel_depth = info_ptr->pixel_depth;
++	int pixel_depth = png_get_bit_depth(png_ptr, info_ptr)
++                        * png_get_channels(png_ptr, info_ptr);
+ 	if (channels == 1 && pixel_depth>8) pixel_depth=8;
+ 	if (channels == 2) pixel_depth=8;
+-	if (channels >= 3) pixel_depth=24;
++	if (channels >= 3) pixel_depth=32;
+ 
+-	if (!Create(info_ptr->width, info_ptr->height, pixel_depth, CXIMAGE_FORMAT_PNG)){
+-		longjmp(png_ptr->jmpbuf, 1);
++	if (!Create(png_get_image_width(png_ptr, info_ptr),
++                    png_get_image_height(png_ptr, info_ptr),
++                    pixel_depth, CXIMAGE_FORMAT_PNG)){
++		longjmp(png_jmpbuf(png_ptr), 1);
+ 	}
+ 
+ 	/* get metrics */
+-	switch (info_ptr->phys_unit_type)
++	png_uint_32 res_x, res_y;
++	int phys_unit_type;
++
++	png_get_pHYs(png_ptr, info_ptr, &res_x, &res_y, &phys_unit_type);
++	switch (phys_unit_type)
+ 	{
+ 	case PNG_RESOLUTION_UNKNOWN:
+-		SetXDPI(info_ptr->x_pixels_per_unit);
+-		SetYDPI(info_ptr->y_pixels_per_unit);
++		SetXDPI(res_x);
++		SetYDPI(res_y);
+ 		break;
+ 	case PNG_RESOLUTION_METER:
+-		SetXDPI((long)floor(info_ptr->x_pixels_per_unit * 254.0 / 10000.0 + 0.5));
+-		SetYDPI((long)floor(info_ptr->y_pixels_per_unit * 254.0 / 10000.0 + 0.5));
++		SetXDPI((long)floor(res_x * 254.0 / 10000.0 + 0.5));
++		SetYDPI((long)floor(res_y * 254.0 / 10000.0 + 0.5));
+ 		break;
+ 	}
+ 
+-	if (info_ptr->num_palette>0){
+-		SetPalette((rgb_color*)info_ptr->palette,info_ptr->num_palette);
+-		SetClrImportant(info_ptr->num_palette);
+-	} else if (info_ptr->bit_depth ==2) { //<DP> needed for 2 bpp grayscale PNGs
++	int num_palette = -1;
++	png_colorp palette;
++	if (png_get_PLTE(png_ptr, info_ptr, &palette, &num_palette)) {
++		SetPalette((rgb_color*)&palette, num_palette);
++		SetClrImportant(num_palette);
++	} else if (png_get_bit_depth(png_ptr, info_ptr)==2) { //<DP> needed for 2 bpp grayscale PNGs
+ 		SetPaletteColor(0,0,0,0);
+ 		SetPaletteColor(1,85,85,85);
+ 		SetPaletteColor(2,170,170,170);
+ 		SetPaletteColor(3,255,255,255);
+-	} else SetGrayPalette(); //<DP> needed for grayscale PNGs
++	} else {
++	   SetGrayPalette(); //<DP> needed for grayscale PNGs
++	}
+ 	
+-	int nshift = max(0,(info_ptr->bit_depth>>3)-1)<<3;
++	int nshift = max(0,(png_get_bit_depth(png_ptr, info_ptr)>>3)-1)<<3;
+ 
+-	if (info_ptr->num_trans!=0){ //palette transparency
+-		if (info_ptr->num_trans==1){
+-			if (info_ptr->color_type == PNG_COLOR_TYPE_PALETTE){
++	int num_trans = -1;
++	png_bytep trans_alpha;
++	png_color_16p trans_color;
++	if (png_get_tRNS(png_ptr, info_ptr, &trans_alpha, &num_trans, &trans_color)) {
++		if (num_trans==1){
++			if (png_get_color_type(png_ptr, info_ptr) == PNG_COLOR_TYPE_PALETTE){
+ #if PNG_LIBPNG_VER_MAJOR >= 1 && PNG_LIBPNG_VER_MINOR >= 4
+-				info.nBkgndIndex = info_ptr->trans_color.index;
++				info.nBkgndIndex = trans_color->index;
+ #else
+-				info.nBkgndIndex = info_ptr->trans_values.index;
++				info.nBkgndIndex = trans_values->index;
+ #endif
+ 			} else{
+ #if PNG_LIBPNG_VER_MAJOR >= 1 && PNG_LIBPNG_VER_MINOR >= 4
+-				info.nBkgndIndex = info_ptr->trans_color.gray>>nshift;
++				info.nBkgndIndex = trans_color->gray>>nshift;
+ #else
+-				info.nBkgndIndex = info_ptr->trans_values.gray>>nshift;
++				info.nBkgndIndex = trans_values->gray>>nshift;
+ #endif
+ 			}
+ 		}
+-		if (info_ptr->num_trans>1){
++		if (num_trans>1){
+ 			RGBQUAD* pal=GetPalette();
+ 			if (pal){
+ 				DWORD ip;
+-				for (ip=0;ip<min(head.biClrUsed,(unsigned long)info_ptr->num_trans);ip++)
++				for (ip=0;ip<min(head.biClrUsed,(unsigned long)num_trans);ip++)
+ #if PNG_LIBPNG_VER_MAJOR >= 1 && PNG_LIBPNG_VER_MINOR >= 4
+-					pal[ip].rgbReserved=info_ptr->trans_alpha[ip];
++					pal[ip].rgbReserved=trans_alpha[ip];
+ #else
+-					pal[ip].rgbReserved=info_ptr->trans[ip];
++					pal[ip].rgbReserved=trans[ip];
+ #endif
+-				for (ip=info_ptr->num_trans;ip<head.biClrUsed;ip++){
++				for (ip=num_trans;ip<head.biClrUsed;ip++){
+ 					pal[ip].rgbReserved=255;
+ 				}
+ 				info.bAlphaPaletteEnabled=true;
+ 			}
+ 		}
+-	}
+-
+-	if (channels == 3){ //check RGB binary transparency
+-		png_bytep trans;
+-		int num_trans;
+-		png_color_16 *image_background;
+-		if (png_get_tRNS(png_ptr, info_ptr, &trans, &num_trans, &image_background)){
++		if (channels == 3){ //check RGB binary transparency
+ #if PNG_LIBPNG_VER_MAJOR >= 1 && PNG_LIBPNG_VER_MINOR >= 4
+-			info.nBkgndColor.rgbRed   = (BYTE)(info_ptr->trans_color.red>>nshift);
+-			info.nBkgndColor.rgbGreen = (BYTE)(info_ptr->trans_color.green>>nshift);
+-			info.nBkgndColor.rgbBlue  = (BYTE)(info_ptr->trans_color.blue>>nshift);
++			info.nBkgndColor.rgbRed   = (BYTE)(trans_color->red>>nshift);
++			info.nBkgndColor.rgbGreen = (BYTE)(trans_color->green>>nshift);
++			info.nBkgndColor.rgbBlue  = (BYTE)(trans_color->blue>>nshift);
+ #else
+-			info.nBkgndColor.rgbRed   = (BYTE)(info_ptr->trans_values.red>>nshift);
+-			info.nBkgndColor.rgbGreen = (BYTE)(info_ptr->trans_values.green>>nshift);
+-			info.nBkgndColor.rgbBlue  = (BYTE)(info_ptr->trans_values.blue>>nshift);
++			info.nBkgndColor.rgbRed   = (BYTE)(trans_values->red>>nshift);
++			info.nBkgndColor.rgbGreen = (BYTE)(trans_values->green>>nshift);
++			info.nBkgndColor.rgbBlue  = (BYTE)(trans_values->blue>>nshift);
+ #endif
+ 			info.nBkgndColor.rgbReserved = 0;
+ 			info.nBkgndIndex = 0;
+ 		}
+ 	}
+ 
++
+ 	int alpha_present = (channels - 1) % 2;
+ 	if (alpha_present){
+ #if CXIMAGE_SUPPORT_ALPHA	// <vho>
+@@ -202,15 +211,15 @@
+ 	}
+ 
+ 	// <vho> - flip the RGB pixels to BGR (or RGBA to BGRA)
+-	if (info_ptr->color_type & PNG_COLOR_MASK_COLOR){
++	if (png_get_color_type(png_ptr, info_ptr) & PNG_COLOR_MASK_COLOR){
+ 		png_set_bgr(png_ptr);
+ 	}
+ 
+ 	// <vho> - handle cancel
+-	if (info.nEscape) longjmp(png_ptr->jmpbuf, 1);
++	if (info.nEscape) longjmp(png_jmpbuf(png_ptr), 1);
+ 
+ 	// row_bytes is the width x number of channels x (bit-depth / 8)
+-	row_pointers = new BYTE[info_ptr->rowbytes + 8];
++	row_pointers = new BYTE[png_get_rowbytes(png_ptr,info_ptr) + 8];
+ 
+ 	// turn on interlace handling
+ 	int number_passes = png_set_interlace_handling(png_ptr);
+@@ -221,8 +230,8 @@
+ 		SetCodecOption(0);
+ 	}
+ 
+-	int chan_offset = info_ptr->bit_depth >> 3;
+-	int pixel_offset = info_ptr->pixel_depth >> 3;
++	int chan_offset = png_get_bit_depth(png_ptr, info_ptr) >> 3;
++	int pixel_offset = pixel_depth >> 3;
+ 
+ 	for (int pass=0; pass < number_passes; pass++) {
+ 		iter.Upset();
+@@ -230,7 +239,7 @@
+ 		do	{
+ 
+ 			// <vho> - handle cancel
+-			if (info.nEscape) longjmp(png_ptr->jmpbuf, 1);
++			if (info.nEscape) longjmp(png_jmpbuf(png_ptr), 1);
+ 
+ #if CXIMAGE_SUPPORT_ALPHA	// <vho>
+ 			if (AlphaIsValid()) {
+@@ -241,7 +250,7 @@
+ 				BYTE* prow= iter.GetRow(ay);
+ 
+ 				//recover data from previous scan
+-				if (info_ptr->interlace_type && pass>0 && pass!=7){
++				if (png_get_interlace_type(png_ptr, info_ptr)&& pass>0 && pass!=7){
+ 					for(ax=0;ax<head.biWidth;ax++){
+ 						long px = ax * pixel_offset;
+ 						if (channels == 2){
+@@ -278,10 +287,10 @@
+ #endif // CXIMAGE_SUPPORT_ALPHA		// vho
+ 			{
+ 				//recover data from previous scan
+-				if (info_ptr->interlace_type && pass>0){
+-					iter.GetRow(row_pointers, info_ptr->rowbytes);
++				if (png_get_interlace_type(png_ptr, info_ptr) && pass>0){
++					iter.GetRow(row_pointers, png_get_rowbytes(png_ptr, info_ptr));
+ 					//re-expand buffer for images with bit depth > 8
+-					if (info_ptr->bit_depth > 8){
++					if (png_get_bit_depth(png_ptr, info_ptr) > 8){
+ 						for(long ax=(head.biWidth*channels-1);ax>=0;ax--)
+ 							row_pointers[ax*chan_offset] = row_pointers[ax];
+ 					}
+@@ -291,15 +300,15 @@
+ 				png_read_row(png_ptr, row_pointers, NULL);
+ 
+ 				//shrink 16 bit depth images down to 8 bits
+-				if (info_ptr->bit_depth > 8){
++				if (png_get_bit_depth(png_ptr, info_ptr) > 8){
+ 					for(long ax=0;ax<(head.biWidth*channels);ax++)
+ 						row_pointers[ax] = row_pointers[ax*chan_offset];
+ 				}
+ 
+ 				//copy the pixels
+-				iter.SetRow(row_pointers, info_ptr->rowbytes);
++				iter.SetRow(row_pointers, png_get_rowbytes(png_ptr, info_ptr));
+ 				//<DP> expand 2 bpp images only in the last pass
+-				if (info_ptr->bit_depth==2 && pass==(number_passes-1))
++				if (png_get_bit_depth(png_ptr, info_ptr)==2 && pass==(number_passes-1))
+ 					expand2to4bpp(iter.GetRow());
+ 
+ 				//go on
+@@ -340,6 +349,16 @@
+ 	BYTE trans[256];	//for transparency (don't move)
+ 	png_struct *png_ptr;
+ 	png_info *info_ptr;
++	int channels;
++	int bit_depth;
++	int color_type;
++	png_byte interlace_type = PNG_INTERLACE_NONE;
++	png_uint_32 valid = 0;
++	png_colorp palette = NULL;
++	png_uint_16 num_palette;
++	int num_trans;
++	png_bytep trans_alpha;
++	png_color_16p trans_color;
+ 
+   cx_try
+   {
+@@ -362,9 +381,9 @@
+    /* Set error handling.  REQUIRED if you aren't supplying your own
+     * error hadnling functions in the png_create_write_struct() call.
+     */
+-	if (setjmp(png_ptr->jmpbuf)){
++	if (setjmp(png_jmpbuf(png_ptr))){
+ 		/* If we get here, we had a problem reading the file */
+-		if (info_ptr->palette) free(info_ptr->palette);
++		if (palette) { free(palette); palette = NULL; }
+ 		png_destroy_write_struct(&png_ptr,  (png_infopp)&info_ptr);
+ 		cx_throw("Error saving PNG file");
+ 	}
+@@ -376,20 +395,11 @@
+     png_set_write_fn(png_ptr,hFile,/*(png_rw_ptr)*/user_write_data,/*(png_flush_ptr)*/user_flush_data);
+ 
+ 	/* set the file information here */
+-	info_ptr->width = GetWidth();
+-	info_ptr->height = GetHeight();
+-	info_ptr->pixel_depth = (BYTE)GetBpp();
+-	info_ptr->channels = (GetBpp()>8) ? (BYTE)3: (BYTE)1;
+-	info_ptr->bit_depth = (BYTE)(GetBpp()/info_ptr->channels);
+-	info_ptr->compression_type = info_ptr->filter_type = 0;
+-	info_ptr->valid = 0;
+-
+-	switch(GetCodecOption(CXIMAGE_FORMAT_PNG)){
+-	case 1:
+-		info_ptr->interlace_type = PNG_INTERLACE_ADAM7;
+-		break;
+-	default:
+-		info_ptr->interlace_type = PNG_INTERLACE_NONE;
++	channels = (GetBpp()>8) ? (BYTE)3: (BYTE)1;
++	bit_depth = (BYTE)(GetBpp()/channels);
++
++	if (GetCodecOption(CXIMAGE_FORMAT_PNG) == 1){
++		interlace_type = PNG_INTERLACE_ADAM7;
+ 	}
+ 
+ 	/* set compression level */
+@@ -399,19 +409,18 @@
+ 
+ 	if (GetNumColors()){
+ 		if (bGrayScale){
+-			info_ptr->color_type = PNG_COLOR_TYPE_GRAY;
++			color_type = PNG_COLOR_TYPE_GRAY;
+ 		} else {
+-			info_ptr->color_type = PNG_COLOR_TYPE_PALETTE;
++			color_type = PNG_COLOR_TYPE_PALETTE;
+ 		}
+ 	} else {
+-		info_ptr->color_type = PNG_COLOR_TYPE_RGB;
++		color_type = PNG_COLOR_TYPE_RGB;
+ 	}
+ #if CXIMAGE_SUPPORT_ALPHA
+ 	if (AlphaIsValid()){
+-		info_ptr->color_type |= PNG_COLOR_MASK_ALPHA;
+-		info_ptr->channels++;
+-		info_ptr->bit_depth = 8;
+-		info_ptr->pixel_depth += 8;
++		color_type |= PNG_COLOR_MASK_ALPHA;
++		channels++;
++		bit_depth = 8;
+ 	}
+ #endif
+ 
+@@ -428,28 +437,27 @@
+ 	/* set metrics */
+ 	png_set_pHYs(png_ptr, info_ptr, head.biXPelsPerMeter, head.biYPelsPerMeter, PNG_RESOLUTION_METER);
+ 
+-	png_set_IHDR(png_ptr, info_ptr, info_ptr->width, info_ptr->height, info_ptr->bit_depth,
+-				info_ptr->color_type, info_ptr->interlace_type,
++	png_set_IHDR(png_ptr, info_ptr, GetWidth(), GetHeight(), bit_depth,
++				color_type, interlace_type,
+ 				PNG_COMPRESSION_TYPE_BASE, PNG_FILTER_TYPE_BASE);
+ 
+ 	//<DP> simple transparency
+ 	if (info.nBkgndIndex >= 0){
+-		info_ptr->num_trans = 1;
+-		info_ptr->valid |= PNG_INFO_tRNS;
++		num_trans = 1;
++		valid |= PNG_INFO_tRNS;
+ #if PNG_LIBPNG_VER_MAJOR >= 1 && PNG_LIBPNG_VER_MINOR >= 4
+-		info_ptr->trans_alpha = trans;
+-		info_ptr->trans_color.index = (BYTE)info.nBkgndIndex;
+-		info_ptr->trans_color.red   = tc.rgbRed;
+-		info_ptr->trans_color.green = tc.rgbGreen;
+-		info_ptr->trans_color.blue  = tc.rgbBlue;
+-		info_ptr->trans_color.gray  = info_ptr->trans_color.index;
++		trans_alpha = trans;
++		trans_color->index = (BYTE)info.nBkgndIndex;
++		trans_color->red   = tc.rgbRed;
++		trans_color->green = tc.rgbGreen;
++		trans_color->blue  = tc.rgbBlue;
++		trans_color->gray  = (BYTE)info.nBkgndIndex;
+ #else
+-		info_ptr->trans = trans;
+-		info_ptr->trans_values.index = (BYTE)info.nBkgndIndex;
+-		info_ptr->trans_values.red   = tc.rgbRed;
+-		info_ptr->trans_values.green = tc.rgbGreen;
+-		info_ptr->trans_values.blue  = tc.rgbBlue;
+-		info_ptr->trans_values.gray  = info_ptr->trans_values.index;
++		trans_values->index = (BYTE)info.nBkgndIndex;
++		trans_values->red   = tc.rgbRed;
++		trans_values->green = tc.rgbGreen;
++		trans_values->blue  = tc.rgbBlue;
++		trans_values->gray  = (BYTE)info.nBkgndIndex;
+ #endif
+ 
+ 		// the transparency indexes start from 0 for non grayscale palette
+@@ -460,7 +468,7 @@
+ 	/* set the palette if there is one */
+ 	if (GetPalette()){
+ 		if (!bGrayScale){
+-			info_ptr->valid |= PNG_INFO_PLTE;
++			valid |= PNG_INFO_PLTE;
+ 		}
+ 
+ 		int nc = GetClrImportant();
+@@ -469,20 +477,18 @@
+ 		if (info.bAlphaPaletteEnabled){
+ 			for(WORD ip=0; ip<nc;ip++)
+ 				trans[ip]=GetPaletteColor((BYTE)ip).rgbReserved;
+-			info_ptr->num_trans = (WORD)nc;
+-			info_ptr->valid |= PNG_INFO_tRNS;
++			num_trans = (WORD)nc;
++			valid |= PNG_INFO_tRNS;
+ #if PNG_LIBPNG_VER_MAJOR >= 1 && PNG_LIBPNG_VER_MINOR >= 4
+-			info_ptr->trans_alpha = trans;
+-#else
+-			info_ptr->trans = trans;
++			trans_alpha = trans;
+ #endif
+ 		}
+ 
+ 		// copy the palette colors
+-		info_ptr->palette = new png_color[nc];
+-		info_ptr->num_palette = (png_uint_16) nc;
++		palette = new png_color[nc];
++		num_palette = (png_uint_16) nc;
+ 		for (int i=0; i<nc; i++)
+-			GetPaletteColor(i, &info_ptr->palette[i].red, &info_ptr->palette[i].green, &info_ptr->palette[i].blue);
++			GetPaletteColor(i, &palette[i].red, &palette[i].green, &palette[i].blue);
+ 	}  
+ 
+ #if CXIMAGE_SUPPORT_ALPHA	// <vho>
+@@ -496,8 +502,7 @@
+ 	}	}	}
+ #endif // CXIMAGE_SUPPORT_ALPHA	// <vho>
+ 
+-	int row_size = max(info.dwEffWidth, info_ptr->width*info_ptr->channels*(info_ptr->bit_depth/8));
+-	info_ptr->rowbytes = row_size;
++	int row_size = max(info.dwEffWidth, GetWidth()*channels*(bit_depth/8));
+ 	BYTE *row_pointers = new BYTE[row_size];
+ 
+ 	/* write the file information */
+@@ -515,7 +520,7 @@
+ 			if (AlphaIsValid()){
+ 				for (long ax=head.biWidth-1; ax>=0;ax--){
+ 					c = BlindGetPixelColor(ax,ay);
+-					int px = ax * info_ptr->channels;
++					int px = ax * channels;
+ 					if (!bGrayScale){
+ 						row_pointers[px++]=c.rgbRed;
+ 						row_pointers[px++]=c.rgbGreen;
+@@ -530,7 +535,7 @@
+ #endif //CXIMAGE_SUPPORT_ALPHA	// <vho>
+ 			{
+ 				iter.GetRow(row_pointers, row_size);
+-				if (info_ptr->color_type == PNG_COLOR_TYPE_RGB) //HACK BY OP
++				if (color_type == PNG_COLOR_TYPE_RGB) //HACK BY OP
+ 					RGBtoBGR(row_pointers, row_size);
+ 				png_write_row(png_ptr, row_pointers);
+ 			}
+@@ -548,9 +553,9 @@
+ 	png_write_end(png_ptr, info_ptr);
+ 
+ 	/* if you malloced the palette, free it here */
+-	if (info_ptr->palette){
+-		delete [] (info_ptr->palette);
+-		info_ptr->palette = NULL;
++	if (palette){
++		delete [] (palette);
++		palette = NULL;
+ 	}
+ 
+ 	/* clean up after the write, and free any memory allocated */
+--- amsn/utils/TkCximage/src/CxImage/ximapng.h	2012/01/14 15:27:06	12356
++++ amsn/utils/TkCximage/src/CxImage/ximapng.h	2012/01/24 21:21:23	12357
+@@ -64,8 +64,8 @@
+ 
+     static void PNGAPI user_error_fn(png_structp png_ptr,png_const_charp error_msg)
+ 	{
+-		strncpy((char*)png_ptr->error_ptr,error_msg,255);
+-		longjmp(png_ptr->jmpbuf, 1);
++		strncpy((char*)png_get_error_ptr(png_ptr),error_msg,255);
++		longjmp(png_jmpbuf(png_ptr), 1);
+ 	}
+ };
+ 
diff --git a/amsn.spec b/amsn.spec
index ad82a75..a4195ec 100644
--- a/amsn.spec
+++ b/amsn.spec
@@ -1,27 +1,28 @@
 %{!?tcl_version: %define tcl_version 8.5}
 %{!?tcl_sitearch: %define tcl_sitearch %{_libdir}/tcl%{tcl_version}}
 
-Name:           amsn
-Version:        0.98.4
-Release:        7%{?dist}
-Summary:        MSN Messenger clone for Linux, Mac and Windows
-
-Group:          Applications/Internet
-License:        GPLv2
-URL:            http://www.amsn-project.net/
-Source0:        http://downloads.sourceforge.net/%{name}/%{name}-%{version}-src.tar.bz2
+Name:		amsn
+Version:	0.98.4
+Release:	8%{?dist}
+Summary:	MSN Messenger clone for Linux, Mac and Windows
+
+Group:		Applications/Internet
+License:	GPLv2
+URL:		http://www.amsn-project.net/
+Source0:	http://downloads.sourceforge.net/%{name}/%{name}-%{version}-src.tar.bz2
 Patch0:		amsn-0.98.1-usesnack.patch
 Patch1:		amsn-0.98.4-useV4L2.patch
-BuildRoot:      %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
-
-BuildRequires:  desktop-file-utils, tk-devel, which, libpng-devel, libjpeg-devel
-BuildRequires:  farsight2-devel
-BuildRequires:  gupnp-igd-devel
-BuildRequires:  libv4l-devel
-BuildRequires:  tcl(abi) = %{tcl_version}
-Requires:       tcl(abi) = %{tcl_version}
+Patch2:		amsn-0.98.4-libpng15.patch
+BuildRoot:	%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
+
+BuildRequires:	desktop-file-utils, tk-devel, which, libpng-devel, libjpeg-devel
+BuildRequires:	farsight2-devel
+BuildRequires:	gupnp-igd-devel
+BuildRequires:	libv4l-devel
+BuildRequires:	tcl(abi) = %{tcl_version}
+Requires:	tcl(abi) = %{tcl_version}
 Requires:	tcl-snack
-Requires:       tcltls, tclsoap, bwidget, tkdnd, xdg-utils
+Requires:	tcltls, tclsoap, bwidget, tkdnd, xdg-utils
 Requires:	tktray
 
 %description
@@ -43,6 +44,7 @@ snapshots with your webcam to use as your display picture.
 %setup -q
 %patch0 -p1
 %patch1 -p1
+%patch2 -p1
 
 rm -r utils/BWidget-1.9.0
 rm -r skins/default/winicons
@@ -96,10 +98,10 @@ rm ${RPM_BUILD_ROOT}%{_datadir}/pixmaps/%{name}.png
 chmod 755 ${RPM_BUILD_ROOT}%{_datadir}/%{name}/skins/Dark\ Matter\ 4.0/pixmapscroll/test.tcl
 chmod 755 ${RPM_BUILD_ROOT}%{_datadir}/%{name}/utils/voipcontrols/test.tcl
 
-desktop-file-install --vendor="fedora"                          \
-        --dir="${RPM_BUILD_ROOT}%{_datadir}/applications"       \
-        --delete-original                                       \
-        ${RPM_BUILD_ROOT}%{_datadir}/applications/amsn.desktop
+desktop-file-install --vendor="fedora"				\
+	--dir="${RPM_BUILD_ROOT}%{_datadir}/applications"	\
+	--delete-original					\
+	${RPM_BUILD_ROOT}%{_datadir}/applications/amsn.desktop
 
 %clean
 rm -rf ${RPM_BUILD_ROOT}
@@ -135,6 +137,9 @@ fi
 
 
 %changelog
+* Fri Jan 27 2012 Sander Hoentjen <sander at hoentjen.eu> 0.98.4-8
+- Fix compile against new libpng (backport http://amsn.svn.sourceforge.net/viewvc/amsn?view=revision&revision=12357)
+
 * Tue Jan 17 2012 Brian Pepple <bpepple at fedoraproject.org> - 0.98.4-7
 - Rebuild for new gupnp-idg.
 


More information about the scm-commits mailing list