X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/1486ed877a09d1a31e269ef7c0bc2cd9426f2e0a..9d8c2f419f522408a922277df330d4c4511befb9:/contrib/src/xml/xmlres.cpp?ds=sidebyside diff --git a/contrib/src/xml/xmlres.cpp b/contrib/src/xml/xmlres.cpp index e3d304dc8a..7a5bd4ee74 100644 --- a/contrib/src/xml/xmlres.cpp +++ b/contrib/src/xml/xmlres.cpp @@ -42,14 +42,16 @@ WX_DEFINE_OBJARRAY(wxXmlResourceDataRecords); wxXmlResource::wxXmlResource(bool use_locale) { - m_Handlers.DeleteContents(TRUE); - m_UseLocale = use_locale; + m_handlers.DeleteContents(TRUE); + m_useLocale = use_locale; + m_version = -1; } wxXmlResource::wxXmlResource(const wxString& filemask, bool use_locale) { - m_UseLocale = use_locale; - m_Handlers.DeleteContents(TRUE); + m_useLocale = use_locale; + m_version = -1; + m_handlers.DeleteContents(TRUE); Load(filemask); } @@ -64,6 +66,7 @@ bool wxXmlResource::Load(const wxString& filemask) wxString fnd; wxXmlResourceDataRecord *drec; bool iswild = wxIsWild(filemask); + bool rt = TRUE; #if wxUSE_FILESYSTEM wxFileSystem fsys; @@ -83,24 +86,15 @@ bool wxXmlResource::Load(const wxString& filemask) if (filemask.Lower().Matches("*.zip") || filemask.Lower().Matches("*.rsc")) { - wxFileSystem fs2; - wxString fnd2; - - fnd2 = fs2.FindFirst(fnd + wxT("#zip:*.xmb"), wxFILE); - while (!!fnd2) - { - drec = new wxXmlResourceDataRecord; - drec->File = fnd2; - m_Data.Add(drec); - fnd2 = fs2.FindNext(); - } + rt = rt && Load(fnd + wxT("#zip:*.xmb")); + rt = rt && Load(fnd + wxT("#zip:*.xrc")); } else #endif { drec = new wxXmlResourceDataRecord; drec->File = fnd; - m_Data.Add(drec); + m_data.Add(drec); } if (iswild) @@ -110,14 +104,14 @@ bool wxXmlResource::Load(const wxString& filemask) } # undef wxXmlFindFirst # undef wxXmlFindNext - return TRUE; + return rt; } void wxXmlResource::AddHandler(wxXmlResourceHandler *handler) { - m_Handlers.Append(handler); + m_handlers.Append(handler); handler->SetParentResource(this); } @@ -125,7 +119,7 @@ void wxXmlResource::AddHandler(wxXmlResourceHandler *handler) void wxXmlResource::ClearHandlers() { - m_Handlers.Clear(); + m_handlers.Clear(); } @@ -255,62 +249,78 @@ void wxXmlResource::UpdateResources() { bool modif; # if wxUSE_FILESYSTEM - wxFSFile *file; + wxFSFile *file = NULL; wxFileSystem fsys; # endif - for (size_t i = 0; i < m_Data.GetCount(); i++) + for (size_t i = 0; i < m_data.GetCount(); i++) { - modif = (m_Data[i].Doc == NULL); + modif = (m_data[i].Doc == NULL); if (!modif) { # if wxUSE_FILESYSTEM - file = fsys.OpenFile(m_Data[i].File); - modif = file && file->GetModificationTime() > m_Data[i].Time; + file = fsys.OpenFile(m_data[i].File); + modif = file && file->GetModificationTime() > m_data[i].Time; if (!file) - wxLogError(_("Cannot open file '%s'."), m_Data[i].File.c_str()); - delete file; + wxLogError(_("Cannot open file '%s'."), m_data[i].File.c_str()); + wxDELETE(file); # else - modif = wxDateTime(wxFileModificationTime(m_Data[i].File)) > m_Data[i].Time; + modif = wxDateTime(wxFileModificationTime(m_data[i].File)) > m_data[i].Time; # endif } if (modif) { - wxInputStream *stream; + wxInputStream *stream = NULL; # if wxUSE_FILESYSTEM - file = fsys.OpenFile(m_Data[i].File); - stream = file->GetStream(); + file = fsys.OpenFile(m_data[i].File); + if (file) + stream = file->GetStream(); # else - stream = new wxFileInputStream(m_Data[i].File); + stream = new wxFileInputStream(m_data[i].File); # endif if (stream) { - delete m_Data[i].Doc; - m_Data[i].Doc = new wxXmlDocument; + delete m_data[i].Doc; + m_data[i].Doc = new wxXmlDocument; } - if (!stream || !m_Data[i].Doc->Load(*stream)) + if (!stream || !m_data[i].Doc->Load(*stream)) { - wxLogError(_("Cannot load resources from file '%s'."), m_Data[i].File.c_str()); - delete m_Data[i].Doc; - m_Data[i].Doc = NULL; + wxLogError(_("Cannot load resources from file '%s'."), m_data[i].File.c_str()); + wxDELETE(m_data[i].Doc); } - else if (m_Data[i].Doc->GetRoot()->GetName() != wxT("resource")) + else if (m_data[i].Doc->GetRoot()->GetName() != wxT("resource")) { - wxLogError(_("Invalid XML resource '%s': doesn't have root node 'resource'."), m_Data[i].File.c_str()); - delete m_Data[i].Doc; - m_Data[i].Doc = NULL; + wxLogError(_("Invalid XML resource '%s': doesn't have root node 'resource'."), m_data[i].File.c_str()); + wxDELETE(m_data[i].Doc); } else - ProcessPlatformProperty(m_Data[i].Doc->GetRoot()); + { + long version; + int v1, v2, v3, v4; + wxString verstr = m_data[i].Doc->GetRoot()->GetPropVal( + wxT("version"), wxT("0.0.0.0")); + if (wxSscanf(verstr.c_str(), wxT("%i.%i.%i.%i"), + &v1, &v2, &v3, &v4) == 4) + version = v1*256*256*256+v2*256*256+v3*256+v4; + else + version = 0; + if (m_version == -1) + m_version = version; + if (m_version != version) + wxLogError(_("Resource files must have same version number!")); + + ProcessPlatformProperty(m_data[i].Doc->GetRoot()); + m_data[i].Time = file->GetModificationTime(); + } # if wxUSE_FILESYSTEM - delete file; + wxDELETE(file); # else - delete stream; + wxDELETE(stream); # endif } } @@ -323,10 +333,10 @@ wxXmlNode *wxXmlResource::FindResource(const wxString& name, const wxString& cla UpdateResources(); //ensure everything is up-to-date wxString dummy; - for (size_t f = 0; f < m_Data.GetCount(); f++) + for (size_t f = 0; f < m_data.GetCount(); f++) { - if (m_Data[f].Doc == NULL || m_Data[f].Doc->GetRoot() == NULL) continue; - for (wxXmlNode *node = m_Data[f].Doc->GetRoot()->GetChildren(); + if (m_data[f].Doc == NULL || m_data[f].Doc->GetRoot() == NULL) continue; + for (wxXmlNode *node = m_data[f].Doc->GetRoot()->GetChildren(); node; node = node->GetNext()) if (node->GetType() == wxXML_ELEMENT_NODE && (!classname || @@ -336,7 +346,7 @@ wxXmlNode *wxXmlResource::FindResource(const wxString& name, const wxString& cla dummy == name) { #if wxUSE_FILESYSTEM - m_CurFileSystem.ChangePathTo(m_Data[f].File); + m_curFileSystem.ChangePathTo(m_data[f].File); #endif return node; } @@ -355,7 +365,7 @@ wxObject *wxXmlResource::CreateResFromNode(wxXmlNode *node, wxObject *parent, wx wxXmlResourceHandler *handler; wxObject *ret; - wxNode * ND = m_Handlers.GetFirst(); + wxNode * ND = m_handlers.GetFirst(); while (ND) { handler = (wxXmlResourceHandler*)ND->GetData(); @@ -382,32 +392,32 @@ wxObject *wxXmlResource::CreateResFromNode(wxXmlNode *node, wxObject *parent, wx wxXmlResourceHandler::wxXmlResourceHandler() - : m_Node(NULL), m_Parent(NULL), m_Instance(NULL), - m_ParentAsWindow(NULL), m_InstanceAsWindow(NULL) + : m_node(NULL), m_parent(NULL), m_instance(NULL), + m_parentAsWindow(NULL), m_instanceAsWindow(NULL) {} wxObject *wxXmlResourceHandler::CreateResource(wxXmlNode *node, wxObject *parent, wxObject *instance) { - wxXmlNode *myNode = m_Node; - wxString myClass = m_Class; - wxObject *myParent = m_Parent, *myInstance = m_Instance; - wxWindow *myParentAW = m_ParentAsWindow, *myInstanceAW = m_InstanceAsWindow; + wxXmlNode *myNode = m_node; + wxString myClass = m_class; + wxObject *myParent = m_parent, *myInstance = m_instance; + wxWindow *myParentAW = m_parentAsWindow, *myInstanceAW = m_instanceAsWindow; - m_Node = node; - m_Class = node->GetPropVal(wxT("class"), wxEmptyString); - m_Parent = parent; - m_Instance = instance; - m_ParentAsWindow = wxDynamicCast(m_Parent, wxWindow); - m_InstanceAsWindow = wxDynamicCast(m_Instance, wxWindow); + m_node = node; + m_class = node->GetPropVal(wxT("class"), wxEmptyString); + m_parent = parent; + m_instance = instance; + m_parentAsWindow = wxDynamicCast(m_parent, wxWindow); + m_instanceAsWindow = wxDynamicCast(m_instance, wxWindow); wxObject *returned = DoCreateResource(); - m_Node = myNode; - m_Class = myClass; - m_Parent = myParent; m_ParentAsWindow = myParentAW; - m_Instance = myInstance; m_InstanceAsWindow = myInstanceAW; + m_node = myNode; + m_class = myClass; + m_parent = myParent; m_parentAsWindow = myParentAW; + m_instance = myInstance; m_instanceAsWindow = myInstanceAW; return returned; } @@ -415,8 +425,8 @@ wxObject *wxXmlResourceHandler::CreateResource(wxXmlNode *node, wxObject *parent void wxXmlResourceHandler::AddStyle(const wxString& name, int value) { - m_StyleNames.Add(name); - m_StyleValues.Add(value); + m_styleNames.Add(name); + m_styleValues.Add(value); } @@ -454,9 +464,9 @@ int wxXmlResourceHandler::GetStyle(const wxString& param, int defaults) while (tkn.HasMoreTokens()) { fl = tkn.GetNextToken(); - index = m_StyleNames.Index(fl); + index = m_styleNames.Index(fl); if (index != wxNOT_FOUND) - style |= m_StyleValues[index]; + style |= m_styleValues[index]; else wxLogError(_("Unknown style flag ") + fl); } @@ -470,30 +480,40 @@ wxString wxXmlResourceHandler::GetText(const wxString& param) wxString str1 = GetParamValue(param); wxString str2; const wxChar *dt; + wxChar amp_char; + + // VS: First version of XML resources used $ instead of & (which is illegal in XML), + // but later I realized that '_' fits this purpose much better (because + // &File means "File with F underlined"). + if (m_resource->CompareVersion(2,3,0,1) < 0) + amp_char = wxT('$'); + else + amp_char = wxT('_'); for (dt = str1.c_str(); *dt; dt++) { - // Remap $ to &, map $$ to $ (for things like "&File..." -- - // this is illegal in XML, so we use "$File..."): - if (*dt == '$') - switch (*(++dt)) - { - case '$' : str2 << '$'; break; - default : str2 << '&' << *dt; break; - } - // Remap \n to CR, \r LF, \t to TAB: - else if (*dt == '\\') + // Remap amp_char to &, map double amp_char to amp_char (for things + // like "&File..." -- this is illegal in XML, so we use "_File..."): + if (*dt == amp_char) + { + if ( *(++dt) == amp_char ) + str2 << amp_char; + else + str2 << wxT('&') << *dt; + } + // Remap \n to CR, \r to LF, \t to TAB: + else if (*dt == wxT('\\')) switch (*(++dt)) { - case 'n' : str2 << '\n'; break; - case 't' : str2 << '\t'; break; - case 'r' : str2 << '\r'; break; - default : str2 << '\\' << *dt; break; + case wxT('n') : str2 << wxT('\n'); break; + case wxT('t') : str2 << wxT('\t'); break; + case wxT('r') : str2 << wxT('\r'); break; + default : str2 << wxT('\\') << *dt; break; } else str2 << *dt; } - if (m_Resource->GetUseLocale()) + if (m_resource->GetUseLocale()) return wxGetTranslation(str2); else return str2; @@ -541,7 +561,7 @@ int wxXmlResourceHandler::GetID() wxString wxXmlResourceHandler::GetName() { - return m_Node->GetPropVal(wxT("name"), wxT("-1")); + return m_node->GetPropVal(wxT("name"), wxT("-1")); } @@ -561,7 +581,7 @@ wxColour wxXmlResourceHandler::GetColour(const wxString& param) wxString v = GetParamValue(param); unsigned long tmp = 0; - if (v.Length() != 7 || v[0] != wxT('#') || + if (v.Length() != 7 || v[0u] != wxT('#') || wxSscanf(v.c_str(), wxT("#%lX"), &tmp) != 1) { wxLogError(_("XML resource: Incorrect colour specification '%s' for property '%s'."), @@ -621,7 +641,7 @@ wxIcon wxXmlResourceHandler::GetIcon(const wxString& param, wxSize size) wxXmlNode *wxXmlResourceHandler::GetParamNode(const wxString& param) { - wxXmlNode *n = m_Node->GetChildren(); + wxXmlNode *n = m_node->GetChildren(); while (n) { @@ -654,7 +674,7 @@ wxString wxXmlResourceHandler::GetNodeContent(wxXmlNode *node) wxString wxXmlResourceHandler::GetParamValue(const wxString& param) { if (param.IsEmpty()) - return GetNodeContent(m_Node); + return GetNodeContent(m_node); else return GetNodeContent(GetParamNode(param)); } @@ -680,10 +700,10 @@ wxSize wxXmlResourceHandler::GetSize(const wxString& param) if (is_dlg) { - if (m_InstanceAsWindow) - return wxDLG_UNIT(m_InstanceAsWindow, wxSize(sx, sy)); - else if (m_ParentAsWindow) - return wxDLG_UNIT(m_ParentAsWindow, wxSize(sx, sy)); + if (m_instanceAsWindow) + return wxDLG_UNIT(m_instanceAsWindow, wxSize(sx, sy)); + else if (m_parentAsWindow) + return wxDLG_UNIT(m_parentAsWindow, wxSize(sx, sy)); else { wxLogError(_("Cannot convert dialog units: dialog unknown.")); @@ -721,10 +741,10 @@ wxCoord wxXmlResourceHandler::GetDimension(const wxString& param, wxCoord defaul if (is_dlg) { - if (m_InstanceAsWindow) - return wxDLG_UNIT(m_InstanceAsWindow, wxSize(sx, 0)).x; - else if (m_ParentAsWindow) - return wxDLG_UNIT(m_ParentAsWindow, wxSize(sx, 0)).x; + if (m_instanceAsWindow) + return wxDLG_UNIT(m_instanceAsWindow, wxSize(sx, 0)).x; + else if (m_parentAsWindow) + return wxDLG_UNIT(m_parentAsWindow, wxSize(sx, 0)).x; else { wxLogError(_("Cannot convert dialog units: dialog unknown.")); @@ -745,8 +765,8 @@ wxFont wxXmlResourceHandler::GetFont(const wxString& param) return wxNullFont; } - wxXmlNode *oldnode = m_Node; - m_Node = font_node; + wxXmlNode *oldnode = m_node; + m_node = font_node; long size = GetLong(wxT("size"), 12); @@ -789,7 +809,7 @@ wxFont wxXmlResourceHandler::GetFont(const wxString& param) } } - m_Node = oldnode; + m_node = oldnode; wxFont font(size, ifamily, istyle, iweight, underlined, facename, enc); return font; @@ -823,7 +843,7 @@ void wxXmlResourceHandler::SetupWindow(wxWindow *wnd) void wxXmlResourceHandler::CreateChildren(wxObject *parent, bool this_hnd_only) { - wxXmlNode *n = m_Node->GetChildren(); + wxXmlNode *n = m_node->GetChildren(); while (n) { @@ -833,7 +853,7 @@ void wxXmlResourceHandler::CreateChildren(wxObject *parent, bool this_hnd_only) if (this_hnd_only && CanHandle(n)) CreateResource(n, parent, NULL); else - m_Resource->CreateResFromNode(n, parent, NULL); + m_resource->CreateResFromNode(n, parent, NULL); } n = n->GetNext(); } @@ -843,7 +863,7 @@ void wxXmlResourceHandler::CreateChildren(wxObject *parent, bool this_hnd_only) void wxXmlResourceHandler::CreateChildrenPrivately(wxObject *parent, wxXmlNode *rootnode) { wxXmlNode *root; - if (rootnode == NULL) root = m_Node; else root = rootnode; + if (rootnode == NULL) root = m_node; else root = rootnode; wxXmlNode *n = root->GetChildren(); while (n)