applied Unicode fixes for mingw32 (patch 429654)

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@10414 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
This commit is contained in:
Vadim Zeitlin
2001-06-03 02:58:56 +00:00
parent 2ec6905c69
commit 161f4f7380
13 changed files with 163 additions and 147 deletions

View File

@@ -66,7 +66,7 @@ WXDLLEXPORT_DATA(extern const wxChar*) wxEmptyString;
// ---------------------------------------------------------------------------- // ----------------------------------------------------------------------------
// Microsoft compiler loves underscores, feed them to it // Microsoft compiler loves underscores, feed them to it
#ifdef __VISUALC__ #if defined( __VISUALC__ ) || defined( __GNUWIN32__ )
// functions // functions
#define wxClose _close #define wxClose _close
#define wxRead _read #define wxRead _read

View File

@@ -60,7 +60,7 @@ public:
virtual ~wxPrinterBase(); virtual ~wxPrinterBase();
virtual wxWindow *CreateAbortWindow(wxWindow *parent, wxPrintout *printout); virtual wxWindow *CreateAbortWindow(wxWindow *parent, wxPrintout *printout);
virtual void ReportError(wxWindow *parent, wxPrintout *printout, char *message); virtual void ReportError(wxWindow *parent, wxPrintout *printout, const wxString& message);
wxPrintDialogData& GetPrintDialogData() const wxPrintDialogData& GetPrintDialogData() const
{ return (wxPrintDialogData&) m_printDialogData; } { return (wxPrintDialogData&) m_printDialogData; }

View File

@@ -144,7 +144,7 @@ static const int wxTREE_HITTEST_ONITEM = wxTREE_HITTEST_ONITEMICON |
wxTREE_HITTEST_ONITEMLABEL; wxTREE_HITTEST_ONITEMLABEL;
// tree ctrl default name // tree ctrl default name
WXDLLEXPORT_DATA(extern const char*) wxTreeCtrlNameStr; WXDLLEXPORT_DATA(extern const wxChar*) wxTreeCtrlNameStr;
// ---------------------------------------------------------------------------- // ----------------------------------------------------------------------------
// wxTreeItemAttr: a structure containing the visual attributes of an item // wxTreeItemAttr: a structure containing the visual attributes of an item

View File

@@ -63,6 +63,11 @@
#define wxHAVE_TCHAR_FUNCTIONS #define wxHAVE_TCHAR_FUNCTIONS
#elif defined(__BORLANDC__) && (__BORLANDC__ >= 0x520) #elif defined(__BORLANDC__) && (__BORLANDC__ >= 0x520)
#define wxHAVE_TCHAR_FUNCTIONS #define wxHAVE_TCHAR_FUNCTIONS
#elif defined(__GNUWIN32__) && wxCHECK_W32API_VERSION( 0, 6 )
#define wxHAVE_TCHAR_FUNCTIONS
#include <stddef.h>
#include <string.h>
#include <ctype.h>
#endif #endif
#elif defined(__VISAGECPP__) && (__IBMCPP__ >= 400) #elif defined(__VISAGECPP__) && (__IBMCPP__ >= 400)
// VisualAge 4.0+ supports TCHAR // VisualAge 4.0+ supports TCHAR
@@ -116,6 +121,12 @@
# include <tchar.h> # include <tchar.h>
# if wxUSE_UNICODE // temporary - preserve binary compatibility # if wxUSE_UNICODE // temporary - preserve binary compatibility
#if defined(__GNUWIN32__)
#define _TCHAR TCHAR
#define _TSCHAR TCHAR
#define _TUCHAR TCHAR
#endif
typedef _TCHAR wxChar; typedef _TCHAR wxChar;
typedef _TSCHAR wxSChar; typedef _TSCHAR wxSChar;
typedef _TUCHAR wxUChar; typedef _TUCHAR wxUChar;

View File

@@ -311,23 +311,26 @@ static HGLOBAL wxCreateDevNames(const wxString& driverName, const wxString& prin
else else
{ {
hDev = GlobalAlloc(GPTR, 4*sizeof(WORD)+ hDev = GlobalAlloc(GPTR, 4*sizeof(WORD)+
driverName.Length() + 1 + ( driverName.Length() + 1 +
printerName.Length() + 1 + printerName.Length() + 1 +
portName.Length()+1); portName.Length()+1 ) * sizeof(wxChar) );
LPDEVNAMES lpDev = (LPDEVNAMES)GlobalLock(hDev); LPDEVNAMES lpDev = (LPDEVNAMES)GlobalLock(hDev);
lpDev->wDriverOffset = sizeof(WORD)*4; lpDev->wDriverOffset = sizeof(WORD)*4;
wxStrcpy((wxChar*)lpDev + lpDev->wDriverOffset, driverName); wxStrcpy((wxChar*)((char*)lpDev + lpDev->wDriverOffset ), driverName);
lpDev->wDeviceOffset = (WORD)(lpDev->wDriverOffset + driverName.Length()+1); lpDev->wDeviceOffset = (WORD)( lpDev->wDriverOffset +
wxStrcpy((wxChar*)lpDev + lpDev->wDeviceOffset, printerName); sizeof(wxChar) * ( driverName.Length() + 1 ) );
wxStrcpy((wxChar*)((char*)lpDev + lpDev->wDeviceOffset ), printerName);
lpDev->wOutputOffset = (WORD)(lpDev->wDeviceOffset + printerName.Length()+1); lpDev->wOutputOffset = (WORD)( lpDev->wDeviceOffset +
wxStrcpy((wxChar*)lpDev + lpDev->wOutputOffset, portName); sizeof(wxChar) * ( printerName.Length() + 1 ) );
wxStrcpy((wxChar*)((char*) lpDev + lpDev->wOutputOffset ), portName);
lpDev->wDefault = 0; lpDev->wDefault = 0;
GlobalUnlock(hDev); GlobalUnlock(hDev);
} }
return hDev; return hDev;
} }
@@ -338,35 +341,35 @@ void wxPrintData::ConvertToNative()
if (!hDevMode) if (!hDevMode)
{ {
// Use PRINTDLG as a way of creating a DEVMODE object // Use PRINTDLG as a way of creating a DEVMODE object
PRINTDLG *pd = new PRINTDLG; PRINTDLG pd;
// GNU-WIN32 has the wrong size PRINTDLG - can't work out why. // GNU-WIN32 has the wrong size PRINTDLG - can't work out why.
#ifdef __GNUWIN32__ #ifdef __GNUWIN32__
memset(pd, 0, 66); memset(&pd, 0, 66);
pd->lStructSize = 66 ; pd.lStructSize = 66;
#else #else
memset(pd, 0, sizeof(PRINTDLG)); memset(&pd, 0, sizeof(PRINTDLG));
pd->lStructSize = sizeof(PRINTDLG); pd.lStructSize = sizeof(PRINTDLG);
#endif #endif
pd->hwndOwner = (HWND)NULL; pd.hwndOwner = (HWND)NULL;
pd->hDevMode = NULL; // Will be created by PrintDlg pd.hDevMode = NULL; // Will be created by PrintDlg
pd->hDevNames = NULL; // Ditto pd.hDevNames = NULL; // Ditto
pd->hInstance = (HINSTANCE) wxGetInstance(); //pd.hInstance = (HINSTANCE) wxGetInstance();
pd->Flags = PD_RETURNDEFAULT; pd.Flags = PD_RETURNDEFAULT;
pd->nCopies = 1; pd.nCopies = 1;
// Fill out the DEVMODE structure // Fill out the DEVMODE structure
// so we can use it as input in the 'real' PrintDlg // so we can use it as input in the 'real' PrintDlg
if (!PrintDlg(pd)) if (!PrintDlg(&pd))
{ {
if ( pd->hDevMode ) if ( pd.hDevMode )
GlobalFree(pd->hDevMode); GlobalFree(pd.hDevMode);
if ( pd->hDevNames ) if ( pd.hDevNames )
GlobalFree(pd->hDevNames); GlobalFree(pd.hDevNames);
pd->hDevMode = NULL; pd.hDevMode = NULL;
pd->hDevNames = NULL; pd.hDevNames = NULL;
#if defined(__WXDEBUG__) && defined(__WIN32__) #if defined(__WXDEBUG__) && defined(__WIN32__)
wxString str(wxT("Printing error: ")); wxString str(wxT("Printing error: "));
@@ -376,22 +379,20 @@ void wxPrintData::ConvertToNative()
} }
else else
{ {
hDevMode = pd->hDevMode; hDevMode = pd.hDevMode;
m_devMode = (void*)(long) hDevMode; m_devMode = (void*)(long) hDevMode;
pd->hDevMode = NULL; pd.hDevMode = NULL;
// We'll create a new DEVNAMEs structure below. // We'll create a new DEVNAMEs structure below.
if ( pd->hDevNames ) if ( pd.hDevNames )
GlobalFree(pd->hDevNames); GlobalFree(pd.hDevNames);
pd->hDevNames = NULL; pd.hDevNames = NULL;
// hDevNames = pd->hDevNames; // hDevNames = pd->hDevNames;
// m_devNames = (void*)(long) hDevNames; // m_devNames = (void*)(long) hDevNames;
// pd->hDevnames = NULL; // pd->hDevnames = NULL;
} }
delete pd;
} }
if ( hDevMode ) if ( hDevMode )
@@ -421,12 +422,9 @@ void wxPrintData::ConvertToNative()
if (m_printerName != wxT("")) if (m_printerName != wxT(""))
{ {
// TODO: make this Unicode compatible //int len = wxMin(31, m_printerName.Len());
int len = wxMin(31, m_printerName.Len()); wxStrncpy((wxChar*)devMode->dmDeviceName,m_printerName.c_str(),31);
int i; devMode->dmDeviceName[31] = wxT('\0');
for (i = 0; i < len; i++)
devMode->dmDeviceName[i] = m_printerName.GetChar(i);
devMode->dmDeviceName[i] = 0;
} }
//// Colour //// Colour
@@ -516,7 +514,7 @@ void wxPrintData::ConvertToNative()
} }
// TODO: I hope it's OK to pass some empty strings to DEVNAMES. // TODO: I hope it's OK to pass some empty strings to DEVNAMES.
m_devNames = (void*) (long) wxCreateDevNames("", m_printerName, ""); m_devNames = (void*) (long) wxCreateDevNames(wxT(""), m_printerName, wxT(""));
} }
void wxPrintData::ConvertFromNative() void wxPrintData::ConvertFromNative()
@@ -709,7 +707,7 @@ void wxPrintData::ConvertFromNative()
// m_printData.SetPortName((LPSTR)lpDevNames + lpDevNames->wDriverOffset); // m_printData.SetPortName((LPSTR)lpDevNames + lpDevNames->wDriverOffset);
// Get the printer name // Get the printer name
wxString printerName = (LPSTR)lpDevNames + lpDevNames->wDeviceOffset; wxString printerName = (LPTSTR)lpDevNames + lpDevNames->wDeviceOffset;
// Not sure if we should check for this mismatch // Not sure if we should check for this mismatch
// wxASSERT_MSG( (m_printerName == "" || (devName == m_printerName)), "Printer name obtained from DEVMODE and DEVNAMES were different!"); // wxASSERT_MSG( (m_printerName == "" || (devName == m_printerName)), "Printer name obtained from DEVMODE and DEVNAMES were different!");
@@ -878,6 +876,7 @@ void wxPrintDialogData::ConvertToNative()
if (!pd) if (!pd)
{ {
pd = new PRINTDLG; pd = new PRINTDLG;
memset( pd, 0, sizeof(PRINTDLG) );
m_printDlgData = (void*) pd; m_printDlgData = (void*) pd;
// GNU-WIN32 has the wrong size PRINTDLG - can't work out why. // GNU-WIN32 has the wrong size PRINTDLG - can't work out why.

View File

@@ -78,6 +78,7 @@
#endif // native Win compiler #endif // native Win compiler
#ifdef __GNUWIN32__ #ifdef __GNUWIN32__
#include <wchar.h>
#ifndef __TWIN32__ #ifndef __TWIN32__
#include <sys/unistd.h> #include <sys/unistd.h>
#endif #endif
@@ -1113,7 +1114,10 @@ wxRenameFile (const wxString& file1, const wxString& file2)
bool wxRemoveFile(const wxString& file) bool wxRemoveFile(const wxString& file)
{ {
#if defined(__VISUALC__) || defined(__BORLANDC__) || defined(__WATCOMC__) #if defined(__VISUALC__) \
|| defined(__BORLANDC__) \
|| defined(__WATCOMC__) \
|| defined(__GNUWIN32__)
int res = wxRemove(file); int res = wxRemove(file);
#else #else
int res = unlink(OS_FILENAME(file)); int res = unlink(OS_FILENAME(file));

View File

@@ -115,7 +115,7 @@ wxWindow *wxPrinterBase::CreateAbortWindow(wxWindow *parent, wxPrintout *WXUNUSE
return dialog; return dialog;
} }
void wxPrinterBase::ReportError(wxWindow *parent, wxPrintout *WXUNUSED(printout), char *message) void wxPrinterBase::ReportError(wxWindow *parent, wxPrintout *WXUNUSED(printout), const wxString& message)
{ {
wxMessageBox(message, _("Printing Error"), wxOK, parent); wxMessageBox(message, _("Printing Error"), wxOK, parent);
} }

View File

@@ -265,7 +265,7 @@ bool wxGridCellEnumEditor::EndEdit(int row, int col, wxGrid* grid)
if (grid->GetTable()->CanSetValueAs(row, col, wxGRID_VALUE_NUMBER)) if (grid->GetTable()->CanSetValueAs(row, col, wxGRID_VALUE_NUMBER))
grid->GetTable()->SetValueAsLong(row, col, pos); grid->GetTable()->SetValueAsLong(row, col, pos);
else else
grid->GetTable()->SetValue(row, col,wxString::Format("%i",pos)); grid->GetTable()->SetValue(row, col,wxString::Format(wxT("%i"),pos));
} }
return changed; return changed;

View File

@@ -215,7 +215,7 @@ bool wxHtmlTag::HasParam(const wxString& par) const
{ {
const wxChar *st = m_Params, *p = par; const wxChar *st = m_Params, *p = par;
const wxChar *st2, *p2; const wxChar *st2, *p2;
const wxChar invalid = wxT(1); const wxChar invalid = wxT('\1');
if (*st == 0) return FALSE; if (*st == 0) return FALSE;
if (*p == 0) return FALSE; if (*p == 0) return FALSE;
@@ -249,7 +249,7 @@ wxString wxHtmlTag::GetParam(const wxString& par, bool with_commas) const
{ {
const wxChar *st = m_Params, *p = par; const wxChar *st = m_Params, *p = par;
const wxChar *st2, *p2; const wxChar *st2, *p2;
const wxChar invalid = wxT(1); const wxChar invalid = wxT('\1');
bool comma; bool comma;
wxChar comma_char; wxChar comma_char;
@@ -315,7 +315,7 @@ wxString wxHtmlTag::GetParam(const wxString& par, bool with_commas) const
int wxHtmlTag::ScanParam(const wxString& par, wxChar *format, void *param) const int wxHtmlTag::ScanParam(const wxString& par, wxChar *format, void *param) const
{ {
wxString parval = GetParam(par); wxString parval = GetParam(par);
return wxSscanf((const wxChar*)parval, format, param); return wxSscanf(parval, format, param);
} }
#endif #endif

View File

@@ -224,16 +224,18 @@ static bool wxGetDefaultDeviceName(wxString& deviceName, wxString& portName)
deviceName.clear(); deviceName.clear();
LPDEVNAMES lpDevNames; LPDEVNAMES lpDevNames;
LPSTR lpszDriverName; LPTSTR lpszDriverName;
LPSTR lpszDeviceName; LPTSTR lpszDeviceName;
LPSTR lpszPortName; LPTSTR lpszPortName;
PRINTDLG pd; PRINTDLG pd;
// Cygwin has trouble believing PRINTDLG is 66 bytes - thinks it is 68 // Cygwin has trouble believing PRINTDLG is 66 bytes - thinks it is 68
#ifdef __GNUWIN32__ #ifdef __GNUWIN32__
memset(&pd, 0, 66);
pd.lStructSize = 66; // sizeof(PRINTDLG); pd.lStructSize = 66; // sizeof(PRINTDLG);
#else #else
memset(&pd, 0, sizeof(PRINTDLG));
pd.lStructSize = sizeof(PRINTDLG); pd.lStructSize = sizeof(PRINTDLG);
#endif #endif
@@ -256,9 +258,9 @@ static bool wxGetDefaultDeviceName(wxString& deviceName, wxString& portName)
if (pd.hDevNames) if (pd.hDevNames)
{ {
lpDevNames = (LPDEVNAMES)GlobalLock(pd.hDevNames); lpDevNames = (LPDEVNAMES)GlobalLock(pd.hDevNames);
lpszDriverName = (LPSTR)lpDevNames + lpDevNames->wDriverOffset; lpszDriverName = (LPTSTR)lpDevNames + lpDevNames->wDriverOffset;
lpszDeviceName = (LPSTR)lpDevNames + lpDevNames->wDeviceOffset; lpszDeviceName = (LPTSTR)lpDevNames + lpDevNames->wDeviceOffset;
lpszPortName = (LPSTR)lpDevNames + lpDevNames->wOutputOffset; lpszPortName = (LPTSTR)lpDevNames + lpDevNames->wOutputOffset;
deviceName = lpszDeviceName; deviceName = lpszDeviceName;
portName = lpszPortName; portName = lpszPortName;

View File

@@ -135,7 +135,7 @@ bool wxFontEnumeratorHelper::SetEncoding(wxFontEncoding encoding)
#if defined(__GNUWIN32__) #if defined(__GNUWIN32__)
#if wxUSE_NORLANDER_HEADERS #if wxUSE_NORLANDER_HEADERS
#define wxFONTENUMPROC int(*)(const LOGFONTA *, const TEXTMETRICA *, long unsigned int, LPARAM) #define wxFONTENUMPROC int(*)(const LOGFONT *, const TEXTMETRIC *, long unsigned int, LPARAM)
#else #else
#define wxFONTENUMPROC int(*)(ENUMLOGFONTEX *, NEWTEXTMETRICEX*, int, LPARAM) #define wxFONTENUMPROC int(*)(ENUMLOGFONTEX *, NEWTEXTMETRICEX*, int, LPARAM)
#endif #endif

View File

@@ -222,7 +222,7 @@ wxDragResult wxDropSource::DoDragDrop(bool bAllowMove)
} }
else { else {
if ( FAILED(hr) ) { if ( FAILED(hr) ) {
wxLogApiError("DoDragDrop", hr); wxLogApiError(wxT("DoDragDrop"), hr);
wxLogError(wxT("Drag & drop operation failed.")); wxLogError(wxT("Drag & drop operation failed."));
} }
else { else {

View File

@@ -305,7 +305,7 @@ bool wxDropTarget::Register(WXHWND hwnd)
{ {
HRESULT hr = ::CoLockObjectExternal(m_pIDropTarget, TRUE, FALSE); HRESULT hr = ::CoLockObjectExternal(m_pIDropTarget, TRUE, FALSE);
if ( FAILED(hr) ) { if ( FAILED(hr) ) {
wxLogApiError("CoLockObjectExternal", hr); wxLogApiError(wxT("CoLockObjectExternal"), hr);
return FALSE; return FALSE;
} }
@@ -313,7 +313,7 @@ bool wxDropTarget::Register(WXHWND hwnd)
if ( FAILED(hr) ) { if ( FAILED(hr) ) {
::CoLockObjectExternal(m_pIDropTarget, FALSE, FALSE); ::CoLockObjectExternal(m_pIDropTarget, FALSE, FALSE);
wxLogApiError("RegisterDragDrop", hr); wxLogApiError(wxT("RegisterDragDrop"), hr);
return FALSE; return FALSE;
} }
@@ -328,7 +328,7 @@ void wxDropTarget::Revoke(WXHWND hwnd)
HRESULT hr = ::RevokeDragDrop((HWND) hwnd); HRESULT hr = ::RevokeDragDrop((HWND) hwnd);
if ( FAILED(hr) ) { if ( FAILED(hr) ) {
wxLogApiError("RevokeDragDrop", hr); wxLogApiError(wxT("RevokeDragDrop"), hr);
} }
::CoLockObjectExternal(m_pIDropTarget, FALSE, TRUE); ::CoLockObjectExternal(m_pIDropTarget, FALSE, TRUE);