Opened 17 years ago

Closed 17 years ago

Last modified 16 years ago

#237 closed Defect (fixed)

current BM fails to build with wxWidgets unicode variant

Reported by: fthomas Owned by: romw
Priority: Critical Milestone: Undetermined
Component: Manager Version:
Keywords: patch Cc:

Description

Hi,

Current BM fails to build with wxWidgets unicode variant under Debian with these errors:

SkinManager.cpp: In member function ‘bool CSkinManager::ReloadSkin(wxLocale*, wxString)’:
SkinManager.cpp:1210: error: cannot convert ‘const wxChar*’ to ‘const char*’ for argument ‘1’ to ‘FILE* fopen(const char*, const char*)’
SkinManager.cpp:1221: error: cannot convert ‘const wxChar*’ to ‘const char*’ for argument ‘1’ to ‘FILE* fopen(const char*, const char*)’
SkinManager.cpp:1232: error: cannot convert ‘const wxChar*’ to ‘const char*’ for argument ‘1’ to ‘FILE* fopen(const char*, const char*)’

SkinManager.cpp: In member function ‘int CSkinManager::Parse(MIOFILE&, wxString)’:
SkinManager.cpp:1346: error: no matching function for call to ‘match_tag(char [256], const wxChar*)’
../lib/parse.h:62: note: candidates are: bool match_tag(const char*, const char*)
../lib/parse.h:67: note:                 bool match_tag(const std::string&, const char*)
SkinManager.cpp:1355: error: no matching function for call to ‘match_tag(char [256], const wxChar*)’
../lib/parse.h:62: note: candidates are: bool match_tag(const char*, const char*)
../lib/parse.h:67: note:                 bool match_tag(const std::string&, const char*)

Something like the attached patch fixes the build failures.

Regards, Frank

Attachments (2)

sg_unicode.patch (2.0 KB) - added by fthomas 17 years ago.
sg_unicode.2.patch (1.8 KB) - added by fthomas 17 years ago.
this new patch is much cleaner

Download all attachments as: .zip

Change History (4)

Changed 17 years ago by fthomas

Attachment: sg_unicode.patch added

Changed 17 years ago by fthomas

Attachment: sg_unicode.2.patch added

this new patch is much cleaner

comment:1 Changed 17 years ago by romw

Resolution: fixed
Status: newclosed

(In [12859]) - fixes #237 - current BM fails to build with wxWidgets

unicode variant

(Applied Frank Thomas's patch)

clientgui/

SkinManager?.cpp

comment:2 Changed 17 years ago by Nicolas

Keywords: patch added
Note: See TracTickets for help on using tickets.