X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/c96ea65713a8dd58ae40b10718723b5c92a303e3..0f5378d414cb68caaa535199c1af12dec1720c3d:/src/common/imagxpm.cpp?ds=inline diff --git a/src/common/imagxpm.cpp b/src/common/imagxpm.cpp index b637c66808..17ec2b9143 100644 --- a/src/common/imagxpm.cpp +++ b/src/common/imagxpm.cpp @@ -1,7 +1,7 @@ ///////////////////////////////////////////////////////////////////////////// -// Name: imagxpm.cpp +// Name: src/common/imagxpm.cpp // Purpose: wxXPMHandler -// Author: Vaclav Slavik +// Author: Vaclav Slavik, Robert Roebling // RCS-ID: $Id$ // Copyright: (c) 2001 Vaclav Slavik // Licence: wxWindows licence @@ -62,29 +62,25 @@ license is as follows: % */ -#ifdef __GNUG__ -#pragma implementation "imagxpm.h" -#endif - // For compilers that support precompilation, includes "wx.h". #include "wx/wxprec.h" #ifdef __BORLANDC__ -# pragma hdrstop + #pragma hdrstop #endif +#if wxUSE_XPM + #ifndef WX_PRECOMP -# include "wx/defs.h" + #include "wx/log.h" + #include "wx/intl.h" + #include "wx/utils.h" #endif -#if wxUSE_IMAGE && wxUSE_XPM - #include "wx/imagxpm.h" #include "wx/wfstream.h" -#include "wx/log.h" -#include "wx/intl.h" -#include "wx/utils.h" - +#include "wx/xpmdecod.h" +#include "wx/filename.h" IMPLEMENT_DYNAMIC_CLASS(wxXPMHandler,wxImageHandler) @@ -94,101 +90,152 @@ IMPLEMENT_DYNAMIC_CLASS(wxXPMHandler,wxImageHandler) #if wxUSE_STREAMS -bool wxXPMHandler::LoadFile(wxImage *WXUNUSED(image), - wxInputStream& WXUNUSED(stream), - bool verbose, int WXUNUSED(index)) +bool wxXPMHandler::LoadFile(wxImage *image, + wxInputStream& stream, + bool WXUNUSED(verbose), int WXUNUSED(index)) { - if (verbose) - wxLogDebug(_("XPM: the handler is write-only!")); - return FALSE; + wxXPMDecoder decoder; + + wxImage img = decoder.ReadFile(stream); + if ( !img.IsOk() ) + return false; + *image = img; + return true; } +namespace +{ + +// Make the given string a valid C identifier. +// +// All invalid characters are simply replaced by underscores and underscore is +// also prepended in the beginning if the initial character is not alphabetic. +void +MakeValidCIdent(wxString* str) +{ + const wxChar chUnderscore = wxT('_'); + + for ( wxString::iterator it = str->begin(); it != str->end(); ++it ) + { + const wxChar ch = *it; + if ( wxIsdigit(ch) ) + { + if ( it == str->begin() ) + { + // Identifiers can't start with a digit. + str->insert(0, chUnderscore); // prepend underscore + it = str->begin(); // restart as string changed + continue; + } + } + else if ( !wxIsalpha(ch) && ch != chUnderscore ) + { + // Not a valid character in C identifiers. + *it = chUnderscore; + } + } + + // Double underscores are not allowed in normal C identifiers and are + // useless anyhow. + str->Replace(wxT("__"), wxT("_")); +} + +} // anonymous namespace + bool wxXPMHandler::SaveFile(wxImage * image, - wxOutputStream& stream, bool verbose) + wxOutputStream& stream, bool WXUNUSED(verbose)) { - wxString tmp; - char tmp_c; - // 1. count colours: #define MaxCixels 92 - static const char Cixel[MaxCixels+1] = + static const char Cixel[MaxCixels+1] = " .XoO+@#$%&*=-;:>,<1234567890qwertyuipasdfghjk" "lzxcvbnmMNBVCZASDFGHJKLPIUYTREWQ!~^/()_`'][{}|"; - int chars_per_pixel; - int cols; int i, j, k; - - cols = image->CountColours(); - chars_per_pixel = 1; + + wxImageHistogram histogram; + int cols = int(image->ComputeHistogram(histogram)); + + int chars_per_pixel = 1; for ( k = MaxCixels; cols > k; k *= MaxCixels) chars_per_pixel++; // 2. write the header: - tmp.Printf("/* XPM */\n" - "static char *xpm_data[] = {\n" + wxString sName; + if ( image->HasOption(wxIMAGE_OPTION_FILENAME) ) + { + sName = wxFileName(image->GetOption(wxIMAGE_OPTION_FILENAME)).GetName(); + MakeValidCIdent(&sName); + sName << wxT("_xpm"); + } + + if ( !sName.empty() ) + sName = wxString(wxT("/* XPM */\nstatic const char *")) + sName; + else + sName = wxT("/* XPM */\nstatic const char *xpm_data"); + stream.Write( (const char*) sName.ToAscii(), sName.Len() ); + + char tmpbuf[200]; + // VS: 200b is safe upper bound for anything produced by sprintf below + // (<101 bytes the string, neither %i can expand into more than 10 chars) + sprintf(tmpbuf, + "[] = {\n" "/* columns rows colors chars-per-pixel */\n" - "\"%i %i %i %i\",\n", + "\"%i %i %i %i\",\n", image->GetWidth(), image->GetHeight(), cols, chars_per_pixel); - stream.Write(tmp.mb_str(), tmp.Length()); + stream.Write(tmpbuf, strlen(tmpbuf)); // 3. create color symbols table: - wxHashTable table(wxKEY_INTEGER); - image->ComputeHistogram(table); - char *symbols_data = new char[cols * (chars_per_pixel+1)]; char **symbols = new char*[cols]; // 2a. find mask colour: - long mask_key = -1; + unsigned long mask_key = 0x1000000 /*invalid RGB value*/; if (image->HasMask()) mask_key = (image->GetMaskRed() << 16) | (image->GetMaskGreen() << 8) | image->GetMaskBlue(); - + // 2b. generate colour table: - table.BeginFind(); - wxNode *node = NULL; - while ((node = table.Next()) != NULL) + for (wxImageHistogram::iterator entry = histogram.begin(); + entry != histogram.end(); ++entry ) { - wxHNode *hnode = (wxHNode*) node->GetData(); - long index = hnode->index; + unsigned long index = entry->second.index; symbols[index] = symbols_data + index * (chars_per_pixel+1); char *sym = symbols[index]; - k = index % MaxCixels; - sym[0] = Cixel[k]; - for (j = 1; j < chars_per_pixel; j++) + for (j = 0; j < chars_per_pixel; j++) { - k = ((index - k) / MaxCixels) % MaxCixels; - sym[j] = Cixel[k]; + sym[j] = Cixel[index % MaxCixels]; + index /= MaxCixels; } sym[j] = '\0'; - long key = node->GetKeyInteger(); + unsigned long key = entry->first; if (key == 0) - tmp.Printf(wxT("\"%s c Black\",\n"), sym); + sprintf( tmpbuf, "\"%s c Black\",\n", sym); else if (key == mask_key) - tmp.Printf(wxT("\"%s c None\",\n"), sym); + sprintf( tmpbuf, "\"%s c None\",\n", sym); else - tmp.Printf(wxT("\"%s c #%s%s%s\",\n"), sym, - wxDecToHex((unsigned char)(key >> 16)).c_str(), - wxDecToHex((unsigned char)(key >> 8)).c_str(), - wxDecToHex((unsigned char)(key)).c_str()); - stream.Write(tmp.mb_str(), tmp.Length()); + { + wxByte r = wxByte(key >> 16); + wxByte g = wxByte(key >> 8); + wxByte b = wxByte(key); + sprintf(tmpbuf, "\"%s c #%02X%02X%02X\",\n", sym, r, g, b); + } + stream.Write( tmpbuf, strlen(tmpbuf) ); } - tmp = wxT("/* pixels */\n"); - stream.Write(tmp.mb_str(), tmp.Length()); + stream.Write("/* pixels */\n", 13); unsigned char *data = image->GetData(); for (j = 0; j < image->GetHeight(); j++) { + char tmp_c; tmp_c = '\"'; stream.Write(&tmp_c, 1); for (i = 0; i < image->GetWidth(); i++, data += 3) { unsigned long key = (data[0] << 16) | (data[1] << 8) | (data[2]); - wxHNode *hnode = (wxHNode*) table.Get(key); - stream.Write(symbols[hnode->index], chars_per_pixel); + stream.Write(symbols[histogram[key].index], chars_per_pixel); } tmp_c = '\"'; stream.Write(&tmp_c, 1); if ( j + 1 < image->GetHeight() ) @@ -197,25 +244,22 @@ bool wxXPMHandler::SaveFile(wxImage * image, } tmp_c = '\n'; stream.Write(&tmp_c, 1); } - tmp = wxT("};\n"); - stream.Write(tmp.mb_str(), 3); - + stream.Write("};\n", 3 ); + + // Clean up: delete[] symbols; delete[] symbols_data; - return TRUE; + return true; } bool wxXPMHandler::DoCanRead(wxInputStream& stream) { - unsigned char buf[9]; - - stream.Read(buf, 9); - stream.SeekI(-9, wxFromCurrent); - - return (memcmp(buf, "/* XPM */", 9) == 0); + wxXPMDecoder decoder; + return decoder.CanRead(stream); + // it's ok to modify the stream position here } #endif // wxUSE_STREAMS -#endif // wxUSE_XPM && wxUSE_IMAGE +#endif // wxUSE_XPM