]> git.saurik.com Git - wxWidgets.git/blobdiff - contrib/src/deprecated/resource.cpp
Warning fixes after recent library changes.
[wxWidgets.git] / contrib / src / deprecated / resource.cpp
index 3d0188a9ec317a4c8091d3de51f397384707a7e8..8837544255a57a37426b017ce72943d0373599b2 100644 (file)
@@ -96,7 +96,7 @@ static inline wxChar* copystring(const wxChar* s)
 
 // 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);
@@ -137,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)
@@ -151,13 +151,13 @@ 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;
 }
@@ -196,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.
@@ -215,7 +215,7 @@ 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 = Next();
+        wxHashTable::Node *node = Next();
         while (node != NULL)
         {
             wxItemResource *parent = (wxItemResource *)node->GetData();
@@ -228,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 )
@@ -239,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) ;
@@ -253,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
@@ -269,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);
@@ -286,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)
@@ -298,21 +298,21 @@ 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();
+        wxHashTable::Node *next = Next();
         wxItemResource *item = (wxItemResource *)node->GetData();
         delete item;
         delete node;
@@ -324,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);
 
@@ -346,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();
@@ -375,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();
@@ -443,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
@@ -572,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
         }
@@ -596,7 +596,7 @@ bool wxResourceInterpretResources(wxResourceTable& table, wxExprDatabase& db)
         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"))
@@ -611,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->GetNext();
     }
-    return TRUE;
+    return true;
 }
 
 static const wxChar *g_ValidControlClasses[] =
@@ -649,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)
@@ -661,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;
@@ -706,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;
@@ -735,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;
@@ -745,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;
@@ -770,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);
@@ -802,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);
@@ -887,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"))
@@ -900,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"));
@@ -1410,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"))
@@ -1480,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);
@@ -1526,7 +1532,7 @@ bool wxReallocateResourceBuffer()
     {
         wxResourceBufferSize = 1000;
         wxResourceBuffer = new char[wxResourceBufferSize];
-        return TRUE;
+        return true;
     }
     if (wxResourceBuffer)
     {
@@ -1537,7 +1543,7 @@ bool wxReallocateResourceBuffer()
         wxResourceBuffer = tmp;
         wxResourceBufferSize = newSize;
     }
-    return TRUE;
+    return true;
 }
 
 static bool wxEatWhiteSpace(FILE *fd)
@@ -1560,7 +1566,7 @@ static bool wxEatWhiteSpace(FILE *fd)
                 if (ch == EOF)
                 {
                     ungetc(prev_ch, fd);
-                    return TRUE;
+                    return true;
                 }
 
                 if (ch == '*')
@@ -1584,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
@@ -1614,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);
@@ -1633,7 +1639,7 @@ static bool wxEatWhiteSpace(wxInputStream *is)
             }
         }
         else // False alarm
-            return FALSE;
+            return false;
     }
     else
         is->Ungetch(ch);
@@ -1659,7 +1665,7 @@ bool wxGetResourceToken(FILE *fd)
             if (ch == EOF)
             {
                 wxResourceBuffer[wxResourceBufferCount] = 0;
-                return FALSE;
+                return false;
             }
             // Escaped characters
             else if (ch == '\\')
@@ -1698,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)
@@ -1722,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)
@@ -1763,9 +1769,9 @@ bool wxGetResourceToken(wxInputStream *is)
         }
         wxResourceBuffer[wxResourceBufferCount] = 0;
         if (ch == EOF)
-            return FALSE;
+            return false;
     }
-    return TRUE;
+    return true;
 }
 
 /*
@@ -1782,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)
@@ -1802,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)
     {
@@ -1824,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);
@@ -1870,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)
@@ -1912,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)
@@ -1932,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)
     {
@@ -1954,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);
@@ -1999,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;
 }
 
 /*
@@ -2347,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);
@@ -2367,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.
 */
 
@@ -2379,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;
@@ -2521,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.
 */
 
@@ -2533,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;
@@ -2646,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)
@@ -2689,7 +2695,7 @@ wxMenu *wxResourceCreateMenu(wxItemResource *item)
     while (node)
     {
         wxItemResource *child = (wxItemResource *)node->GetData();
-        if ((child->GetType() != wxT("")) && (child->GetType() == wxT("wxMenuSeparator")))
+        if ((!child->GetType().empty()) && (child->GetType() == wxT("wxMenuSeparator")))
             menu->AppendSeparator();
         else if (child->GetChildren().GetCount() > 0)
         {
@@ -2712,7 +2718,7 @@ 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;
@@ -2736,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;
@@ -2805,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)
@@ -2828,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))
     {
@@ -2848,7 +2854,7 @@ bool wxResourceParseIncludeFile(const wxString& f, wxResourceTable *table)
         }
     }
     fclose(fd);
-    return TRUE;
+    return true;
 }
 
 /*
@@ -2892,7 +2898,7 @@ bool wxEatWhiteSpaceString(char *s)
                 if (ch == EOF)
                 {
                     ungetc_string();
-                    return TRUE;
+                    return true;
                 }
 
                 if (ch == '*')
@@ -2910,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)
@@ -2942,7 +2948,7 @@ bool wxGetResourceTokenString(char *s)
             if (ch == EOF)
             {
                 wxResourceBuffer[wxResourceBufferCount] = 0;
-                return FALSE;
+                return false;
             }
             // Escaped characters
             else if (ch == '\\')
@@ -2981,9 +2987,9 @@ bool wxGetResourceTokenString(char *s)
         }
         wxResourceBuffer[wxResourceBufferCount] = 0;
         if (ch == EOF)
-            return FALSE;
+            return false;
     }
-    return TRUE;
+    return true;
 }
 
 /*
@@ -3000,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)
@@ -3020,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)
@@ -3045,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)
@@ -3053,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);
@@ -3092,38 +3098,38 @@ 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 *WXUNUSED(table))
@@ -3141,7 +3147,7 @@ bool wxResourceParseString(char *s, wxResourceTable *table)
         table = wxDefaultResourceTable;
 
     if (!s)
-        return FALSE;
+        return false;
 
     // Turn backslashes into spaces
     if (s)
@@ -3159,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
@@ -3178,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();
@@ -3198,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)
@@ -3208,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;
         }
     }
 
@@ -3258,7 +3264,7 @@ bool wxLoadFromResource(wxWindow* thisWindow, wxWindow *parent, const wxString&
 
         node = node->GetNext();
     }
-    return TRUE;
+    return true;
 }
 
 wxControl *wxCreateItem(wxWindow* thisWindow, const wxItemResource *resource, const wxItemResource* parentResource, const wxResourceTable *table)