X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/15e8daecf594d38a57bc750f58de0ea33b79f4ee..b2c13097deb7736a86a278c02d0b3f7cd9873750:/contrib/src/deprecated/resource.cpp?ds=sidebyside diff --git a/contrib/src/deprecated/resource.cpp b/contrib/src/deprecated/resource.cpp index a179f370d2..8837544255 100644 --- a/contrib/src/deprecated/resource.cpp +++ b/contrib/src/deprecated/resource.cpp @@ -89,9 +89,14 @@ #include "wx/deprecated/resource.h" #include "wx/deprecated/wxexpr.h" +#if !WXWIN_COMPATIBILITY_2_4 +static inline wxChar* copystring(const wxChar* s) + { return wxStrcpy(new wxChar[wxStrlen(s) + 1], s); } +#endif + // Forward (private) declarations bool wxResourceInterpretResources(wxResourceTable& table, wxExprDatabase& db); -wxItemResource *wxResourceInterpretDialog(wxResourceTable& table, wxExpr *expr, bool isPanel = FALSE); +wxItemResource *wxResourceInterpretDialog(wxResourceTable& table, wxExpr *expr, bool isPanel = false); wxItemResource *wxResourceInterpretControl(wxResourceTable& table, wxExpr *expr); wxItemResource *wxResourceInterpretMenu(wxResourceTable& table, wxExpr *expr); wxItemResource *wxResourceInterpretMenuBar(wxResourceTable& table, wxExpr *expr); @@ -132,7 +137,7 @@ class wxResourceModule: public wxModule { public: wxResourceModule() : wxModule() {} - virtual bool OnInit() { wxInitializeResourceSystem(); return TRUE; } + virtual bool OnInit() { wxInitializeResourceSystem(); return true; } virtual void OnExit() { wxCleanUpResourceSystem(); } DECLARE_DYNAMIC_CLASS(wxResourceModule) @@ -146,26 +151,26 @@ IMPLEMENT_DYNAMIC_CLASS(wxResourceTable, wxHashTable) wxItemResource::wxItemResource() { - m_itemType = wxT(""); - m_title = wxT(""); - m_name = wxT(""); + m_itemType = wxEmptyString; + m_title = wxEmptyString; + m_name = wxEmptyString; m_windowStyle = 0; m_x = m_y = m_width = m_height = 0; m_value1 = m_value2 = m_value3 = m_value5 = 0; - m_value4 = wxT(""); + m_value4 = wxEmptyString; m_windowId = 0; m_exStyle = 0; } wxItemResource::~wxItemResource() { - wxNode *node = m_children.First(); + wxNode *node = m_children.GetFirst(); while (node) { - wxItemResource *item = (wxItemResource *)node->Data(); + wxItemResource *item = (wxItemResource *)node->GetData(); delete item; delete node; - node = m_children.First(); + node = m_children.GetFirst(); } } @@ -191,9 +196,9 @@ wxItemResource *wxResourceTable::FindResource(const wxString& name) const void wxResourceTable::AddResource(wxItemResource *item) { wxString name = item->GetName(); - if (name == wxT("")) + if (name.empty()) name = item->GetTitle(); - if (name == wxT("")) + if (name.empty()) name = wxT("no name"); // Delete existing resource, if any. @@ -210,11 +215,10 @@ bool wxResourceTable::DeleteResource(const wxString& name) // See if any resource has this as its child; if so, delete from // parent's child list. BeginFind(); - wxNode *node = (wxNode *) NULL; - node = Next(); + wxHashTable::Node *node = Next(); while (node != NULL) { - wxItemResource *parent = (wxItemResource *)node->Data(); + wxItemResource *parent = (wxItemResource *)node->GetData(); if (parent->GetChildren().Member(item)) { parent->GetChildren().DeleteObject(item); @@ -224,10 +228,10 @@ bool wxResourceTable::DeleteResource(const wxString& name) } delete item; - return TRUE; + return true; } else - return FALSE; + return false; } bool wxResourceTable::ParseResourceFile( wxInputStream *is ) @@ -235,7 +239,7 @@ bool wxResourceTable::ParseResourceFile( wxInputStream *is ) wxExprDatabase db; int len = is->GetSize() ; - bool eof = FALSE; + bool eof = false; while ( is->TellI() + 10 < len) // it's a hack because the streams dont support EOF { wxResourceReadOneResource(is, db, &eof, this) ; @@ -249,8 +253,8 @@ bool wxResourceTable::ParseResourceFile(const wxString& filename) FILE *fd = wxFopen(filename, wxT("r")); if (!fd) - return FALSE; - bool eof = FALSE; + return false; + bool eof = false; while (wxResourceReadOneResource(fd, db, &eof, this) && !eof) { // Loop @@ -265,7 +269,7 @@ bool wxResourceTable::ParseResourceData(const wxString& data) if (!db.ReadFromString(data)) { wxLogWarning(_("Ill-formed resource file syntax.")); - return FALSE; + return false; } return wxResourceInterpretResources(*this, db); @@ -282,7 +286,7 @@ bool wxResourceTable::RegisterResourceBitmapData(const wxString& name, char bits item->SetValue2((long)width); item->SetValue3((long)height); AddResource(item); - return TRUE; + return true; } bool wxResourceTable::RegisterResourceBitmapData(const wxString& name, char **data) @@ -294,22 +298,22 @@ bool wxResourceTable::RegisterResourceBitmapData(const wxString& name, char **da item->SetName(name); item->SetValue1((long)data); AddResource(item); - return TRUE; + return true; } bool wxResourceTable::SaveResource(const wxString& WXUNUSED(filename)) { - return FALSE; + return false; } void wxResourceTable::ClearTable() { BeginFind(); - wxNode *node = Next(); + wxHashTable::Node *node = Next(); while (node) { - wxNode *next = Next(); - wxItemResource *item = (wxItemResource *)node->Data(); + wxHashTable::Node *next = Next(); + wxItemResource *item = (wxItemResource *)node->GetData(); delete item; delete node; node = next; @@ -320,7 +324,7 @@ wxControl *wxResourceTable::CreateItem(wxWindow *parent, const wxItemResource* c { int id = childResource->GetId(); if ( id == 0 ) - id = -1; + id = wxID_ANY; bool dlgUnits = ((parentResource->GetResourceStyle() & wxRESOURCE_DIALOG_UNITS) != 0); @@ -342,7 +346,7 @@ wxControl *wxResourceTable::CreateItem(wxWindow *parent, const wxItemResource* c if (itemType == wxString(wxT("wxButton")) || itemType == wxString(wxT("wxBitmapButton"))) { - if (childResource->GetValue4() != wxT("")) + if (!childResource->GetValue4().empty()) { // Bitmap button wxBitmap bitmap = childResource->GetBitmap(); @@ -371,7 +375,7 @@ wxControl *wxResourceTable::CreateItem(wxWindow *parent, const wxItemResource* c else if (itemType == wxString(wxT("wxMessage")) || itemType == wxString(wxT("wxStaticText")) || itemType == wxString(wxT("wxStaticBitmap"))) { - if (childResource->GetValue4() != wxT("") || itemType == wxString(wxT("wxStaticBitmap")) ) + if (!childResource->GetValue4().empty() || itemType == wxString(wxT("wxStaticBitmap")) ) { // Bitmap message wxBitmap bitmap = childResource->GetBitmap(); @@ -439,7 +443,7 @@ wxControl *wxResourceTable::CreateItem(wxWindow *parent, const wxItemResource* c ((wxScrollBar *)control)->SetViewLength((int)(long)childResource->GetValue5()); */ ((wxScrollBar *)control)->SetScrollbar((int)childResource->GetValue1(),(int)childResource->GetValue2(), - (int)childResource->GetValue3(),(int)(long)childResource->GetValue5(),FALSE); + (int)childResource->GetValue3(),(int)(long)childResource->GetValue5(),false); } #endif @@ -459,17 +463,17 @@ wxControl *wxResourceTable::CreateItem(wxWindow *parent, const wxItemResource* c wxStringList& stringList = childResource->GetStringValues(); wxString *strings = (wxString *) NULL; int noStrings = 0; - if (stringList.Number() > 0) + if (stringList.GetCount() > 0) { - noStrings = stringList.Number(); + noStrings = stringList.GetCount(); strings = new wxString[noStrings]; - wxNode *node = stringList.First(); + wxStringListNode *node = stringList.GetFirst(); int i = 0; while (node) { - strings[i] = (wxChar *)node->Data(); + strings[i] = (wxChar *)node->GetData(); i ++; - node = node->Next(); + node = node->GetNext(); } } control = new wxListBox(parent, id, pos, size, @@ -483,17 +487,17 @@ wxControl *wxResourceTable::CreateItem(wxWindow *parent, const wxItemResource* c wxStringList& stringList = childResource->GetStringValues(); wxString *strings = (wxString *) NULL; int noStrings = 0; - if (stringList.Number() > 0) + if (stringList.GetCount() > 0) { - noStrings = stringList.Number(); + noStrings = stringList.GetCount(); strings = new wxString[noStrings]; - wxNode *node = stringList.First(); + wxStringListNode *node = stringList.GetFirst(); int i = 0; while (node) { - strings[i] = (wxChar *)node->Data(); + strings[i] = (wxChar *)node->GetData(); i ++; - node = node->Next(); + node = node->GetNext(); } } control = new wxChoice(parent, id, pos, size, @@ -508,17 +512,17 @@ wxControl *wxResourceTable::CreateItem(wxWindow *parent, const wxItemResource* c wxStringList& stringList = childResource->GetStringValues(); wxString *strings = (wxString *) NULL; int noStrings = 0; - if (stringList.Number() > 0) + if (stringList.GetCount() > 0) { - noStrings = stringList.Number(); + noStrings = stringList.GetCount(); strings = new wxString[noStrings]; - wxNode *node = stringList.First(); + wxStringListNode *node = stringList.GetFirst(); int i = 0; while (node) { - strings[i] = (wxChar *)node->Data(); + strings[i] = (wxChar *)node->GetData(); i ++; - node = node->Next(); + node = node->GetNext(); } } control = new wxComboBox(parent, id, childResource->GetValue4(), pos, size, @@ -533,17 +537,17 @@ wxControl *wxResourceTable::CreateItem(wxWindow *parent, const wxItemResource* c wxStringList& stringList = childResource->GetStringValues(); wxString *strings = (wxString *) NULL; int noStrings = 0; - if (stringList.Number() > 0) + if (stringList.GetCount() > 0) { - noStrings = stringList.Number(); + noStrings = stringList.GetCount(); strings = new wxString[noStrings]; - wxNode *node = stringList.First(); + wxStringListNode *node = stringList.GetFirst(); int i = 0; while (node) { - strings[i] = (wxChar *)node->Data(); + strings[i] = (wxChar *)node->GetData(); i ++; - node = node->Next(); + node = node->GetNext(); } } control = new wxRadioBox(parent, (wxWindowID) id, wxString(childResource->GetTitle()), pos, size, @@ -568,7 +572,7 @@ wxControl *wxResourceTable::CreateItem(wxWindow *parent, const wxItemResource* c // Force the layout algorithm since the size changes the layout if (control->IsKindOf(CLASSINFO(wxRadioBox))) { - control->SetSize(-1, -1, -1, -1, wxSIZE_AUTO_WIDTH|wxSIZE_AUTO_HEIGHT); + control->SetSize(wxDefaultCoord, wxDefaultCoord, wxDefaultCoord, wxDefaultCoord, wxSIZE_AUTO_WIDTH|wxSIZE_AUTO_HEIGHT); } #endif } @@ -582,17 +586,17 @@ wxControl *wxResourceTable::CreateItem(wxWindow *parent, const wxItemResource* c bool wxResourceInterpretResources(wxResourceTable& table, wxExprDatabase& db) { - wxNode *node = db.First(); + wxNode *node = db.GetFirst(); while (node) { - wxExpr *clause = (wxExpr *)node->Data(); + wxExpr *clause = (wxExpr *)node->GetData(); wxString functor(clause->Functor()); wxItemResource *item = (wxItemResource *) NULL; if (functor == wxT("dialog")) item = wxResourceInterpretDialog(table, clause); else if (functor == wxT("panel")) - item = wxResourceInterpretDialog(table, clause, TRUE); + item = wxResourceInterpretDialog(table, clause, true); else if (functor == wxT("menubar")) item = wxResourceInterpretMenuBar(table, clause); else if (functor == wxT("menu")) @@ -607,13 +611,13 @@ bool wxResourceInterpretResources(wxResourceTable& table, wxExprDatabase& db) if (item) { // Remove any existing resource of same name - if (item->GetName() != wxT("")) + if (!item->GetName().empty()) table.DeleteResource(item->GetName()); table.AddResource(item); } - node = node->Next(); + node = node->GetNext(); } - return TRUE; + return true; } static const wxChar *g_ValidControlClasses[] = @@ -645,9 +649,9 @@ static bool wxIsValidControlClass(const wxString& c) for ( size_t i = 0; i < WXSIZEOF(g_ValidControlClasses); i++ ) { if ( c == g_ValidControlClasses[i] ) - return TRUE; + return true; } - return FALSE; + return false; } wxItemResource *wxResourceInterpretDialog(wxResourceTable& table, wxExpr *expr, bool isPanel) @@ -657,18 +661,18 @@ wxItemResource *wxResourceInterpretDialog(wxResourceTable& table, wxExpr *expr, dialogItem->SetType(wxT("wxPanel")); else dialogItem->SetType(wxT("wxDialog")); - wxString style = wxT(""); - wxString title = wxT(""); - wxString name = wxT(""); - wxString backColourHex = wxT(""); - wxString labelColourHex = wxT(""); - wxString buttonColourHex = wxT(""); + wxString style = wxEmptyString; + wxString title = wxEmptyString; + wxString name = wxEmptyString; + wxString backColourHex = wxEmptyString; + wxString labelColourHex = wxEmptyString; + wxString buttonColourHex = wxEmptyString; long windowStyle = wxDEFAULT_DIALOG_STYLE; if (isPanel) windowStyle = 0; - int x = 0; int y = 0; int width = -1; int height = -1; + int x = 0; int y = 0; int width = wxDefaultCoord; int height = wxDefaultCoord; int isModal = 0; wxExpr *labelFontExpr = (wxExpr *) NULL; wxExpr *buttonFontExpr = (wxExpr *) NULL; @@ -702,26 +706,32 @@ wxItemResource *wxResourceInterpretDialog(wxResourceTable& table, wxExpr *expr, expr->GetAttributeValue(wxT("id"), id); dialogItem->SetId(id); - if (style != wxT("")) + if (!style.empty()) { windowStyle = wxParseWindowStyle(style); } dialogItem->SetStyle(windowStyle); dialogItem->SetValue1(isModal); - if (windowStyle & wxDIALOG_MODAL) // Uses style in wxWin 2 - dialogItem->SetValue1(TRUE); +#ifdef __VMS +#pragma message disable CODCAUUNR +#endif + if (windowStyle & wxDIALOG_MODAL) // Uses style in wxWin 2 + dialogItem->SetValue1(true); +#ifdef __VMS +#pragma message enable CODCAUUNR +#endif dialogItem->SetName(name); dialogItem->SetTitle(title); dialogItem->SetSize(x, y, width, height); // Check for wxWin 1.68-style specifications - if (style.Find(wxT("VERTICAL_LABEL")) != -1) + if (style.Find(wxT("VERTICAL_LABEL")) != wxNOT_FOUND) dialogItem->SetResourceStyle(dialogItem->GetResourceStyle() | wxRESOURCE_VERTICAL_LABEL); - else if (style.Find(wxT("HORIZONTAL_LABEL")) != -1) + else if (style.Find(wxT("HORIZONTAL_LABEL")) != wxNOT_FOUND) dialogItem->SetResourceStyle(dialogItem->GetResourceStyle() | wxRESOURCE_HORIZONTAL_LABEL); - if (backColourHex != wxT("")) + if (!backColourHex.empty()) { int r = 0; int g = 0; @@ -731,7 +741,7 @@ wxItemResource *wxResourceInterpretDialog(wxResourceTable& table, wxExpr *expr, b = wxHexToDec(backColourHex.Mid(4, 2)); dialogItem->SetBackgroundColour(wxColour((unsigned char)r,(unsigned char)g,(unsigned char)b)); } - if (labelColourHex != wxT("")) + if (!labelColourHex.empty()) { int r = 0; int g = 0; @@ -741,7 +751,7 @@ wxItemResource *wxResourceInterpretDialog(wxResourceTable& table, wxExpr *expr, b = wxHexToDec(labelColourHex.Mid(4, 2)); dialogItem->SetLabelColour(wxColour((unsigned char)r,(unsigned char)g,(unsigned char)b)); } - if (buttonColourHex != wxT("")) + if (!buttonColourHex.empty()) { int r = 0; int g = 0; @@ -766,7 +776,7 @@ wxItemResource *wxResourceInterpretDialog(wxResourceTable& table, wxExpr *expr, if (controlExpr->Number() == 3) { wxString controlKeyword(controlExpr->Nth(1)->StringValue()); - if (controlKeyword != wxT("") && controlKeyword == wxT("control")) + if (!controlKeyword.empty() && controlKeyword == wxT("control")) { // The value part: always a list. wxExpr *listExpr = controlExpr->Nth(2); @@ -798,7 +808,7 @@ wxItemResource *wxResourceInterpretControl(wxResourceTable& table, wxExpr *expr) wxString name; int id = 0; long windowStyle = 0; - int x = 0; int y = 0; int width = -1; int height = -1; + int x = 0; int y = 0; int width = wxDefaultCoord; int height = wxDefaultCoord; int count = 0; wxExpr *expr1 = expr->Nth(0); @@ -883,9 +893,9 @@ wxItemResource *wxResourceInterpretControl(wxResourceTable& table, wxExpr *expr) controlItem->SetId(id); // Check for wxWin 1.68-style specifications - if (style.Find(wxT("VERTICAL_LABEL")) != -1) + if (style.Find(wxT("VERTICAL_LABEL")) != wxNOT_FOUND) controlItem->SetResourceStyle(controlItem->GetResourceStyle() | wxRESOURCE_VERTICAL_LABEL); - else if (style.Find(wxT("HORIZONTAL_LABEL")) != -1) + else if (style.Find(wxT("HORIZONTAL_LABEL")) != wxNOT_FOUND) controlItem->SetResourceStyle(controlItem->GetResourceStyle() | wxRESOURCE_HORIZONTAL_LABEL); if (controlType == wxT("wxButton")) @@ -896,7 +906,7 @@ wxItemResource *wxResourceInterpretControl(wxResourceTable& table, wxExpr *expr) wxString str(expr->Nth(count)->StringValue()); count ++; - if (str != wxT("")) + if (!str.empty()) { controlItem->SetValue4(str); controlItem->SetType(wxT("wxBitmapButton")); @@ -1406,18 +1416,18 @@ wxItemResource *wxResourceInterpretBitmap(wxResourceTable& WXUNUSED(table), wxEx wxExpr *coloursExpr = listExpr->Nth(3); wxExpr *xresExpr = listExpr->Nth(4); wxExpr *yresExpr = listExpr->Nth(5); - if (nameExpr && nameExpr->StringValue() != wxT("")) + if (nameExpr && !nameExpr->StringValue().empty()) { bitmapSpec->SetName(nameExpr->StringValue()); } - if (typeExpr && typeExpr->StringValue() != wxT("")) + if (typeExpr && !typeExpr->StringValue().empty()) { bitmapSpec->SetValue1(wxParseWindowStyle(typeExpr->StringValue())); } else bitmapSpec->SetValue1(0); - if (platformExpr && platformExpr->StringValue() != wxT("")) + if (platformExpr && !platformExpr->StringValue().empty()) { wxString plat(platformExpr->StringValue()); if (plat == wxT("windows") || plat == wxT("WINDOWS")) @@ -1476,7 +1486,7 @@ wxFont wxResourceInterpretFontSpec(wxExpr *expr) int style = wxNORMAL; int weight = wxNORMAL; int underline = 0; - wxString faceName(wxT("")); + wxString faceName; wxExpr *pointExpr = expr->Nth(0); wxExpr *familyExpr = expr->Nth(1); @@ -1522,7 +1532,7 @@ bool wxReallocateResourceBuffer() { wxResourceBufferSize = 1000; wxResourceBuffer = new char[wxResourceBufferSize]; - return TRUE; + return true; } if (wxResourceBuffer) { @@ -1533,12 +1543,12 @@ bool wxReallocateResourceBuffer() wxResourceBuffer = tmp; wxResourceBufferSize = newSize; } - return TRUE; + return true; } static bool wxEatWhiteSpace(FILE *fd) { - int ch = 0; + int ch; while ((ch = getc(fd)) != EOF) { @@ -1556,7 +1566,7 @@ static bool wxEatWhiteSpace(FILE *fd) if (ch == EOF) { ungetc(prev_ch, fd); - return TRUE; + return true; } if (ch == '*') @@ -1580,25 +1590,25 @@ static bool wxEatWhiteSpace(FILE *fd) { ungetc(prev_ch, fd); ungetc(ch, fd); - return TRUE; + return true; } } break; default: ungetc(ch, fd); - return TRUE; + return true; } } - return FALSE; + return false; } static bool wxEatWhiteSpace(wxInputStream *is) { - int ch = is->GetC() ; + char ch = is->GetC() ; if ((ch != ' ') && (ch != '/') && (ch != ' ') && (ch != 10) && (ch != 13) && (ch != 9)) { is->Ungetch(ch); - return TRUE; + return true; } // Eat whitespace @@ -1610,17 +1620,17 @@ static bool wxEatWhiteSpace(wxInputStream *is) ch = is->GetC(); if (ch == '*') { - bool finished = FALSE; + bool finished = false; while (!finished) { ch = is->GetC(); - if (ch == EOF) - return FALSE; + if (is->LastRead() == 0) + return false; if (ch == '*') { int newCh = is->GetC(); if (newCh == '/') - finished = TRUE; + finished = true; else { is->Ungetch(ch); @@ -1629,7 +1639,7 @@ static bool wxEatWhiteSpace(wxInputStream *is) } } else // False alarm - return FALSE; + return false; } else is->Ungetch(ch); @@ -1655,7 +1665,7 @@ bool wxGetResourceToken(FILE *fd) if (ch == EOF) { wxResourceBuffer[wxResourceBufferCount] = 0; - return FALSE; + return false; } // Escaped characters else if (ch == '\\') @@ -1694,9 +1704,9 @@ bool wxGetResourceToken(FILE *fd) } wxResourceBuffer[wxResourceBufferCount] = 0; if (ch == EOF) - return FALSE; + return false; } - return TRUE; + return true; } bool wxGetResourceToken(wxInputStream *is) @@ -1718,12 +1728,12 @@ bool wxGetResourceToken(wxInputStream *is) if (ch == EOF) { wxResourceBuffer[wxResourceBufferCount] = 0; - return FALSE; + return false; } // Escaped characters else if (ch == '\\') { - int newCh = is->GetC(); + char newCh = is->GetC(); if (newCh == '"') actualCh = '"'; else if (newCh == 10) @@ -1759,9 +1769,9 @@ bool wxGetResourceToken(wxInputStream *is) } wxResourceBuffer[wxResourceBufferCount] = 0; if (ch == EOF) - return FALSE; + return false; } - return TRUE; + return true; } /* @@ -1778,8 +1788,8 @@ bool wxResourceReadOneResource(FILE *fd, wxExprDatabase& db, bool *eof, wxResour // static or #define if (!wxGetResourceToken(fd)) { - *eof = TRUE; - return FALSE; + *eof = true; + return false; } if (strcmp(wxResourceBuffer, "#define") == 0) @@ -1798,12 +1808,12 @@ bool wxResourceReadOneResource(FILE *fd, wxExprDatabase& db, bool *eof, wxResour wxLogWarning(_("#define %s must be an integer."), name); delete[] name; delete[] value; - return FALSE; + return false; } delete[] name; delete[] value; - return TRUE; + return true; } else if (strcmp(wxResourceBuffer, "#include") == 0) { @@ -1820,44 +1830,44 @@ bool wxResourceReadOneResource(FILE *fd, wxExprDatabase& db, bool *eof, wxResour wxLogWarning(_("Could not find resource include file %s."), actualName); } delete[] name; - return TRUE; + return true; } else if (strcmp(wxResourceBuffer, "static") != 0) { wxChar buf[300]; wxStrcpy(buf, _("Found ")); wxStrncat(buf, wxConvCurrent->cMB2WX(wxResourceBuffer), 30); - wxStrcat(buf, _(", expected static, #include or #define\nwhilst parsing resource.")); + wxStrcat(buf, _(", expected static, #include or #define\nwhile parsing resource.")); wxLogWarning(buf); - return FALSE; + return false; } // char if (!wxGetResourceToken(fd)) { - wxLogWarning(_("Unexpected end of file whilst parsing resource.")); - *eof = TRUE; - return FALSE; + wxLogWarning(_("Unexpected end of file while parsing resource.")); + *eof = true; + return false; } if (strcmp(wxResourceBuffer, "char") != 0) { - wxLogWarning(_("Expected 'char' whilst parsing resource.")); - return FALSE; + wxLogWarning(_("Expected 'char' while parsing resource.")); + return false; } // *name if (!wxGetResourceToken(fd)) { - wxLogWarning(_("Unexpected end of file whilst parsing resource.")); - *eof = TRUE; - return FALSE; + wxLogWarning(_("Unexpected end of file while parsing resource.")); + *eof = true; + return false; } if (wxResourceBuffer[0] != '*') { - wxLogWarning(_("Expected '*' whilst parsing resource.")); - return FALSE; + wxLogWarning(_("Expected '*' while parsing resource.")); + return false; } wxChar nameBuf[100]; wxMB2WX(nameBuf, wxResourceBuffer+1, 99); @@ -1866,38 +1876,38 @@ bool wxResourceReadOneResource(FILE *fd, wxExprDatabase& db, bool *eof, wxResour // = if (!wxGetResourceToken(fd)) { - wxLogWarning(_("Unexpected end of file whilst parsing resource.")); - *eof = TRUE; - return FALSE; + wxLogWarning(_("Unexpected end of file while parsing resource.")); + *eof = true; + return false; } if (strcmp(wxResourceBuffer, "=") != 0) { - wxLogWarning(_("Expected '=' whilst parsing resource.")); - return FALSE; + wxLogWarning(_("Expected '=' while parsing resource.")); + return false; } // String if (!wxGetResourceToken(fd)) { - wxLogWarning(_("Unexpected end of file whilst parsing resource.")); - *eof = TRUE; - return FALSE; + wxLogWarning(_("Unexpected end of file while parsing resource.")); + *eof = true; + return false; } else { if (!db.ReadPrologFromString(wxResourceBuffer)) { wxLogWarning(_("%s: ill-formed resource file syntax."), nameBuf); - return FALSE; + return false; } } // Semicolon if (!wxGetResourceToken(fd)) { - *eof = TRUE; + *eof = true; } - return TRUE; + return true; } bool wxResourceReadOneResource(wxInputStream *fd, wxExprDatabase& db, bool *eof, wxResourceTable *table) @@ -1908,8 +1918,8 @@ bool wxResourceReadOneResource(wxInputStream *fd, wxExprDatabase& db, bool *eof, // static or #define if (!wxGetResourceToken(fd)) { - *eof = TRUE; - return FALSE; + *eof = true; + return false; } if (strcmp(wxResourceBuffer, "#define") == 0) @@ -1928,12 +1938,12 @@ bool wxResourceReadOneResource(wxInputStream *fd, wxExprDatabase& db, bool *eof, wxLogWarning(_("#define %s must be an integer."), name); delete[] name; delete[] value; - return FALSE; + return false; } delete[] name; delete[] value; - return TRUE; + return true; } else if (strcmp(wxResourceBuffer, "#include") == 0) { @@ -1950,44 +1960,44 @@ bool wxResourceReadOneResource(wxInputStream *fd, wxExprDatabase& db, bool *eof, wxLogWarning(_("Could not find resource include file %s."), actualName); } delete[] name; - return TRUE; + return true; } else if (strcmp(wxResourceBuffer, "static") != 0) { wxChar buf[300]; wxStrcpy(buf, _("Found ")); wxStrncat(buf, wxConvLibc.cMB2WX(wxResourceBuffer), 30); - wxStrcat(buf, _(", expected static, #include or #define\nwhilst parsing resource.")); + wxStrcat(buf, _(", expected static, #include or #define\nwhile parsing resource.")); wxLogWarning(buf); - return FALSE; + return false; } // char if (!wxGetResourceToken(fd)) { - wxLogWarning(_("Unexpected end of file whilst parsing resource.")); - *eof = TRUE; - return FALSE; + wxLogWarning(_("Unexpected end of file while parsing resource.")); + *eof = true; + return false; } if (strcmp(wxResourceBuffer, "char") != 0) { - wxLogWarning(_("Expected 'char' whilst parsing resource.")); - return FALSE; + wxLogWarning(_("Expected 'char' while parsing resource.")); + return false; } // *name if (!wxGetResourceToken(fd)) { - wxLogWarning(_("Unexpected end of file whilst parsing resource.")); - *eof = TRUE; - return FALSE; + wxLogWarning(_("Unexpected end of file while parsing resource.")); + *eof = true; + return false; } if (wxResourceBuffer[0] != '*') { - wxLogWarning(_("Expected '*' whilst parsing resource.")); - return FALSE; + wxLogWarning(_("Expected '*' while parsing resource.")); + return false; } char nameBuf[100]; strncpy(nameBuf, wxResourceBuffer+1, 99); @@ -1995,38 +2005,38 @@ bool wxResourceReadOneResource(wxInputStream *fd, wxExprDatabase& db, bool *eof, // = if (!wxGetResourceToken(fd)) { - wxLogWarning(_("Unexpected end of file whilst parsing resource.")); - *eof = TRUE; - return FALSE; + wxLogWarning(_("Unexpected end of file while parsing resource.")); + *eof = true; + return false; } if (strcmp(wxResourceBuffer, "=") != 0) { - wxLogWarning(_("Expected '=' whilst parsing resource.")); - return FALSE; + wxLogWarning(_("Expected '=' while parsing resource.")); + return false; } // String if (!wxGetResourceToken(fd)) { - wxLogWarning(_("Unexpected end of file whilst parsing resource.")); - *eof = TRUE; - return FALSE; + wxLogWarning(_("Unexpected end of file while parsing resource.")); + *eof = true; + return false; } else { if (!db.ReadPrologFromString(wxResourceBuffer)) { wxLogWarning(_("%s: ill-formed resource file syntax."), nameBuf); - return FALSE; + return false; } } // Semicolon if (!wxGetResourceToken(fd)) { - *eof = TRUE; + *eof = true; } - return TRUE; + return true; } /* @@ -2343,18 +2353,18 @@ long wxParseWindowStyle(const wxString& bitListString) word = wxResourceParseWord(WXSTRINGCAST bitListString, &i); while (word != NULL) { - bool found = FALSE; + bool found = false; int j; for (j = 0; j < wxResourceBitListCount; j++) if (wxStrcmp(wxResourceBitListTable[j].word, word) == 0) { bitList |= wxResourceBitListTable[j].bits; - found = TRUE; + found = true; break; } if (!found) { - wxLogWarning(_("Unrecognized style %s whilst parsing resource."), word); + wxLogWarning(_("Unrecognized style %s while parsing resource."), word); return 0; } word = wxResourceParseWord(WXSTRINGCAST bitListString, &i); @@ -2363,7 +2373,7 @@ long wxParseWindowStyle(const wxString& bitListString) } /* -* Load a bitmap from a wxWindows resource, choosing an optimum +* Load a bitmap from a wxWidgets resource, choosing an optimum * depth and appropriate type. */ @@ -2375,7 +2385,7 @@ wxBitmap wxResourceCreateBitmap(const wxString& resource, wxResourceTable *table wxItemResource *item = table->FindResource(resource); if (item) { - if ((item->GetType() == wxT("")) || (item->GetType() != wxT("wxBitmap"))) + if ((item->GetType().empty()) || (item->GetType() != wxT("wxBitmap"))) { wxLogWarning(_("%s not a bitmap resource specification."), (const wxChar*) resource); return wxNullBitmap; @@ -2386,10 +2396,10 @@ wxBitmap wxResourceCreateBitmap(const wxString& resource, wxResourceTable *table wxItemResource *optResource = (wxItemResource *) NULL; // Try to find optimum bitmap for this platform/colour depth - wxNode *node = item->GetChildren().First(); + wxNode *node = item->GetChildren().GetFirst(); while (node) { - wxItemResource *child = (wxItemResource *)node->Data(); + wxItemResource *child = (wxItemResource *)node->GetData(); int platform = (int)child->GetValue2(); int noColours = (int)child->GetValue3(); /* @@ -2460,7 +2470,7 @@ wxBitmap wxResourceCreateBitmap(const wxString& resource, wxResourceTable *table default: break; } - node = node->Next(); + node = node->GetNext(); } // If no matching resource, fail. if (!optResource) @@ -2517,7 +2527,7 @@ wxBitmap wxResourceCreateBitmap(const wxString& resource, wxResourceTable *table } /* -* Load an icon from a wxWindows resource, choosing an optimum +* Load an icon from a wxWidgets resource, choosing an optimum * depth and appropriate type. */ @@ -2529,7 +2539,7 @@ wxIcon wxResourceCreateIcon(const wxString& resource, wxResourceTable *table) wxItemResource *item = table->FindResource(resource); if (item) { - if ((item->GetType() == wxT("")) || wxStrcmp(item->GetType(), wxT("wxIcon")) != 0) + if ((item->GetType().empty()) || wxStrcmp(item->GetType(), wxT("wxIcon")) != 0) { wxLogWarning(_("%s not an icon resource specification."), (const wxChar*) resource); return wxNullIcon; @@ -2540,10 +2550,10 @@ wxIcon wxResourceCreateIcon(const wxString& resource, wxResourceTable *table) wxItemResource *optResource = (wxItemResource *) NULL; // Try to find optimum icon for this platform/colour depth - wxNode *node = item->GetChildren().First(); + wxNode *node = item->GetChildren().GetFirst(); while (node) { - wxItemResource *child = (wxItemResource *)node->Data(); + wxItemResource *child = (wxItemResource *)node->GetData(); int platform = (int)child->GetValue2(); int noColours = (int)child->GetValue3(); /* @@ -2614,7 +2624,7 @@ wxIcon wxResourceCreateIcon(const wxString& resource, wxResourceTable *table) default: break; } - node = node->Next(); + node = node->GetNext(); } // If no matching resource, fail. if (!optResource) @@ -2642,7 +2652,7 @@ wxIcon wxResourceCreateIcon(const wxString& resource, wxResourceTable *table) } case wxBITMAP_TYPE_XPM_DATA: { - // *** XPM ICON NOT YET IMPLEMENTED IN WXWINDOWS *** + // *** XPM ICON NOT YET IMPLEMENTED IN wxWidgets *** /* wxItemResource *item = table->FindResource(name); if (!item) @@ -2663,7 +2673,7 @@ wxIcon wxResourceCreateIcon(const wxString& resource, wxResourceTable *table) wxLogWarning(_("Icon resource specification %s not found."), (const wxChar*) resource); break; #else - return wxIcon(name, bitmapType); + return wxIcon(name, (wxBitmapType) bitmapType); #endif } } @@ -2681,13 +2691,13 @@ wxIcon wxResourceCreateIcon(const wxString& resource, wxResourceTable *table) wxMenu *wxResourceCreateMenu(wxItemResource *item) { wxMenu *menu = new wxMenu; - wxNode *node = item->GetChildren().First(); + wxNode *node = item->GetChildren().GetFirst(); while (node) { - wxItemResource *child = (wxItemResource *)node->Data(); - if ((child->GetType() != wxT("")) && (child->GetType() == wxT("wxMenuSeparator"))) + wxItemResource *child = (wxItemResource *)node->GetData(); + if ((!child->GetType().empty()) && (child->GetType() == wxT("wxMenuSeparator"))) menu->AppendSeparator(); - else if (child->GetChildren().Number() > 0) + else if (child->GetChildren().GetCount() > 0) { wxMenu *subMenu = wxResourceCreateMenu(child); if (subMenu) @@ -2697,7 +2707,7 @@ wxMenu *wxResourceCreateMenu(wxItemResource *item) { menu->Append((int)child->GetValue1(), child->GetTitle(), child->GetValue4(), (child->GetValue2() != 0)); } - node = node->Next(); + node = node->GetNext(); } return menu; } @@ -2708,18 +2718,18 @@ wxMenuBar *wxResourceCreateMenuBar(const wxString& resource, wxResourceTable *ta table = wxDefaultResourceTable; wxItemResource *menuResource = table->FindResource(resource); - if (menuResource && (menuResource->GetType() != wxT("")) && (menuResource->GetType() == wxT("wxMenu"))) + if (menuResource && (!menuResource->GetType().empty()) && (menuResource->GetType() == wxT("wxMenu"))) { if (!menuBar) menuBar = new wxMenuBar; - wxNode *node = menuResource->GetChildren().First(); + wxNode *node = menuResource->GetChildren().GetFirst(); while (node) { - wxItemResource *child = (wxItemResource *)node->Data(); + wxItemResource *child = (wxItemResource *)node->GetData(); wxMenu *menu = wxResourceCreateMenu(child); if (menu) menuBar->Append(menu, child->GetTitle()); - node = node->Next(); + node = node->GetNext(); } return menuBar; } @@ -2732,7 +2742,7 @@ wxMenu *wxResourceCreateMenu(const wxString& resource, wxResourceTable *table) table = wxDefaultResourceTable; wxItemResource *menuResource = table->FindResource(resource); - if (menuResource && (menuResource->GetType() != wxT("")) && (menuResource->GetType() == wxT("wxMenu"))) + if (menuResource && (!menuResource->GetType().empty()) && (menuResource->GetType() == wxT("wxMenu"))) // if (menuResource && (menuResource->GetType() == wxTYPE_MENU)) return wxResourceCreateMenu(menuResource); return (wxMenu *) NULL; @@ -2801,7 +2811,7 @@ bool wxResourceAddIdentifier(const wxString& name, int value, wxResourceTable *t table = wxDefaultResourceTable; table->identifiers.Put(name, (wxObject *)(long)value); - return TRUE; + return true; } int wxResourceGetIdentifier(const wxString& name, wxResourceTable *table) @@ -2824,7 +2834,7 @@ bool wxResourceParseIncludeFile(const wxString& f, wxResourceTable *table) FILE *fd = wxFopen(f, wxT("r")); if (!fd) { - return FALSE; + return false; } while (wxGetResourceToken(fd)) { @@ -2844,7 +2854,7 @@ bool wxResourceParseIncludeFile(const wxString& f, wxResourceTable *table) } } fclose(fd); - return TRUE; + return true; } /* @@ -2871,7 +2881,7 @@ static int ungetc_string() bool wxEatWhiteSpaceString(char *s) { - int ch = 0; + int ch; while ((ch = getc_string(s)) != EOF) { @@ -2884,18 +2894,17 @@ bool wxEatWhiteSpaceString(char *s) break; case '/': { - int prev_ch = ch; ch = getc_string(s); if (ch == EOF) { ungetc_string(); - return TRUE; + return true; } if (ch == '*') { // Eat C comment - prev_ch = 0; + int prev_ch = 0; while ((ch = getc_string(s)) != EOF) { if (ch == '/' && prev_ch == '*') @@ -2907,17 +2916,17 @@ bool wxEatWhiteSpaceString(char *s) { ungetc_string(); ungetc_string(); - return TRUE; + return true; } } break; default: ungetc_string(); - return TRUE; + return true; } } - return FALSE; + return false; } bool wxGetResourceTokenString(char *s) @@ -2939,7 +2948,7 @@ bool wxGetResourceTokenString(char *s) if (ch == EOF) { wxResourceBuffer[wxResourceBufferCount] = 0; - return FALSE; + return false; } // Escaped characters else if (ch == '\\') @@ -2978,9 +2987,9 @@ bool wxGetResourceTokenString(char *s) } wxResourceBuffer[wxResourceBufferCount] = 0; if (ch == EOF) - return FALSE; + return false; } - return TRUE; + return true; } /* @@ -2997,8 +3006,8 @@ bool wxResourceReadOneResourceString(char *s, wxExprDatabase& db, bool *eof, wxR // static or #define if (!wxGetResourceTokenString(s)) { - *eof = TRUE; - return FALSE; + *eof = true; + return false; } if (strcmp(wxResourceBuffer, "#define") == 0) @@ -3017,12 +3026,12 @@ bool wxResourceReadOneResourceString(char *s, wxExprDatabase& db, bool *eof, wxR wxLogWarning(_("#define %s must be an integer."), name); delete[] name; delete[] value; - return FALSE; + return false; } delete[] name; delete[] value; - return TRUE; + return true; } /* else if (strcmp(wxResourceBuffer, "#include") == 0) @@ -3042,7 +3051,7 @@ bool wxResourceReadOneResourceString(char *s, wxExprDatabase& db, bool *eof, wxR wxLogWarning(buf); } delete[] name; - return TRUE; + return true; } */ else if (strcmp(wxResourceBuffer, "static") != 0) @@ -3050,37 +3059,37 @@ bool wxResourceReadOneResourceString(char *s, wxExprDatabase& db, bool *eof, wxR wxChar buf[300]; wxStrcpy(buf, _("Found ")); wxStrncat(buf, wxConvCurrent->cMB2WX(wxResourceBuffer), 30); - wxStrcat(buf, _(", expected static, #include or #define\nwhilst parsing resource.")); + wxStrcat(buf, _(", expected static, #include or #define\nwhile parsing resource.")); wxLogWarning(buf); - return FALSE; + return false; } // char if (!wxGetResourceTokenString(s)) { - wxLogWarning(_("Unexpected end of file whilst parsing resource.")); - *eof = TRUE; - return FALSE; + wxLogWarning(_("Unexpected end of file while parsing resource.")); + *eof = true; + return false; } if (strcmp(wxResourceBuffer, "char") != 0) { - wxLogWarning(_("Expected 'char' whilst parsing resource.")); - return FALSE; + wxLogWarning(_("Expected 'char' while parsing resource.")); + return false; } // *name if (!wxGetResourceTokenString(s)) { - wxLogWarning(_("Unexpected end of file whilst parsing resource.")); - *eof = TRUE; - return FALSE; + wxLogWarning(_("Unexpected end of file while parsing resource.")); + *eof = true; + return false; } if (wxResourceBuffer[0] != '*') { - wxLogWarning(_("Expected '*' whilst parsing resource.")); - return FALSE; + wxLogWarning(_("Expected '*' while parsing resource.")); + return false; } wxChar nameBuf[100]; wxMB2WX(nameBuf, wxResourceBuffer+1, 99); @@ -3089,41 +3098,41 @@ bool wxResourceReadOneResourceString(char *s, wxExprDatabase& db, bool *eof, wxR // = if (!wxGetResourceTokenString(s)) { - wxLogWarning(_("Unexpected end of file whilst parsing resource.")); - *eof = TRUE; - return FALSE; + wxLogWarning(_("Unexpected end of file while parsing resource.")); + *eof = true; + return false; } if (strcmp(wxResourceBuffer, "=") != 0) { - wxLogWarning(_("Expected '=' whilst parsing resource.")); - return FALSE; + wxLogWarning(_("Expected '=' while parsing resource.")); + return false; } // String if (!wxGetResourceTokenString(s)) { - wxLogWarning(_("Unexpected end of file whilst parsing resource.")); - *eof = TRUE; - return FALSE; + wxLogWarning(_("Unexpected end of file while parsing resource.")); + *eof = true; + return false; } else { if (!db.ReadPrologFromString(wxResourceBuffer)) { wxLogWarning(_("%s: ill-formed resource file syntax."), nameBuf); - return FALSE; + return false; } } // Semicolon if (!wxGetResourceTokenString(s)) { - *eof = TRUE; + *eof = true; } - return TRUE; + return true; } -bool wxResourceParseString(const wxString& s, wxResourceTable *table) +bool wxResourceParseString(const wxString& s, wxResourceTable *WXUNUSED(table)) { #if wxUSE_UNICODE return wxResourceParseString( (char*)s.mb_str().data() ); @@ -3138,7 +3147,7 @@ bool wxResourceParseString(char *s, wxResourceTable *table) table = wxDefaultResourceTable; if (!s) - return FALSE; + return false; // Turn backslashes into spaces if (s) @@ -3156,7 +3165,7 @@ bool wxResourceParseString(char *s, wxResourceTable *table) wxExprDatabase db; wxResourceStringPtr = 0; - bool eof = FALSE; + bool eof = false; while (wxResourceReadOneResourceString(s, db, &eof, table) && !eof) { // Loop @@ -3175,9 +3184,9 @@ bool wxLoadFromResource(wxWindow* thisWindow, wxWindow *parent, const wxString& wxItemResource *resource = table->FindResource((const wxChar *)resourceName); // if (!resource || (resource->GetType() != wxTYPE_DIALOG_BOX)) - if (!resource || (resource->GetType() == wxT("")) || + if (!resource || (resource->GetType().empty()) || ! ((resource->GetType() == wxT("wxDialog")) || (resource->GetType() == wxT("wxPanel")))) - return FALSE; + return false; wxString title(resource->GetTitle()); long theWindowStyle = resource->GetStyle(); @@ -3195,8 +3204,8 @@ bool wxLoadFromResource(wxWindow* thisWindow, wxWindow *parent, const wxString& { wxDialog *dialogBox = (wxDialog *)thisWindow; long modalStyle = isModal ? wxDIALOG_MODAL : 0; - if (!dialogBox->Create(parent, -1, title, wxPoint(x, y), wxSize(width, height), theWindowStyle|modalStyle, name)) - return FALSE; + if (!dialogBox->Create(parent, wxID_ANY, title, wxPoint(x, y), wxSize(width, height), theWindowStyle|modalStyle, name)) + return false; // Only reset the client size if we know we're not going to do it again below. if ((resource->GetResourceStyle() & wxRESOURCE_DIALOG_UNITS) == 0) @@ -3205,13 +3214,13 @@ bool wxLoadFromResource(wxWindow* thisWindow, wxWindow *parent, const wxString& else if (thisWindow->IsKindOf(CLASSINFO(wxPanel))) { wxPanel* panel = (wxPanel *)thisWindow; - if (!panel->Create(parent, -1, wxPoint(x, y), wxSize(width, height), theWindowStyle | wxTAB_TRAVERSAL, name)) - return FALSE; + if (!panel->Create(parent, wxID_ANY, wxPoint(x, y), wxSize(width, height), theWindowStyle | wxTAB_TRAVERSAL, name)) + return false; } else { - if (!((wxWindow *)thisWindow)->Create(parent, -1, wxPoint(x, y), wxSize(width, height), theWindowStyle, name)) - return FALSE; + if (!((wxWindow *)thisWindow)->Create(parent, wxID_ANY, wxPoint(x, y), wxSize(width, height), theWindowStyle, name)) + return false; } } @@ -3246,16 +3255,16 @@ bool wxLoadFromResource(wxWindow* thisWindow, wxWindow *parent, const wxString& } // Now create children - wxNode *node = resource->GetChildren().First(); + wxNode *node = resource->GetChildren().GetFirst(); while (node) { - wxItemResource *childResource = (wxItemResource *)node->Data(); + wxItemResource *childResource = (wxItemResource *)node->GetData(); (void) wxCreateItem(thisWindow, childResource, resource, table); - node = node->Next(); + node = node->GetNext(); } - return TRUE; + return true; } wxControl *wxCreateItem(wxWindow* thisWindow, const wxItemResource *resource, const wxItemResource* parentResource, const wxResourceTable *table)