Take basic style into account when applying style sheet

[wxWidgets.git] / src / common / imagxpm.cpp

diff --git a/src/common/imagxpm.cpp b/src/common/imagxpm.cpp
index b637c66808410feb9e3ad29bc716811fb6298207..7d02ebe3724c7c7d9a85ecc54e1203964fc70052 100644 (file)
--- a/src/common/imagxpm.cpp
+++ b/src/common/imagxpm.cpp
@@ -1,7 +1,7 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////

-// Name:        imagxpm.cpp
+// Name:        src/common/imagxpm.cpp

 // Purpose:     wxXPMHandler
 // Purpose:     wxXPMHandler

-// Author:      Vaclav Slavik
+// Author:      Vaclav Slavik, Robert Roebling

 // RCS-ID:      $Id$
 // Copyright:   (c) 2001 Vaclav Slavik
 // Licence:     wxWindows licence
 // RCS-ID:      $Id$
 // Copyright:   (c) 2001 Vaclav Slavik
 // Licence:     wxWindows licence


@@ -62,29 +62,24 @@ 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__
 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
 
 #ifdef __BORLANDC__

-#  pragma hdrstop
+    #pragma hdrstop

 #endif
 
 #endif
 

+#if wxUSE_XPM
+

 #ifndef WX_PRECOMP
 #ifndef WX_PRECOMP

-#  include "wx/defs.h"
+    #include "wx/log.h"
+    #include "wx/intl.h"
+    #include "wx/utils.h"

 #endif
 
 #endif
 

-#if wxUSE_IMAGE && wxUSE_XPM
-

 #include "wx/imagxpm.h"
 #include "wx/wfstream.h"
 #include "wx/imagxpm.h"
 #include "wx/wfstream.h"

-#include "wx/log.h"
-#include "wx/intl.h"
-#include "wx/utils.h"
-
+#include "wx/xpmdecod.h"

 
 IMPLEMENT_DYNAMIC_CLASS(wxXPMHandler,wxImageHandler)
 
 
 IMPLEMENT_DYNAMIC_CLASS(wxXPMHandler,wxImageHandler)
 


@@ -94,101 +89,113 @@ IMPLEMENT_DYNAMIC_CLASS(wxXPMHandler,wxImageHandler)
 
 #if wxUSE_STREAMS
 
 
 #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.Ok() )
+        return false;
+    *image = img;
+    return true;

 }
 
 bool wxXPMHandler::SaveFile(wxImage * image,
 }
 
 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
     // 1. count colours:
     #define MaxCixels  92

-    static const char Cixel[MaxCixels+1] = 
+    static const char Cixel[MaxCixels+1] =

                          " .XoO+@#$%&*=-;:>,<1234567890qwertyuipasdfghjk"
                          "lzxcvbnmMNBVCZASDFGHJKLPIUYTREWQ!~^/()_`'][{}|";
                          " .XoO+@#$%&*=-;:>,<1234567890qwertyuipasdfghjk"
                          "lzxcvbnmMNBVCZASDFGHJKLPIUYTREWQ!~^/()_`'][{}|";

-    int chars_per_pixel;
-    int cols;

     int i, j, k;
     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:
     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) )
+    {
+        wxSplitPath(image->GetOption(wxIMAGE_OPTION_FILENAME),
+                    NULL, &sName, NULL);
+        sName << wxT("_xpm");
+    }
+
+    if ( !sName.empty() )
+        sName = wxString(wxT("/* XPM */\nstatic char *")) + sName;
+    else
+        sName = wxT("/* XPM */\nstatic 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"
                "/* 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);
                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:
 
     // 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:
     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();
     if (image->HasMask())
         mask_key = (image->GetMaskRed() << 16) |
                    (image->GetMaskGreen() << 8) | image->GetMaskBlue();

-    
+

     // 2b. generate colour table:
     // 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];
 
         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';
 
         }
         sym[j] = '\0';
 

-        long key = node->GetKeyInteger();
+        unsigned long key = entry->first;

 
         if (key == 0)
 
         if (key == 0)

-            tmp.Printf(wxT("\"%s c Black\",\n"), sym);
+            sprintf( tmpbuf, "\"%s c Black\",\n", sym);

         else if (key == mask_key)
         else if (key == mask_key)

-            tmp.Printf(wxT("\"%s c None\",\n"), sym);
+            sprintf( tmpbuf, "\"%s c None\",\n", sym);

         else
         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++)
     {
 
     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]);
         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() )
         }
         tmp_c = '\"'; stream.Write(&tmp_c, 1);
         if ( j + 1 < image->GetHeight() )


@@ -197,25 +204,21 @@ bool wxXPMHandler::SaveFile(wxImage * image,
         }
         tmp_c = '\n'; stream.Write(&tmp_c, 1);
     }
         }
         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;
 
     delete[] symbols;
     delete[] symbols_data;
 

-    return TRUE;
+    return true;

 }
 
 bool wxXPMHandler::DoCanRead(wxInputStream& stream)
 {
 }
 
 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);

 }
 
 #endif  // wxUSE_STREAMS
 
 }
 
 #endif  // wxUSE_STREAMS
 

-#endif // wxUSE_XPM && wxUSE_IMAGE
+#endif // wxUSE_XPM