// declarations
// ============================================================================
-#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)
-#pragma implementation "imagpng.h"
-#endif
-
// ----------------------------------------------------------------------------
// headers
// ----------------------------------------------------------------------------
#include "wx/wxprec.h"
#ifdef __BORLANDC__
- #pragma hdrstop
-#endif
-
-#ifndef WX_PRECOMP
- #include "wx/defs.h"
+ #pragma hdrstop
#endif
#if wxUSE_IMAGE && wxUSE_LIBPNG
#include "wx/imagpng.h"
-#include "wx/bitmap.h"
-#include "wx/debug.h"
-#include "wx/log.h"
-#include "wx/app.h"
+
+#ifndef WX_PRECOMP
+ #include "wx/log.h"
+ #include "wx/app.h"
+ #include "wx/bitmap.h"
+ #include "wx/module.h"
+#endif
+
#include "png.h"
#include "wx/filefn.h"
#include "wx/wfstream.h"
#include "wx/intl.h"
-#include "wx/module.h"
+#include "wx/palette.h"
// For memcpy
#include <string.h>
WX_PNG_INFO(png_ptr)->stream.out->Write(data, length);
}
-// from pngerror.c
-// so that the libpng doesn't send anything on stderr
-void
-PNGLINKAGEMODE wx_png_error(png_structp WXUNUSED(png_ptr), png_const_charp message)
-{
- wxLogFatalError( wxString::FromAscii(message) );
-}
-
void
PNGLINKAGEMODE wx_png_warning(png_structp png_ptr, png_const_charp message)
{
wxLogWarning( wxString::FromAscii(message) );
}
+// from pngerror.c
+// so that the libpng doesn't send anything on stderr
+void
+PNGLINKAGEMODE wx_png_error(png_structp png_ptr, png_const_charp message)
+{
+ wx_png_warning(NULL, message);
+
+ // we're not using libpng built-in jump buffer (see comment before
+ // wxPNGInfoStruct above) so we have to return ourselves, otherwise libpng
+ // would just abort
+ longjmp(WX_PNG_INFO(png_ptr)->jmpbuf, 1);
+}
+
} // extern "C"
// ----------------------------------------------------------------------------
r2 = *p++;
g2 = *p++;
b2 = *p++;
+ ++p; // jump over alpha
wxImageHistogramEntry&
entry = h[wxImageHistogram:: MakeKey(r2, g2, b2)];
if ( IsTransparent(a) )
{
*ptrDst++ = rMask;
- *ptrDst++ = bMask;
*ptrDst++ = gMask;
+ *ptrDst++ = bMask;
break;
}
// else: !transparent
if ( IsTransparent(a) )
{
*ptrDst++ = rMask;
- *ptrDst++ = bMask;
*ptrDst++ = gMask;
+ *ptrDst++ = bMask;
break;
}
else // !transparent
bool verbose,
int WXUNUSED(index))
{
- // VZ: as this function uses setjmp() the only fool proof error handling
+ // VZ: as this function uses setjmp() the only fool-proof error handling
// method is to use goto (setjmp is not really C++ dtors friendly...)
unsigned char **lines = NULL;
png_infop info_ptr = (png_infop) NULL;
wxPNGInfoStruct wxinfo;
+ png_uint_32 i, width, height = 0;
+ int bit_depth, color_type, interlace_type;
+
wxinfo.verbose = verbose;
wxinfo.stream.in = &stream;
if (setjmp(wxinfo.jmpbuf))
goto error;
- png_uint_32 i, width, height;
- int bit_depth, color_type, interlace_type;
-
png_read_info( png_ptr, info_ptr );
png_get_IHDR( png_ptr, info_ptr, &width, &height, &bit_depth, &color_type, &interlace_type, (int*) NULL, (int*) NULL );
png_read_image( png_ptr, lines );
png_read_end( png_ptr, info_ptr );
+
+#if wxUSE_PALETTE
+ if (color_type == PNG_COLOR_TYPE_PALETTE)
+ {
+ const size_t ncolors = info_ptr->num_palette;
+ unsigned char* r = new unsigned char[ncolors];
+ unsigned char* g = new unsigned char[ncolors];
+ unsigned char* b = new unsigned char[ncolors];
+
+ for (size_t j = 0; j < ncolors; j++)
+ {
+ r[j] = info_ptr->palette[j].red;
+ g[j] = info_ptr->palette[j].green;
+ b[j] = info_ptr->palette[j].blue;
+ }
+
+ image->SetPalette(wxPalette(ncolors, r, g, b));
+ delete[] r;
+ delete[] g;
+ delete[] b;
+ }
+#endif // wxUSE_PALETTE
+
png_destroy_read_struct( &png_ptr, &info_ptr, (png_infopp) NULL );
// loaded successfully, now init wxImage with this data
if ( lines )
{
+ for ( unsigned int n = 0; n < height; n++ )
+ free( lines[n] );
+
free( lines );
}
iElements++;
}
+ if ( iBitDepth == 16 )
+ iElements *= 2;
+
png_set_sBIT( png_ptr, info_ptr, &sig_bit );
png_write_info( png_ptr, info_ptr );
png_set_shift( png_ptr, &sig_bit );
int iHeight = image->GetHeight();
int iWidth = image->GetWidth();
- unsigned char uchMaskRed = bHasMask ? image->GetMaskRed() : 0;
- unsigned char uchMaskGreen = bHasMask ? image->GetMaskGreen() : 0;
- unsigned char uchMaskBlue = bHasMask ? image->GetMaskBlue() : 0;
+ unsigned char uchMaskRed = 0, uchMaskGreen = 0, uchMaskBlue = 0;
+
+ if ( bHasMask )
+ {
+ uchMaskRed = image->GetMaskRed();
+ uchMaskGreen = image->GetMaskGreen();
+ uchMaskBlue = image->GetMaskBlue();
+ }
+
unsigned char *pColors = image->GetData();
for (int y = 0; y != iHeight; ++y)
case wxPNG_TYPE_COLOUR:
*pData++ = uchRed;
- if (iBitDepth > 8)
+ if ( iBitDepth == 16 )
*pData++ = 0;
*pData++ = uchGreen;
- if (iBitDepth > 8)
+ if ( iBitDepth == 16 )
*pData++ = 0;
*pData++ = uchBlue;
- if (iBitDepth > 8)
+ if ( iBitDepth == 16 )
*pData++ = 0;
break;
case wxPNG_TYPE_GREY:
{
+ // where do these coefficients come from? maybe we
+ // should have image options for them as well?
unsigned uiColor =
(unsigned) (76.544*(unsigned)uchRed +
150.272*(unsigned)uchGreen +
36.864*(unsigned)uchBlue);
- uiColor >>= (16 - iBitDepth);
- if (iBitDepth > 8)
- {
- *pData++ = (unsigned char)((uiColor >> 8) & 0xFF);
- *pData++ = (unsigned char)(uiColor & 0xFF);
- } else {
+
+ *pData++ = (unsigned char)((uiColor >> 8) & 0xFF);
+ if ( iBitDepth == 16 )
*pData++ = (unsigned char)(uiColor & 0xFF);
- }
}
break;
case wxPNG_TYPE_GREY_RED:
*pData++ = uchRed;
- if (iBitDepth > 8)
+ if ( iBitDepth == 16 )
*pData++ = 0;
break;
}
}
*pData++ = uchAlpha;
- if (iBitDepth > 8)
+ if ( iBitDepth == 16 )
*pData++ = 0;
}
}