// Licence: wxWindows licence
/////////////////////////////////////////////////////////////////////////////
-#ifdef __GNUG__
+#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)
#pragma implementation "imagbmp.h"
#endif
#include "wx/defs.h"
-#if wxUSE_IMAGE && wxUSE_STREAMS
+#if wxUSE_IMAGE
#include "wx/imagbmp.h"
#include "wx/bitmap.h"
#endif
#ifdef __WXMSW__
-#include <windows.h>
+#include "wx/msw/wrapwin.h"
#endif
//-----------------------------------------------------------------------------
IMPLEMENT_DYNAMIC_CLASS(wxBMPHandler,wxImageHandler)
+#if wxUSE_STREAMS
+
#ifndef BI_RGB
-#define BI_RGB 0
+ #define BI_RGB 0
+#endif
+
+#ifndef BI_RLE8
#define BI_RLE8 1
+#endif
+
+#ifndef BI_RLE4
#define BI_RLE4 2
#endif
wxOutputStream& stream,
bool verbose)
{
- return SaveDib(image, stream, verbose, TRUE/*IsBmp*/, FALSE/*IsMask*/);
+ return SaveDib(image, stream, verbose, true/*IsBmp*/, false/*IsMask*/);
}
bool wxBMPHandler::SaveDib(wxImage *image,
bool IsMask)
{
- wxCHECK_MSG( image, FALSE, _T("invalid pointer in wxBMPHandler::SaveFile") );
+ wxCHECK_MSG( image, false, _T("invalid pointer in wxBMPHandler::SaveFile") );
if ( !image->Ok() )
{
if ( verbose )
wxLogError(_("BMP: Couldn't save invalid image."));
- return FALSE;
+ return false;
}
// get the format of the BMP file to save, else use 24bpp
{
if ( verbose )
wxLogError(_("BMP: wxImage doesn't have own wxPalette."));
- return FALSE;
+ return false;
}
bpp = 8;
palette_size = 256;
{
if (verbose)
wxLogError(_("BMP: Couldn't write the file (Bitmap) header."));
- return FALSE;
+ return false;
}
}
if ( !IsMask )
{
if (verbose)
wxLogError(_("BMP: Couldn't write the file (BitmapInfo) header."));
- return FALSE;
+ return false;
}
}
else if ( (format == wxBMP_8BPP_GREY) || (format == wxBMP_8BPP_RED) ||
(format == wxBMP_1BPP_BW) )
{
- int i;
rgbquad = new wxUint8 [palette_size*4];
- for (i = 0; i < palette_size; i++)
+ for ( int i = 0; i < palette_size; i++ )
{
- // if 1BPP_BW then just 0 and 255 then exit
- if (( i > 0) && (format == wxBMP_1BPP_BW)) i = 255;
- rgbquad[i*4] = i;
- rgbquad[i*4+1] = i;
- rgbquad[i*4+2] = i;
+ // if 1BPP_BW then the value should be either 0 or 255
+ wxUint8 c = (wxUint8)((i > 0) && (format == wxBMP_1BPP_BW) ? 255 : i);
+
+ rgbquad[i*4] =
+ rgbquad[i*4+1] =
+ rgbquad[i*4+2] = c;
rgbquad[i*4+3] = 0;
}
}
delete palette;
#endif // wxUSE_PALETTE
delete q_image;
- return FALSE;
+ return false;
}
}
delete []rgbquad;
{
pixel = 3*(y*width + x);
#if wxUSE_PALETTE
- buffer[x] = palette->GetPixel( data[pixel],
- data[pixel+1],
- data[pixel+2] );
+ buffer[x] = (wxUint8)palette->GetPixel( data[pixel],
+ data[pixel+1],
+ data[pixel+2] );
#else
// FIXME: what should this be? use some std palette maybe?
buffer[x] = 0;
// fill buffer, ignore if > width
#if wxUSE_PALETTE
- buffer[x/2] =
+ buffer[x/2] = (wxUint8)(
((wxUint8)palette->GetPixel(data[pixel],
data[pixel+1],
data[pixel+2]) << 4) |
? 0
: ((wxUint8)palette->GetPixel(data[pixel+3],
data[pixel+4],
- data[pixel+5]) ));
+ data[pixel+5]) )) );
#else
// FIXME: what should this be? use some std palette maybe?
buffer[x/2] = 0;
pixel = 3*(y*width + x);
#if wxUSE_PALETTE
- buffer[x/8] = ((wxUint8)palette->GetPixel(data[pixel], data[pixel+1], data[pixel+2]) << 7) |
+ buffer[x/8] = (wxUint8)(
+ ((wxUint8)palette->GetPixel(data[pixel], data[pixel+1], data[pixel+2]) << 7) |
(((x+1) > width) ? 0 : ((wxUint8)palette->GetPixel(data[pixel+3], data[pixel+4], data[pixel+5]) << 6)) |
(((x+2) > width) ? 0 : ((wxUint8)palette->GetPixel(data[pixel+6], data[pixel+7], data[pixel+8]) << 5)) |
(((x+3) > width) ? 0 : ((wxUint8)palette->GetPixel(data[pixel+9], data[pixel+10], data[pixel+11]) << 4)) |
(((x+4) > width) ? 0 : ((wxUint8)palette->GetPixel(data[pixel+12], data[pixel+13], data[pixel+14]) << 3)) |
(((x+5) > width) ? 0 : ((wxUint8)palette->GetPixel(data[pixel+15], data[pixel+16], data[pixel+17]) << 2)) |
(((x+6) > width) ? 0 : ((wxUint8)palette->GetPixel(data[pixel+18], data[pixel+19], data[pixel+20]) << 1)) |
- (((x+7) > width) ? 0 : ((wxUint8)palette->GetPixel(data[pixel+21], data[pixel+22], data[pixel+23]) ));
+ (((x+7) > width) ? 0 : ((wxUint8)palette->GetPixel(data[pixel+21], data[pixel+22], data[pixel+23]) )) );
#else
// FIXME: what should this be? use some std palette maybe?
buffer[x/8] = 0;
{
pixel = 3*(y*width + x);
- buffer[x/8] =
+ buffer[x/8] = (wxUint8)(
(((wxUint8)(data[pixel] /128.)) << 7) |
(((x+1) > width) ? 0 : (((wxUint8)(data[pixel+3] /128.)) << 6)) |
(((x+2) > width) ? 0 : (((wxUint8)(data[pixel+6] /128.)) << 5)) |
(((x+4) > width) ? 0 : (((wxUint8)(data[pixel+12]/128.)) << 3)) |
(((x+5) > width) ? 0 : (((wxUint8)(data[pixel+15]/128.)) << 2)) |
(((x+6) > width) ? 0 : (((wxUint8)(data[pixel+18]/128.)) << 1)) |
- (((x+7) > width) ? 0 : (((wxUint8)(data[pixel+21]/128.)) ));
+ (((x+7) > width) ? 0 : (((wxUint8)(data[pixel+21]/128.)) )) );
}
}
delete palette;
#endif // wxUSE_PALETTE
delete q_image;
- return FALSE;
+ return false;
}
}
delete[] buffer;
#endif // wxUSE_PALETTE
delete q_image;
- return TRUE;
+ return true;
}
bool wxBMPHandler::DoLoadDib(wxImage * image, int width, int height,
int bpp, int ncolors, int comp,
- off_t bmpOffset, wxInputStream& stream,
+ wxFileOffset bmpOffset, wxInputStream& stream,
bool verbose, bool IsBmp, bool hasPalette)
{
wxInt32 aDword, rmask = 0, gmask = 0, bmask = 0;
int rshift = 0, gshift = 0, bshift = 0;
+ int rbits = 0, gbits = 0, bbits = 0;
wxInt32 dbuf[4];
wxInt8 bbuf[4];
wxUint8 aByte;
wxUint16 aWord;
// allocate space for palette if needed:
- _cmap *cmap = NULL;
+ _cmap *cmap;
if ( bpp < 16 )
{
{
if (verbose)
wxLogError(_("BMP: Couldn't allocate memory."));
- return FALSE;
+ return false;
}
}
else
wxLogError( _("BMP: Couldn't allocate memory.") );
if ( cmap )
delete[] cmap;
- return FALSE;
+ return false;
}
// Reading the palette, if it exists:
else
{
//used in reading .ico file mask
- r[j] = cmap[j].r = j * 255;
- g[j] = cmap[j].g = j * 255;
- b[j] = cmap[j].b = j * 255;
+ r[j] = cmap[j].r =
+ g[j] = cmap[j].g =
+ b[j] = cmap[j].b = ( j ? 255 : 0 );
}
}
{
int bit = 0;
stream.Read(dbuf, 4 * 3);
- bmask = wxINT32_SWAP_ON_BE(dbuf[0]);
+ rmask = wxINT32_SWAP_ON_BE(dbuf[0]);
gmask = wxINT32_SWAP_ON_BE(dbuf[1]);
- rmask = wxINT32_SWAP_ON_BE(dbuf[2]);
- // find shift amount.. ugly, but i can't think of a better way:
- for (bit = 0; bit < bpp; bit++)
+ bmask = wxINT32_SWAP_ON_BE(dbuf[2]);
+ // find shift amount (Least significant bit of mask)
+ for (bit = bpp-1; bit>=0; bit--)
{
if (bmask & (1 << bit))
bshift = bit;
if (rmask & (1 << bit))
rshift = bit;
}
+ // Find number of bits in mask (MSB-LSB+1)
+ for (bit = 0; bit < bpp; bit++)
+ {
+ if (bmask & (1 << bit))
+ bbits = bit-bshift+1;
+ if (gmask & (1 << bit))
+ gbits = bit-gshift+1;
+ if (rmask & (1 << bit))
+ rbits = bit-rshift+1;
+ }
}
else if ( bpp == 16 )
{
rshift = 10;
gshift = 5;
bshift = 0;
+ rbits = 5;
+ gbits = 5;
+ bbits = 5;
}
else if ( bpp == 32 )
{
rshift = 16;
gshift = 8;
bshift = 0;
+ rbits = 8;
+ gbits = 8;
+ bbits = 8;
}
}
ptr = data;
}
- int line = 0;
- int column = 0;
int linesize = ((width * bpp + 31) / 32) * 4;
/* BMPs are stored upside down */
- for ( line = (height - 1); line >= 0; line-- )
+ for ( int line = (height - 1); line >= 0; line-- )
{
int linepos = 0;
- for ( column = 0; column < width; )
+ for ( int column = 0; column < width ; )
{
if ( bpp < 16 )
{
- int index = 0;
linepos++;
aByte = stream.GetC();
if ( bpp == 1 )
{
- int bit = 0;
- for (bit = 0; bit < 8 && column < width; bit++)
+ for (int bit = 0; bit < 8 && column < width; bit++)
{
- index = ((aByte & (0x80 >> bit)) ? 1 : 0);
+ int index = ((aByte & (0x80 >> bit)) ? 1 : 0);
ptr[poffset] = cmap[index].r;
ptr[poffset + 1] = cmap[index].g;
ptr[poffset + 2] = cmap[index].b;
{
if ( comp == BI_RLE4 )
{
- if ( verbose )
- wxLogError(_("DIB Header: Cannot deal with 4bit encoded yet."));
- image->Destroy();
- delete[] cmap;
- return FALSE;
+ wxUint8 first;
+ first = aByte;
+ aByte = stream.GetC();
+ if ( first == 0 )
+ {
+ if ( aByte == 0 )
+ {
+ if ( column > 0 )
+ column = width;
+ }
+ else if ( aByte == 1 )
+ {
+ column = width;
+ line = -1;
+ }
+ else if ( aByte == 2 )
+ {
+ aByte = stream.GetC();
+ column += aByte;
+ linepos = column * bpp / 4;
+ aByte = stream.GetC();
+ line -= aByte; // upside down
+ }
+ else
+ {
+ int absolute = aByte;
+ wxUint8 nibble[2] ;
+ int readBytes = 0 ;
+ for (int k = 0; k < absolute; k++)
+ {
+ if ( !(k % 2 ) )
+ {
+ ++readBytes ;
+ aByte = stream.GetC();
+ nibble[0] = (wxUint8)( (aByte & 0xF0) >> 4 ) ;
+ nibble[1] = (wxUint8)( aByte & 0x0F ) ;
+ }
+ ptr[poffset ] = cmap[nibble[k%2]].r;
+ ptr[poffset + 1] = cmap[nibble[k%2]].g;
+ ptr[poffset + 2] = cmap[nibble[k%2]].b;
+ column++;
+ if ( k % 2 )
+ linepos++;
+ }
+ if ( readBytes & 0x01 )
+ aByte = stream.GetC();
+ }
+ }
+ else
+ {
+ wxUint8 nibble[2] ;
+ nibble[0] = (wxUint8)( (aByte & 0xF0) >> 4 ) ;
+ nibble[1] = (wxUint8)( aByte & 0x0F ) ;
+
+ for ( int l = 0; l < first && column < width; l++ )
+ {
+ ptr[poffset ] = cmap[nibble[l%2]].r;
+ ptr[poffset + 1] = cmap[nibble[l%2]].g;
+ ptr[poffset + 2] = cmap[nibble[l%2]].b;
+ column++;
+ if ( l % 2 )
+ linepos++;
+ }
+ }
}
else
{
- int nibble = 0;
- for (nibble = 0; nibble < 2 && column < width; nibble++)
+ for (int nibble = 0; nibble < 2 && column < width; nibble++)
{
- index = ((aByte & (0xF0 >> nibble * 4)) >> (!nibble * 4));
+ int index = ((aByte & (0xF0 >> nibble * 4)) >> (!nibble * 4));
if ( index >= 16 )
index = 15;
ptr[poffset] = cmap[index].r;
// linepos += size; seems to be wrong, RR
}
}
- }
- else if ( bpp == 24 )
- {
- stream.Read(bbuf, 3);
- linepos += 3;
- ptr[poffset ] = (unsigned char)bbuf[2];
- ptr[poffset + 1] = (unsigned char)bbuf[1];
- ptr[poffset + 2] = (unsigned char)bbuf[0];
- column++;
- }
- else if ( bpp == 16 )
- {
- unsigned char temp;
- stream.Read(&aWord, 2);
- aWord = wxUINT16_SWAP_ON_BE(aWord);
- linepos += 2;
- temp = (aWord & rmask) >> rshift;
- ptr[poffset] = temp;
- temp = (aWord & gmask) >> gshift;
- ptr[poffset + 1] = temp;
- temp = (aWord & bmask) >> bshift;
- ptr[poffset + 2] = temp;
- column++;
- }
- else
- {
- unsigned char temp;
- stream.Read(&aDword, 4);
- aDword = wxINT32_SWAP_ON_BE(aDword);
- linepos += 4;
- temp = (aDword & rmask) >> rshift;
- ptr[poffset] = temp;
- temp = (aDword & gmask) >> gshift;
- ptr[poffset + 1] = temp;
- temp = (aDword & bmask) >> bshift;
- ptr[poffset + 2] = temp;
- column++;
- }
- }
- while ( (linepos < linesize) && (comp != 1) && (comp != 2) )
- {
- stream.Read(&aByte, 1);
- linepos += 1;
- if ( stream.LastError() != wxStream_NOERROR )
- break;
- }
- }
- if (cmap)
- delete[] cmap;
-
- image->SetMask(FALSE);
-
- return stream.IsOk();
+ }
+ else if ( bpp == 24 )
+ {
+ stream.Read(bbuf, 3);
+ linepos += 3;
+ ptr[poffset ] = (unsigned char)bbuf[2];
+ ptr[poffset + 1] = (unsigned char)bbuf[1];
+ ptr[poffset + 2] = (unsigned char)bbuf[0];
+ column++;
+ }
+ else if ( bpp == 16 )
+ {
+ unsigned char temp;
+ stream.Read(&aWord, 2);
+ aWord = wxUINT16_SWAP_ON_BE(aWord);
+ linepos += 2;
+ /* use the masks and calculated amonut of shift
+ to retrieve the color data out of the word. Then
+ shift it left by (8 - number of bits) such that
+ the image has the proper dynamic range */
+ temp = (unsigned char)((aWord & rmask) >> rshift << (8-rbits));
+ ptr[poffset] = temp;
+ temp = (unsigned char)((aWord & gmask) >> gshift << (8-gbits));
+ ptr[poffset + 1] = temp;
+ temp = (unsigned char)((aWord & bmask) >> bshift << (8-bbits));
+ ptr[poffset + 2] = temp;
+ column++;
+ }
+ else
+ {
+ unsigned char temp;
+ stream.Read(&aDword, 4);
+ aDword = wxINT32_SWAP_ON_BE(aDword);
+ linepos += 4;
+ temp = (unsigned char)((aDword & rmask) >> rshift);
+ ptr[poffset] = temp;
+ temp = (unsigned char)((aDword & gmask) >> gshift);
+ ptr[poffset + 1] = temp;
+ temp = (unsigned char)((aDword & bmask) >> bshift);
+ ptr[poffset + 2] = temp;
+ column++;
+ }
+ }
+ while ( (linepos < linesize) && (comp != 1) && (comp != 2) )
+ {
+ stream.Read(&aByte, 1);
+ linepos += 1;
+ if ( !stream )
+ break;
+ }
+ }
+
+ delete[] cmap;
+
+ image->SetMask(false);
+
+ const wxStreamError err = stream.GetLastError();
+ return err == wxSTREAM_NO_ERROR || err == wxSTREAM_EOF;
}
bool wxBMPHandler::LoadDib(wxImage *image, wxInputStream& stream,
wxUint16 aWord;
wxInt32 dbuf[4];
wxInt8 bbuf[4];
- off_t offset;
- offset = 0; // keep gcc quiet
+ wxFileOffset offset = 0; // keep gcc quiet
if ( IsBmp )
{
// read the header off the .BMP format file
offset = stream.TellI();
- if (offset == wxInvalidOffset) offset = 0;
+ if (offset == wxInvalidOffset)
+ offset = 0;
stream.Read(bbuf, 2);
stream.Read(dbuf, 16);
offset = offset + wxINT32_SWAP_ON_BE(dbuf[2]);
stream.Read(dbuf, 4 * 2);
- int width = (int)wxINT32_SWAP_ON_BE(dbuf[0]);
- int height = (int)wxINT32_SWAP_ON_BE(dbuf[1]);
+ int width = wxINT32_SWAP_ON_BE((int)dbuf[0]);
+ int height = wxINT32_SWAP_ON_BE((int)dbuf[1]);
if ( !IsBmp)height = height / 2; // for icons divide by 2
if ( width > 32767 )
{
if (verbose)
wxLogError( _("DIB Header: Image width > 32767 pixels for file.") );
- return FALSE;
+ return false;
}
if ( height > 32767 )
{
if (verbose)
wxLogError( _("DIB Header: Image height > 32767 pixels for file.") );
- return FALSE;
+ return false;
}
stream.Read(&aWord, 2);
int planes = (int)wxUINT16_SWAP_ON_BE( aWord );
*/
stream.Read(&aWord, 2);
- int bpp = (int)wxUINT16_SWAP_ON_BE(aWord);
+ int bpp = wxUINT16_SWAP_ON_BE((int)aWord);
if ( bpp != 1 && bpp != 4 && bpp != 8 && bpp != 16 && bpp != 24 && bpp != 32 )
{
if (verbose)
wxLogError( _("DIB Header: Unknown bitdepth in file.") );
- return FALSE;
+ return false;
}
stream.Read(dbuf, 4 * 4);
- int comp = (int)wxINT32_SWAP_ON_BE(dbuf[0]);
+ int comp = wxINT32_SWAP_ON_BE((int)dbuf[0]);
if ( comp != BI_RGB && comp != BI_RLE4 && comp != BI_RLE8 &&
comp != BI_BITFIELDS )
{
if (verbose)
wxLogError( _("DIB Header: Unknown encoding in file.") );
- return FALSE;
+ return false;
}
stream.Read(dbuf, 4 * 2);
- int ncolors = (int)wxINT32_SWAP_ON_BE( dbuf[0] );
+ int ncolors = wxINT32_SWAP_ON_BE( (int)dbuf[0] );
if (ncolors == 0)
ncolors = 1 << bpp;
/* some more sanity checks */
{
if (verbose)
wxLogError( _("DIB Header: Encoding doesn't match bitdepth.") );
- return FALSE;
+ return false;
}
//read DIB; this is the BMP image or the XOR part of an icon image
if ( !DoLoadDib(image, width, height, bpp, ncolors, comp, offset, stream,
- verbose, IsBmp, TRUE) )
+ verbose, IsBmp, true) )
{
if (verbose)
wxLogError( _("Error in reading image DIB .") );
- return FALSE;
+ return false;
}
if ( !IsBmp )
//there is no palette, so we will create one
wxImage mask;
if ( !DoLoadDib(&mask, width, height, 1, 2, BI_RGB, offset, stream,
- verbose, IsBmp, FALSE) )
+ verbose, IsBmp, false) )
{
if (verbose)
wxLogError( _("ICO: Error in reading mask DIB.") );
- return FALSE;
+ return false;
}
image->SetMaskFromImage(mask, 255, 255, 255);
}
- return TRUE;
+ return true;
}
bool wxBMPHandler::LoadFile(wxImage *image, wxInputStream& stream,
bool verbose, int WXUNUSED(index))
{
// Read a single DIB fom the file:
- return LoadDib(image, stream, verbose, TRUE/*isBmp*/);
+ return LoadDib(image, stream, verbose, true/*isBmp*/);
}
bool wxBMPHandler::DoCanRead(wxInputStream& stream)
{
unsigned char hdr[2];
- stream.Read(hdr, 2);
- stream.SeekI(-2, wxFromCurrent);
- return (hdr[0] == 'B' && hdr[1] == 'M');
+ if ( !stream.Read(hdr, WXSIZEOF(hdr)) )
+ return false;
+
+ // do we have the BMP file signature?
+ return hdr[0] == 'B' && hdr[1] == 'M';
}
+#endif // wxUSE_STREAMS
+
#if wxUSE_ICO_CUR
//-----------------------------------------------------------------------------
IMPLEMENT_DYNAMIC_CLASS(wxICOHandler, wxBMPHandler)
+#if wxUSE_STREAMS
+
struct ICONDIRENTRY
{
wxUint8 bWidth; // Width of the image
bool verbose)
{
- bool bResult = FALSE;
//sanity check; icon must be less than 127 pixels high and 255 wide
if ( image->GetHeight () > 127 )
{
if ( verbose )
wxLogError(_("ICO: Image too tall for an icon."));
- return FALSE;
+ return false;
}
if ( image->GetWidth () > 255 )
{
if ( verbose )
wxLogError(_("ICO: Image too wide for an icon."));
- return FALSE;
+ return false;
}
int images = 1; // only generate one image
ICONDIR IconDir;
IconDir.idReserved = 0;
- IconDir.idType = wxUINT16_SWAP_ON_BE(type);
- IconDir.idCount = wxUINT16_SWAP_ON_BE(images);
+ IconDir.idType = wxUINT16_SWAP_ON_BE((wxUint16)type);
+ IconDir.idCount = wxUINT16_SWAP_ON_BE((wxUint16)images);
stream.Write(&IconDir.idReserved, sizeof(IconDir.idReserved));
stream.Write(&IconDir.idType, sizeof(IconDir.idType));
stream.Write(&IconDir.idCount, sizeof(IconDir.idCount));
{
if ( verbose )
wxLogError(_("ICO: Error writing the image file!"));
- return FALSE;
+ return false;
}
// for each iamage write a description ICONDIRENTRY:
// monochome bitmap:
mask.SetOption(wxIMAGE_OPTION_BMP_FORMAT, wxBMP_1BPP_BW);
- bool IsBmp = FALSE;
- bool IsMask = FALSE;
+ bool IsBmp = false;
+ bool IsMask = false;
//calculate size and offset of image and mask
wxCountingOutputStream cStream;
- bResult = SaveDib(image, cStream, verbose, IsBmp, IsMask);
+ bool bResult = SaveDib(image, cStream, verbose, IsBmp, IsMask);
if ( !bResult )
{
if ( verbose )
wxLogError(_("ICO: Error writing the image file!"));
- return FALSE;
+ return false;
}
- IsMask = TRUE;
+ IsMask = true;
bResult = SaveDib(&mask, cStream, verbose, IsBmp, IsMask);
if ( !bResult )
{
if ( verbose )
wxLogError(_("ICO: Error writing the image file!"));
- return FALSE;
+ return false;
}
wxUint32 Size = cStream.GetSize();
- // wxCountingOutputStream::Ok() always returns TRUE for now and this
+ // wxCountingOutputStream::Ok() always returns true for now and this
// "if" provokes VC++ warnings in optimized build
#if 0
if ( !cStream.Ok() )
{
if ( verbose )
wxLogError(_("ICO: Error writing the image file!"));
- return FALSE;
+ return false;
}
#endif // 0
offset = offset + sizeof(ICONDIRENTRY);
- icondirentry.bWidth = image->GetWidth();
- icondirentry.bHeight = 2 * image->GetHeight();
+ icondirentry.bWidth = (wxUint8)image->GetWidth();
+ icondirentry.bHeight = (wxUint8)(2 * image->GetHeight());
icondirentry.bColorCount = 0;
icondirentry.bReserved = 0;
icondirentry.wPlanes = wxUINT16_SWAP_ON_BE(1);
{
if ( verbose )
wxLogError(_("ICO: Error writing the image file!"));
- return FALSE;
+ return false;
}
// actually save it:
- IsMask = FALSE;
+ IsMask = false;
bResult = SaveDib(image, stream, verbose, IsBmp, IsMask);
if ( !bResult )
{
if ( verbose )
wxLogError(_("ICO: Error writing the image file!"));
- return FALSE;
+ return false;
}
- IsMask = TRUE;
+ IsMask = true;
bResult = SaveDib(&mask, stream, verbose, IsBmp, IsMask);
if ( !bResult )
{
if ( verbose )
wxLogError(_("ICO: Error writing the image file!"));
- return FALSE;
+ return false;
}
} // end of for loop
- return TRUE;
+ return true;
}
bool wxICOHandler::LoadFile(wxImage *image, wxInputStream& stream,
bool wxICOHandler::DoLoadFile(wxImage *image, wxInputStream& stream,
bool WXUNUSED(verbose), int index)
{
- bool bResult = FALSE;
- bool IsBmp = FALSE;
+ bool bResult wxDUMMY_INITIALIZE(false);
+ bool IsBmp = false;
ICONDIR IconDir;
- off_t iPos = stream.TellI();
+ wxFileOffset iPos = stream.TellI();
stream.Read(&IconDir, sizeof(IconDir));
wxUint16 nIcons = wxUINT16_SWAP_ON_BE(IconDir.idCount);
// nType is 1 for Icons, 2 for Cursors:
if ( iSel == wxNOT_FOUND || iSel < 0 || iSel >= nIcons )
{
wxLogError(_("ICO: Invalid icon index."));
- bResult = FALSE;
+ bResult = false;
}
else
{
// seek to selected icon:
pCurrentEntry = pIconDirEntry + iSel;
stream.SeekI(iPos + wxUINT32_SWAP_ON_BE(pCurrentEntry->dwImageOffset), wxFromStart);
- bResult = LoadDib(image, stream, TRUE, IsBmp);
+ bResult = LoadDib(image, stream, true, IsBmp);
bool bIsCursorType = (this->GetType() == wxBITMAP_TYPE_CUR) || (this->GetType() == wxBITMAP_TYPE_ANI);
if ( bResult && bIsCursorType && nType == 2 )
{
int wxICOHandler::GetImageCount(wxInputStream& stream)
{
ICONDIR IconDir;
- off_t iPos = stream.TellI();
+ wxFileOffset iPos = stream.TellI();
stream.SeekI(0);
stream.Read(&IconDir, sizeof(IconDir));
wxUint16 nIcons = wxUINT16_SWAP_ON_BE(IconDir.idCount);
bool wxICOHandler::DoCanRead(wxInputStream& stream)
{
+ stream.SeekI(0);
unsigned char hdr[4];
- off_t iPos = stream.TellI();
- stream.SeekI (0);
- stream.Read(hdr, 4);
- stream.SeekI(iPos);
- //hdr[2] is one for an icon and two for a cursor
- return (hdr[0] == '\0' && hdr[1] == '\0' && hdr[2] == '\1' && hdr[3] == '\0');
+ if ( !stream.Read(hdr, WXSIZEOF(hdr)) )
+ return false;
+
+ // hdr[2] is one for an icon and two for a cursor
+ return hdr[0] == '\0' && hdr[1] == '\0' && hdr[2] == '\1' && hdr[3] == '\0';
}
+#endif // wxUSE_STREAMS
//-----------------------------------------------------------------------------
IMPLEMENT_DYNAMIC_CLASS(wxCURHandler, wxICOHandler)
+#if wxUSE_STREAMS
+
bool wxCURHandler::DoCanRead(wxInputStream& stream)
{
+ stream.SeekI(0);
unsigned char hdr[4];
- off_t iPos = stream.TellI();
- stream.SeekI (0);
- stream.Read(hdr, 4);
- stream.SeekI(iPos);
- //hdr[2] is one for an icon and two for a cursor
- return (hdr[0] == '\0' && hdr[1] == '\0' && hdr[2] == '\2' && hdr[3] == '\0');
+ if ( !stream.Read(hdr, WXSIZEOF(hdr)) )
+ return false;
+
+ // hdr[2] is one for an icon and two for a cursor
+ return hdr[0] == '\0' && hdr[1] == '\0' && hdr[2] == '\2' && hdr[3] == '\0';
}
+#endif // wxUSE_STREAMS
+
//-----------------------------------------------------------------------------
// wxANIHandler
//-----------------------------------------------------------------------------
IMPLEMENT_DYNAMIC_CLASS(wxANIHandler, wxCURHandler)
+#if wxUSE_STREAMS
+
bool wxANIHandler::LoadFile(wxImage *image, wxInputStream& stream,
bool verbose, int index)
{
wxInt32 FCC1, FCC2;
wxUint32 datalen;
- static const char *rifftxt = "RIFF";
- static const char *listtxt = "LIST";
- static const char *icotxt = "icon";
-
- wxInt32 *riff32 = (wxInt32 *) rifftxt;
- wxInt32 *list32 = (wxInt32 *) listtxt;
- wxInt32 *ico32 = (wxInt32 *) icotxt;
+ wxInt32 riff32;
+ memcpy( &riff32, "RIFF", 4 );
+ wxInt32 list32;
+ memcpy( &list32, "LIST", 4 );
+ wxInt32 ico32;
+ memcpy( &ico32, "icon", 4 );
int iIcon = 0;
stream.SeekI(0);
stream.Read(&FCC1, 4);
- if ( FCC1 != *riff32 )
- return FALSE;
+ if ( FCC1 != riff32 )
+ return false;
// we have a riff file:
while (stream.IsOk())
// we always have a data size
stream.Read(&datalen, 4);
datalen = wxINT32_SWAP_ON_BE(datalen) ;
+ //data should be padded to make even number of bytes
+ if (datalen % 2 == 1) datalen ++ ;
//now either data or a FCC
- if ( (FCC1 == *riff32) || (FCC1 == *list32) )
+ if ( (FCC1 == riff32) || (FCC1 == list32) )
{
- stream.Read(&FCC2, 4);
+ stream.Read(&FCC2, 4);
}
else
{
- if (FCC1 == *ico32 && iIcon >= index)
+ if (FCC1 == ico32 && iIcon >= index)
{
return DoLoadFile(image, stream, verbose, -1);
}
else
{
stream.SeekI(stream.TellI() + datalen);
- if ( FCC1 == *ico32 )
+ if ( FCC1 == ico32 )
iIcon ++;
}
}
// try to read next data chunk:
stream.Read(&FCC1, 4);
}
- return FALSE;
+ return false;
}
bool wxANIHandler::DoCanRead(wxInputStream& stream)
{
wxInt32 FCC1, FCC2;
wxUint32 datalen ;
- static const char *rifftxt = "RIFF";
- static const char *listtxt = "LIST";
- static const char *anihtxt = "anih";
- wxInt32 *riff32 = (wxInt32 *) rifftxt;
- wxInt32 *list32 = (wxInt32 *) listtxt;
- wxInt32 *anih32 = (wxInt32 *) anihtxt;
+ wxInt32 riff32;
+ memcpy( &riff32, "RIFF", 4 );
+ wxInt32 list32;
+ memcpy( &list32, "LIST", 4 );
+ wxInt32 ico32;
+ memcpy( &ico32, "icon", 4 );
+ wxInt32 anih32;
+ memcpy( &anih32, "anih", 4 );
stream.SeekI(0);
- stream.Read(&FCC1, 4);
- if ( FCC1 != *riff32 )
- return FALSE;
+ if ( !stream.Read(&FCC1, 4) )
+ return false;
+
+ if ( FCC1 != riff32 )
+ return false;
// we have a riff file:
while ( stream.IsOk() )
{
- if ( FCC1 == *anih32 )
- return TRUE;
+ if ( FCC1 == anih32 )
+ return true;
// we always have a data size:
stream.Read(&datalen, 4);
- datalen = wxINT32_SWAP_ON_BE(datalen) ;
+ datalen = wxINT32_SWAP_ON_BE(datalen) ;
+ //data should be padded to make even number of bytes
+ if (datalen % 2 == 1) datalen ++ ;
// now either data or a FCC:
- if ( (FCC1 == *riff32) || (FCC1 == *list32) )
+ if ( (FCC1 == riff32) || (FCC1 == list32) )
{
- stream.Read(&FCC2, 4);
+ stream.Read(&FCC2, 4);
}
else
{
}
// try to read next data chunk:
- stream.Read(&FCC1, 4);
+ if ( !stream.Read(&FCC1, 4) )
+ {
+ // reading failed -- either EOF or IO error, bail out anyhow
+ return false;
+ }
}
- return FALSE;
+ return false;
}
int wxANIHandler::GetImageCount(wxInputStream& stream)
{
wxInt32 FCC1, FCC2;
wxUint32 datalen ;
- static const char *rifftxt = "RIFF";
- static const char *listtxt = "LIST";
- static const char *anihtxt = "anih";
- wxInt32 *riff32 = (wxInt32 *) rifftxt;
- wxInt32 *list32 = (wxInt32 *) listtxt;
- wxInt32 *anih32 = (wxInt32 *) anihtxt;
+ wxInt32 riff32;
+ memcpy( &riff32, "RIFF", 4 );
+ wxInt32 list32;
+ memcpy( &list32, "LIST", 4 );
+ wxInt32 ico32;
+ memcpy( &ico32, "icon", 4 );
+ wxInt32 anih32;
+ memcpy( &anih32, "anih", 4 );
stream.SeekI(0);
stream.Read(&FCC1, 4);
- if ( FCC1 != *riff32 )
+ if ( FCC1 != riff32 )
return wxNOT_FOUND;
// we have a riff file:
// we always have a data size:
stream.Read(&datalen, 4);
datalen = wxINT32_SWAP_ON_BE(datalen) ;
+ //data should be padded to make even number of bytes
+ if (datalen % 2 == 1) datalen ++ ;
// now either data or a FCC:
- if ( (FCC1 == *riff32) || (FCC1 == *list32) )
- {
- stream.Read(&FCC2, 4);
+ if ( (FCC1 == riff32) || (FCC1 == list32) )
+ {
+ stream.Read(&FCC2, 4);
}
else
{
- if ( FCC1 == *anih32 )
+ if ( FCC1 == anih32 )
{
wxUint32 *pData = new wxUint32[datalen/4];
stream.Read(pData, datalen);
return wxNOT_FOUND;
}
+#endif // wxUSE_STREAMS
+
#endif // wxUSE_ICO_CUR
-#endif // wxUSE_IMAGE && wxUSE_STREAMS
+#endif // wxUSE_IMAGE
projects / wxWidgets.git / blobdiff