Kevin Kofler ec83e6b
$NetBSD: patch-as,v 1.5 2011/03/25 15:28:26 wiz Exp $
Kevin Kofler ec83e6b
Kevin Kofler ec83e6b
--- src/kernel/qpngio.cpp.orig	2007-02-02 10:01:15.000000000 -0400
Kevin Kofler ec83e6b
+++ src/kernel/qpngio.cpp
Kevin Kofler ec83e6b
@@ -43,6 +43,7 @@
Kevin Kofler ec83e6b
 #include "qiodevice.h"
Kevin Kofler ec83e6b
 
Kevin Kofler ec83e6b
 #include <png.h>
Kevin Kofler ec83e6b
+#include <zlib.h>
Kevin Kofler ec83e6b
 
Kevin Kofler ec83e6b
 
Kevin Kofler ec83e6b
 #ifdef Q_OS_TEMP
Kevin Kofler ec83e6b
@@ -123,9 +124,24 @@ void setup_qt( QImage& image, png_struct
Kevin Kofler ec83e6b
     png_get_IHDR(png_ptr, info_ptr, &width, &height, &bit_depth, &color_type,
Kevin Kofler ec83e6b
 	0, 0, 0);
Kevin Kofler ec83e6b
 
Kevin Kofler ec83e6b
+    png_colorp info_ptr_palette = NULL;
Kevin Kofler ec83e6b
+    int info_ptr_num_palette = 0;
Kevin Kofler ec83e6b
+    if (png_get_valid(png_ptr, info_ptr, PNG_INFO_PLTE)) {
Kevin Kofler ec83e6b
+	png_get_PLTE(png_ptr, info_ptr, &info_ptr_palette, &info_ptr_num_palette);
Kevin Kofler ec83e6b
+    }
Kevin Kofler ec83e6b
+
Kevin Kofler ec83e6b
+    png_bytep info_ptr_trans_alpha = NULL;
Kevin Kofler ec83e6b
+    int info_ptr_num_trans = 0;
Kevin Kofler ec83e6b
+    png_color_16p info_ptr_trans_color = NULL;
Kevin Kofler ec83e6b
+
Kevin Kofler ec83e6b
+    if (png_get_valid(png_ptr, info_ptr, PNG_INFO_tRNS)) {
Kevin Kofler ec83e6b
+	png_get_tRNS(png_ptr, info_ptr, &info_ptr_trans_alpha, &info_ptr_num_trans, &info_ptr_trans_color);
Kevin Kofler ec83e6b
+    }
Kevin Kofler ec83e6b
+
Kevin Kofler ec83e6b
+
Kevin Kofler ec83e6b
     if ( color_type == PNG_COLOR_TYPE_GRAY ) {
Kevin Kofler ec83e6b
 	// Black & White or 8-bit grayscale
Kevin Kofler ec83e6b
-	if ( bit_depth == 1 && info_ptr->channels == 1 ) {
Kevin Kofler ec83e6b
+	if ( bit_depth == 1 && png_get_channels(png_ptr, info_ptr) == 1 ) {
Kevin Kofler ec83e6b
 	    png_set_invert_mono( png_ptr );
Kevin Kofler ec83e6b
 	    png_read_update_info( png_ptr, info_ptr );
Kevin Kofler ec83e6b
 	    if (!image.create( width, height, 1, 2, QImage::BigEndian ))
Kevin Kofler ec83e6b
@@ -159,7 +175,7 @@ void setup_qt( QImage& image, png_struct
Kevin Kofler ec83e6b
 		image.setColor( i, qRgba(c,c,c,0xff) );
Kevin Kofler ec83e6b
 	    }
Kevin Kofler ec83e6b
 	    if ( png_get_valid(png_ptr, info_ptr, PNG_INFO_tRNS) ) {
Kevin Kofler ec83e6b
-		const int g = info_ptr->trans_values.gray;
Kevin Kofler ec83e6b
+		const int g = info_ptr_trans_color->gray;
Kevin Kofler ec83e6b
 		if (g < ncols) {
Kevin Kofler ec83e6b
 		    image.setAlphaBuffer(TRUE);
Kevin Kofler ec83e6b
 		    image.setColor(g, image.color(g) & RGB_MASK);
Kevin Kofler ec83e6b
@@ -168,7 +184,7 @@ void setup_qt( QImage& image, png_struct
Kevin Kofler ec83e6b
 	}
Kevin Kofler ec83e6b
     } else if ( color_type == PNG_COLOR_TYPE_PALETTE
Kevin Kofler ec83e6b
      && png_get_valid(png_ptr, info_ptr, PNG_INFO_PLTE)
Kevin Kofler ec83e6b
-     && info_ptr->num_palette <= 256 )
Kevin Kofler ec83e6b
+     && info_ptr_num_palette <= 256 )
Kevin Kofler ec83e6b
     {
Kevin Kofler ec83e6b
 	// 1-bit and 8-bit color
Kevin Kofler ec83e6b
 	if ( bit_depth != 1 )
Kevin Kofler ec83e6b
@@ -176,28 +192,28 @@ void setup_qt( QImage& image, png_struct
Kevin Kofler ec83e6b
 	png_read_update_info( png_ptr, info_ptr );
Kevin Kofler ec83e6b
 	png_get_IHDR(png_ptr, info_ptr,
Kevin Kofler ec83e6b
 	    &width, &height, &bit_depth, &color_type, 0, 0, 0);
Kevin Kofler ec83e6b
-	if (!image.create(width, height, bit_depth, info_ptr->num_palette,
Kevin Kofler ec83e6b
+	if (!image.create(width, height, bit_depth, info_ptr_num_palette,
Kevin Kofler ec83e6b
 	    QImage::BigEndian))
Kevin Kofler ec83e6b
 	    return;
Kevin Kofler ec83e6b
 	int i = 0;
Kevin Kofler ec83e6b
 	if ( png_get_valid(png_ptr, info_ptr, PNG_INFO_tRNS) ) {
Kevin Kofler ec83e6b
 	    image.setAlphaBuffer( TRUE );
Kevin Kofler ec83e6b
-	    while ( i < info_ptr->num_trans ) {
Kevin Kofler ec83e6b
+	    while ( i < info_ptr_num_trans ) {
Kevin Kofler ec83e6b
 		image.setColor(i, qRgba(
Kevin Kofler ec83e6b
-		    info_ptr->palette[i].red,
Kevin Kofler ec83e6b
-		    info_ptr->palette[i].green,
Kevin Kofler ec83e6b
-		    info_ptr->palette[i].blue,
Kevin Kofler ec83e6b
-		    info_ptr->trans[i]
Kevin Kofler ec83e6b
+		    info_ptr_palette[i].red,
Kevin Kofler ec83e6b
+		    info_ptr_palette[i].green,
Kevin Kofler ec83e6b
+		    info_ptr_palette[i].blue,
Kevin Kofler ec83e6b
+		    info_ptr_trans_alpha[i]
Kevin Kofler ec83e6b
 		    )
Kevin Kofler ec83e6b
 		);
Kevin Kofler ec83e6b
 		i++;
Kevin Kofler ec83e6b
 	    }
Kevin Kofler ec83e6b
 	}
Kevin Kofler ec83e6b
-	while ( i < info_ptr->num_palette ) {
Kevin Kofler ec83e6b
+	while ( i < info_ptr_num_palette ) {
Kevin Kofler ec83e6b
 	    image.setColor(i, qRgba(
Kevin Kofler ec83e6b
-		info_ptr->palette[i].red,
Kevin Kofler ec83e6b
-		info_ptr->palette[i].green,
Kevin Kofler ec83e6b
-		info_ptr->palette[i].blue,
Kevin Kofler ec83e6b
+		info_ptr_palette[i].red,
Kevin Kofler ec83e6b
+		info_ptr_palette[i].green,
Kevin Kofler ec83e6b
+		info_ptr_palette[i].blue,
Kevin Kofler ec83e6b
 		0xff
Kevin Kofler ec83e6b
 		)
Kevin Kofler ec83e6b
 	    );
Kevin Kofler ec83e6b
@@ -284,7 +300,7 @@ void read_png_image(QImageIO* iio)
Kevin Kofler ec83e6b
 	return;
Kevin Kofler ec83e6b
     }
Kevin Kofler ec83e6b
 
Kevin Kofler ec83e6b
-    if (setjmp(png_ptr->jmpbuf)) {
Kevin Kofler ec83e6b
+    if (setjmp(png_jmpbuf(png_ptr))) {
Kevin Kofler ec83e6b
 	png_destroy_read_struct(&png_ptr, &info_ptr, &end_info);
Kevin Kofler ec83e6b
 	iio->setStatus(-4);
Kevin Kofler ec83e6b
 	return;
Kevin Kofler ec83e6b
@@ -469,7 +485,7 @@ bool QPNGImageWriter::writeImage(const Q
Kevin Kofler ec83e6b
 	return FALSE;
Kevin Kofler ec83e6b
     }
Kevin Kofler ec83e6b
 
Kevin Kofler ec83e6b
-    if (setjmp(png_ptr->jmpbuf)) {
Kevin Kofler ec83e6b
+    if (setjmp(png_jmpbuf(png_ptr))) {
Kevin Kofler ec83e6b
 	png_destroy_write_struct(&png_ptr, &info_ptr);
Kevin Kofler ec83e6b
 	return FALSE;
Kevin Kofler ec83e6b
     }
Kevin Kofler ec83e6b
@@ -491,10 +507,16 @@ bool QPNGImageWriter::writeImage(const Q
Kevin Kofler ec83e6b
 
Kevin Kofler ec83e6b
     png_set_write_fn(png_ptr, (void*)this, qpiw_write_fn, qpiw_flush_fn);
Kevin Kofler ec83e6b
 
Kevin Kofler ec83e6b
+#warning XXXtnn not too sure about this
Kevin Kofler ec83e6b
+/* 
Kevin Kofler ec83e6b
+according to png.h, channels is only used on read, not writes, so we
Kevin Kofler ec83e6b
+should be able to comment this out.
Kevin Kofler ec83e6b
+
Kevin Kofler ec83e6b
     info_ptr->channels =
Kevin Kofler ec83e6b
 	(image.depth() == 32)
Kevin Kofler ec83e6b
 	    ? (image.hasAlphaBuffer() ? 4 : 3)
Kevin Kofler ec83e6b
 	    : 1;
Kevin Kofler ec83e6b
+*/
Kevin Kofler ec83e6b
 
Kevin Kofler ec83e6b
     png_set_IHDR(png_ptr, info_ptr, image.width(), image.height(),
Kevin Kofler ec83e6b
 	image.depth() == 1 ? 1 : 8 /* per channel */,
Kevin Kofler ec83e6b
@@ -504,11 +526,12 @@ bool QPNGImageWriter::writeImage(const Q
Kevin Kofler ec83e6b
 		: PNG_COLOR_TYPE_RGB
Kevin Kofler ec83e6b
 	    : PNG_COLOR_TYPE_PALETTE, 0, 0, 0);
Kevin Kofler ec83e6b
 
Kevin Kofler ec83e6b
+    png_color_8 sig_bit;
Kevin Kofler ec83e6b
+    sig_bit.red = 8;
Kevin Kofler ec83e6b
+    sig_bit.green = 8;
Kevin Kofler ec83e6b
+    sig_bit.blue = 8;
Kevin Kofler ec83e6b
+    png_set_sBIT(png_ptr, info_ptr, &sig_bit);
Kevin Kofler ec83e6b
 
Kevin Kofler ec83e6b
-    //png_set_sBIT(png_ptr, info_ptr, 8);
Kevin Kofler ec83e6b
-    info_ptr->sig_bit.red = 8;
Kevin Kofler ec83e6b
-    info_ptr->sig_bit.green = 8;
Kevin Kofler ec83e6b
-    info_ptr->sig_bit.blue = 8;
Kevin Kofler ec83e6b
 
Kevin Kofler ec83e6b
     if (image.depth() == 1 && image.bitOrder() == QImage::LittleEndian)
Kevin Kofler ec83e6b
        png_set_packswap(png_ptr);
Kevin Kofler ec83e6b
@@ -522,11 +545,14 @@ bool QPNGImageWriter::writeImage(const Q
Kevin Kofler ec83e6b
 	png_set_PLTE(png_ptr, info_ptr, palette, num_palette);
Kevin Kofler ec83e6b
 	int* trans = new int[num_palette];
Kevin Kofler ec83e6b
 	int num_trans = 0;
Kevin Kofler ec83e6b
+	png_colorp info_ptr_palette = NULL;
Kevin Kofler ec83e6b
+	int tmp;
Kevin Kofler ec83e6b
+	png_get_PLTE(png_ptr, info_ptr, &info_ptr_palette, &tmp);
Kevin Kofler ec83e6b
 	for (int i=0; i
Kevin Kofler ec83e6b
 	    QRgb rgb=image.color(i);
Kevin Kofler ec83e6b
-	    info_ptr->palette[i].red = qRed(rgb);
Kevin Kofler ec83e6b
-	    info_ptr->palette[i].green = qGreen(rgb);
Kevin Kofler ec83e6b
-	    info_ptr->palette[i].blue = qBlue(rgb);
Kevin Kofler ec83e6b
+	    info_ptr_palette[i].red = qRed(rgb);
Kevin Kofler ec83e6b
+	    info_ptr_palette[i].green = qGreen(rgb);
Kevin Kofler ec83e6b
+	    info_ptr_palette[i].blue = qBlue(rgb);
Kevin Kofler ec83e6b
 	    if (image.hasAlphaBuffer()) {
Kevin Kofler ec83e6b
 		trans[i] = rgb >> 24;
Kevin Kofler ec83e6b
 		if (trans[i] < 255) {
Kevin Kofler ec83e6b
@@ -534,6 +560,7 @@ bool QPNGImageWriter::writeImage(const Q
Kevin Kofler ec83e6b
 		}
Kevin Kofler ec83e6b
 	    }
Kevin Kofler ec83e6b
 	}
Kevin Kofler ec83e6b
+	png_set_PLTE(png_ptr, info_ptr, info_ptr_palette, num_palette);
Kevin Kofler ec83e6b
 	if (num_trans) {
Kevin Kofler ec83e6b
 	    copy_trans = new png_byte[num_trans];
Kevin Kofler ec83e6b
 	    for (int i=0; i
Kevin Kofler ec83e6b
@@ -544,7 +571,10 @@ bool QPNGImageWriter::writeImage(const Q
Kevin Kofler ec83e6b
     }
Kevin Kofler ec83e6b
 
Kevin Kofler ec83e6b
     if ( image.hasAlphaBuffer() ) {
Kevin Kofler ec83e6b
-	info_ptr->sig_bit.alpha = 8;
Kevin Kofler ec83e6b
+        png_color_8p sig_bit;
Kevin Kofler ec83e6b
+        png_get_sBIT(png_ptr, info_ptr, &sig_bit);
Kevin Kofler ec83e6b
+        sig_bit->alpha = 8;
Kevin Kofler ec83e6b
+        png_set_sBIT(png_ptr, info_ptr, sig_bit);
Kevin Kofler ec83e6b
     }
Kevin Kofler ec83e6b
 
Kevin Kofler ec83e6b
     // Swap ARGB to RGBA (normal PNG format) before saving on
Kevin Kofler ec83e6b
@@ -1030,7 +1060,7 @@ int QPNGFormat::decode(QImage& img, QIma
Kevin Kofler ec83e6b
 	    return -1;
Kevin Kofler ec83e6b
 	}
Kevin Kofler ec83e6b
 
Kevin Kofler ec83e6b
-	if (setjmp((png_ptr)->jmpbuf)) {
Kevin Kofler ec83e6b
+	if (setjmp(png_jmpbuf(png_ptr))) {
Kevin Kofler ec83e6b
 	    png_destroy_read_struct(&png_ptr, &info_ptr, 0);
Kevin Kofler ec83e6b
 	    image = 0;
Kevin Kofler ec83e6b
 	    return -1;
Kevin Kofler ec83e6b
@@ -1057,7 +1087,7 @@ int QPNGFormat::decode(QImage& img, QIma
Kevin Kofler ec83e6b
 
Kevin Kofler ec83e6b
     if ( !png_ptr ) return 0;
Kevin Kofler ec83e6b
 
Kevin Kofler ec83e6b
-    if (setjmp(png_ptr->jmpbuf)) {
Kevin Kofler ec83e6b
+    if (setjmp(png_jmpbuf(png_ptr))) {
Kevin Kofler ec83e6b
 	png_destroy_read_struct(&png_ptr, &info_ptr, 0);
Kevin Kofler ec83e6b
 	image = 0;
Kevin Kofler ec83e6b
 	state = MovieStart;
Kevin Kofler ec83e6b
@@ -1117,7 +1147,7 @@ void QPNGFormat::end(png_structp png, pn
Kevin Kofler ec83e6b
     consumer->frameDone(QPoint(offx,offy),r);
Kevin Kofler ec83e6b
     consumer->end();
Kevin Kofler ec83e6b
     state = FrameStart;
Kevin Kofler ec83e6b
-    unused_data = (int)png->buffer_size; // Since libpng doesn't tell us
Kevin Kofler ec83e6b
+    unused_data = png_process_data_pause(png, 0);
Kevin Kofler ec83e6b
 }
Kevin Kofler ec83e6b
 
Kevin Kofler ec83e6b
 #ifdef PNG_USER_CHUNKS_SUPPORTED