#ifndef WX_PRECOMP
#include "wx/string.h"
+ #include "wx/log.h"
#endif // WX_PRECOMP
#include "wx/msw/private.h"
#include "wx/app.h"
+#ifndef __WXMICROWIN__
#include "wx/msw/dib.h"
+#endif
+
#include "wx/msw/bitmap.h"
#include "wx/msw/gdiimage.h"
#include "wx/bitmap.h"
-#if wxUSE_XPM_IN_MSW
-# include "wx/xpmhand.h"
-#endif // wxUSE_XPM_IN_MSW
-
#ifdef __WIN16__
# include "wx/msw/curico.h"
#endif // __WIN16__
// private classes
// ----------------------------------------------------------------------------
+#ifndef __WXMICROWIN__
+
// all image handlers are declared/defined in this file because the outside
// world doesn't have to know about them (but only about wxBITMAP_TYPE_XXX ids)
// wxWin macros
// ----------------------------------------------------------------------------
- IMPLEMENT_DYNAMIC_CLASS(wxBMPFileHandler, wxBitmapHandler)
- IMPLEMENT_DYNAMIC_CLASS(wxBMPResourceHandler, wxBitmapHandler)
- IMPLEMENT_DYNAMIC_CLASS(wxICOFileHandler, wxGDIImageHandler)
- IMPLEMENT_DYNAMIC_CLASS(wxICOResourceHandler, wxGDIImageHandler)
+IMPLEMENT_DYNAMIC_CLASS(wxBMPFileHandler, wxBitmapHandler)
+IMPLEMENT_DYNAMIC_CLASS(wxBMPResourceHandler, wxBitmapHandler)
+IMPLEMENT_DYNAMIC_CLASS(wxICOFileHandler, wxObject)
+IMPLEMENT_DYNAMIC_CLASS(wxICOResourceHandler, wxObject)
// ----------------------------------------------------------------------------
// private functions
// ----------------------------------------------------------------------------
static wxSize GetHiconSize(HICON hicon);
+#endif
+ // __MICROWIN__
// ============================================================================
// implementation
void wxGDIImage::InitStandardHandlers()
{
+#ifndef __WXMICROWIN__
AddHandler(new wxBMPResourceHandler);
AddHandler(new wxBMPFileHandler);
-
- // GRG: Add these handlers by default if XPM support is enabled
-
-#if wxUSE_XPM_IN_MSW
- AddHandler(new wxXPMFileHandler);
- AddHandler(new wxXPMDataHandler);
-#endif // wxUSE_XPM_IN_MSW
-
AddHandler(new wxICOResourceHandler);
AddHandler(new wxICOFileHandler);
+#endif
}
+#ifndef __WXMICROWIN__
+
// ----------------------------------------------------------------------------
// wxBitmap handlers
// ----------------------------------------------------------------------------
BITMAP bm;
if ( !::GetObject(GetHbitmapOf(*bitmap), sizeof(BITMAP), (LPSTR) &bm) )
{
- wxLogLastError("GetObject(HBITMAP)");
+ wxLogLastError(wxT("GetObject(HBITMAP)"));
}
data->m_width = bm.bmWidth;
#if wxUSE_IMAGE_LOADING_IN_MSW
wxPalette *palette = NULL;
bool success = wxLoadIntoBitmap(WXSTRINGCAST name, bitmap, &palette) != 0;
+
+#if wxUSE_PALETTE
if ( success && palette )
{
bitmap->SetPalette(*palette);
// it was copied by the bitmap if it was loaded successfully
delete palette;
+#endif // wxUSE_PALETTE
return success;
#else
const wxPalette *pal)
{
#if wxUSE_IMAGE_LOADING_IN_MSW
+
+#if wxUSE_PALETTE
wxPalette *actualPalette = (wxPalette *)pal;
if ( !actualPalette )
actualPalette = bitmap->GetPalette();
+#else
+ wxPalette *actualPalette = NULL;
+#endif // wxUSE_PALETTE
+
return wxSaveBitmap(WXSTRINGCAST name, bitmap, actualPalette) != 0;
#else
return FALSE;
bool wxICOFileHandler::LoadIcon(wxIcon *icon,
const wxString& name,
- long flags,
+ long WXUNUSED(flags),
int desiredWidth, int desiredHeight)
{
#if wxUSE_RESOURCE_LOADING_IN_MSW
wxSize size;
#ifdef __WIN32__
- HICON hicon = ::ExtractIcon(wxGetInstance(), name, 0 /* first */);
+ HICON hicon = NULL;
+
+ // Parse the filename: it may be of the form "filename;n" in order to
+ // specify the nth icon in the file.
+ //
+ // For the moment, ignore the issue of possible semicolons in the
+ // filename.
+ int iconIndex = 0;
+ wxString nameReal(name);
+ wxString strIconIndex = name.AfterLast(wxT(';'));
+ if (strIconIndex != name)
+ {
+ iconIndex = wxAtoi(strIconIndex);
+ nameReal = name.BeforeLast(wxT(';'));
+ }
+
+ // were we asked for a large icon?
+ if ( desiredWidth == ::GetSystemMetrics(SM_CXICON) &&
+ desiredHeight == ::GetSystemMetrics(SM_CYICON) )
+ {
+ // get the specified large icon from file
+ if ( !::ExtractIconEx(nameReal, iconIndex, &hicon, NULL, 1) )
+ {
+ // it is not an error, but it might still be useful to be informed
+ // about it optionally
+ wxLogTrace(_T("iconload"),
+ _T("No large icons found in the file '%s'."),
+ name.c_str());
+ }
+ }
+ else if ( desiredWidth == ::GetSystemMetrics(SM_CXSMICON) &&
+ desiredHeight == ::GetSystemMetrics(SM_CYSMICON) )
+ {
+ // get the specified small icon from file
+ if ( !::ExtractIconEx(nameReal, iconIndex, NULL, &hicon, 1) )
+ {
+ wxLogTrace(_T("iconload"),
+ _T("No small icons found in the file '%s'."),
+ name.c_str());
+ }
+ }
+ //else: not standard size, load below
+
+ if ( !hicon )
+ {
+ // take any (the first one) icon from the file by default
+ hicon = ::ExtractIcon(wxGetInstance(), nameReal, 0 /* first */);
+ }
+
if ( !hicon )
{
wxLogSysError(_T("Failed to load icon from the file '%s'"),
if ( (desiredWidth != -1 && desiredWidth != size.x) ||
(desiredHeight != -1 && desiredHeight != size.y) )
{
- wxLogDebug(_T("Returning FALSE from wxICOFileHandler::Load because "
- "of the size mismatch: actual (%d, %d), "
- "requested (%d, %d)"),
- size.x, size.y,
- desiredWidth, desiredHeight);
+ wxLogTrace(_T("iconload"),
+ _T("Returning FALSE from wxICOFileHandler::Load because of the size mismatch: actual (%d, %d), requested (%d, %d)"),
+ size.x, size.y,
+ desiredWidth, desiredHeight);
::DestroyIcon(hicon);
bool wxICOResourceHandler::LoadIcon(wxIcon *icon,
const wxString& name,
- long flags,
+ long WXUNUSED(flags),
int desiredWidth, int desiredHeight)
{
HICON hicon;
+ // do we need the icon of the specific size or would any icon do?
+ bool hasSize = desiredWidth != -1 || desiredHeight != -1;
+
+ wxASSERT_MSG( !hasSize || (desiredWidth != -1 && desiredHeight != -1),
+ _T("width and height should be either both -1 or not") );
+
+ // try to load the icon from this program first to allow overriding the
+ // standard icons (although why one would want to do it considering that
+ // we already have wxApp::GetStdIcon() is unclear)
+
+ // note that we can't just always call LoadImage() because it seems to do
+ // some icon rescaling internally which results in very ugly 16x16 icons
#if defined(__WIN32__) && !defined(__SC__)
- if ( desiredWidth != -1 && desiredHeight != -1 )
+ if ( hasSize )
{
hicon = (HICON)::LoadImage(wxGetInstance(), name, IMAGE_ICON,
desiredWidth, desiredHeight,
LR_DEFAULTCOLOR);
}
else
-#endif // Win32
+#endif // Win32/!Win32
{
hicon = ::LoadIcon(wxGetInstance(), name);
}
- wxSize size = GetHiconSize(hicon);
- icon->SetSize(size.x, size.y);
-
- // Override the found values with desired values
- if ( desiredWidth > -1 && desiredHeight > -1 )
+ // next check if it's not a standard icon
+ if ( !hicon && !hasSize )
{
- icon->SetSize(desiredWidth, desiredHeight);
+ static const struct
+ {
+ const wxChar *name;
+ LPTSTR id;
+ } stdIcons[] =
+ {
+ { wxT("wxICON_QUESTION"), IDI_QUESTION },
+ { wxT("wxICON_WARNING"), IDI_EXCLAMATION },
+ { wxT("wxICON_ERROR"), IDI_HAND },
+ { wxT("wxICON_INFO"), IDI_ASTERISK },
+ };
+
+ for ( size_t nIcon = 0; !hicon && nIcon < WXSIZEOF(stdIcons); nIcon++ )
+ {
+ if ( name == stdIcons[nIcon].name )
+ {
+ hicon = ::LoadIcon((HINSTANCE)NULL, stdIcons[nIcon].id);
+ }
+ }
}
+ wxSize size = GetHiconSize(hicon);
+ icon->SetSize(size.x, size.y);
+
icon->SetHICON((WXHICON)hicon);
return icon->Ok();
ICONINFO info;
if ( !::GetIconInfo(hicon, &info) )
{
- wxLogLastError("GetIconInfo");
+ wxLogLastError(wxT("GetIconInfo"));
}
else
{
return size;
}
+#endif
+ // __WXMICROWIN__