Forgot to commit

[wxWidgets.git] / src / common / fs_mem.cpp

diff --git a/src/common/fs_mem.cpp b/src/common/fs_mem.cpp
index 33cc18dcd086f4b70ebfb54534e876a0510dc59c..f13ab26c5d4511d29d33af931c4616c281b77c8c 100644 (file)
--- a/src/common/fs_mem.cpp
+++ b/src/common/fs_mem.cpp
@@ -20,7 +20,6 @@
 #ifndef WX_PRECOMP
     #include "wx/intl.h"
     #include "wx/log.h"
 #ifndef WX_PRECOMP
     #include "wx/intl.h"
     #include "wx/log.h"

-    #include "wx/hash.h"

     #include "wx/wxcrtvararg.h"
     #if wxUSE_GUI
         #include "wx/image.h"
     #include "wx/wxcrtvararg.h"
     #if wxUSE_GUI
         #include "wx/image.h"


@@ -29,10 +28,11 @@
 
 #include "wx/mstream.h"
 
 
 #include "wx/mstream.h"
 

-class MemFSHashObj : public wxObject
+// represents a file entry in wxMemoryFS
+class wxMemoryFSFile

 {
 public:
 {
 public:

-    MemFSHashObj(const void *data, size_t len, const wxString& mime)
+    wxMemoryFSFile(const void *data, size_t len, const wxString& mime)

     {
         m_Data = new char[len];
         memcpy(m_Data, data, len);
     {
         m_Data = new char[len];
         memcpy(m_Data, data, len);


@@ -41,7 +41,7 @@ public:
         InitTime();
     }
 
         InitTime();
     }
 

-    MemFSHashObj(const wxMemoryOutputStream& stream, const wxString& mime)
+    wxMemoryFSFile(const wxMemoryOutputStream& stream, const wxString& mime)

     {
         m_Len = stream.GetSize();
         m_Data = new char[m_Len];
     {
         m_Len = stream.GetSize();
         m_Data = new char[m_Len];


@@ -50,7 +50,7 @@ public:
         InitTime();
     }
 
         InitTime();
     }
 

-    virtual ~MemFSHashObj()
+    virtual ~wxMemoryFSFile()

     {
         delete[] m_Data;
     }
     {
         delete[] m_Data;
     }


@@ -62,8 +62,6 @@ public:
     wxDateTime m_Time;
 #endif // wxUSE_DATETIME
 
     wxDateTime m_Time;
 #endif // wxUSE_DATETIME
 

-DECLARE_NO_COPY_CLASS(MemFSHashObj)
-

 private:
     void InitTime()
     {
 private:
     void InitTime()
     {


@@ -71,6 +69,8 @@ private:
         m_Time = wxDateTime::Now();
 #endif // wxUSE_DATETIME
     }
         m_Time = wxDateTime::Now();
 #endif // wxUSE_DATETIME
     }

+
+    DECLARE_NO_COPY_CLASS(wxMemoryFSFile)

 };
 
 #if wxUSE_BASE
 };
 
 #if wxUSE_BASE


@@ -81,7 +81,7 @@ private:
 //--------------------------------------------------------------------------------
 
 
 //--------------------------------------------------------------------------------
 
 

-wxHashTable *wxMemoryFSHandlerBase::m_Hash = NULL;
+wxMemoryFSHash wxMemoryFSHandlerBase::m_Hash;

 
 
 wxMemoryFSHandlerBase::wxMemoryFSHandlerBase() : wxFileSystemHandler()
 
 
 wxMemoryFSHandlerBase::wxMemoryFSHandlerBase() : wxFileSystemHandler()


@@ -93,13 +93,7 @@ wxMemoryFSHandlerBase::~wxMemoryFSHandlerBase()
     // as only one copy of FS handler is supposed to exist, we may silently
     // delete static data here. (There is no way how to remove FS handler from
     // wxFileSystem other than releasing _all_ handlers.)
     // as only one copy of FS handler is supposed to exist, we may silently
     // delete static data here. (There is no way how to remove FS handler from
     // wxFileSystem other than releasing _all_ handlers.)

-
-    if (m_Hash)
-    {
-        WX_CLEAR_HASH_TABLE(*m_Hash);
-        delete m_Hash;
-        m_Hash = NULL;
-    }
+    WX_CLEAR_HASH_MAP(wxMemoryFSHash, m_Hash);

 }
 
 bool wxMemoryFSHandlerBase::CanOpen(const wxString& location)
 }
 
 bool wxMemoryFSHandlerBase::CanOpen(const wxString& location)


@@ -110,46 +104,70 @@ bool wxMemoryFSHandlerBase::CanOpen(const wxString& location)
 wxFSFile * wxMemoryFSHandlerBase::OpenFile(wxFileSystem& WXUNUSED(fs),
                                            const wxString& location)
 {
 wxFSFile * wxMemoryFSHandlerBase::OpenFile(wxFileSystem& WXUNUSED(fs),
                                            const wxString& location)
 {

-    if ( !m_Hash )
+    wxMemoryFSHash::const_iterator i = m_Hash.find(GetRightLocation(location));
+    if ( i == m_Hash.end() )

         return NULL;
 
         return NULL;
 

-    MemFSHashObj *obj = (MemFSHashObj*) m_Hash->Get(GetRightLocation(location));
-    if ( !obj )
-        return NULL;
+    const wxMemoryFSFile * const obj = i->second;

 
 

-    else return new wxFSFile(new wxMemoryInputStream(obj->m_Data, obj->m_Len),
-                        location,
-                        obj->m_MimeType,
-                        GetAnchor(location)
+    return new wxFSFile
+               (
+                    new wxMemoryInputStream(obj->m_Data, obj->m_Len),
+                    location,
+                    obj->m_MimeType,
+                    GetAnchor(location)

 #if wxUSE_DATETIME
 #if wxUSE_DATETIME

-                        , obj->m_Time
+                    , obj->m_Time

 #endif // wxUSE_DATETIME
 #endif // wxUSE_DATETIME

-                        );
+               );

 }
 
 }
 

-wxString wxMemoryFSHandlerBase::FindFirst(const wxString& WXUNUSED(spec),
-                                          int WXUNUSED(flags))
+wxString wxMemoryFSHandlerBase::FindFirst(const wxString& spec, int flags)

 {
 {

-    wxFAIL_MSG(wxT("wxMemoryFSHandlerBase::FindFirst not implemented"));
+    if ( (flags & wxDIR) && !(flags & wxFILE) )
+    {
+        // we only store files, not directories, so we don't risk finding
+        // anything
+        return wxString();
+    }

 
 

-    return wxEmptyString;
-}
+    if ( spec.find_first_of("?*") == wxString::npos )
+    {
+        // simple case: there are no wildcard characters so we can return
+        // either 0 or 1 results and we can find the potential match quickly
+        return m_Hash.count(spec) ? spec : wxString();
+    }
+    //else: deal with wildcards in FindNext()

 
 

-wxString wxMemoryFSHandlerBase::FindNext()
-{
-    wxFAIL_MSG(wxT("wxMemoryFSHandlerBase::FindNext not implemented"));
+    m_findArgument = spec;
+    m_findIter = m_Hash.begin();

 
 

-    return wxEmptyString;
+    return FindNext();

 }
 
 }
 

-bool wxMemoryFSHandlerBase::CheckHash(const wxString& filename)
+wxString wxMemoryFSHandlerBase::FindNext()

 {
 {

-    if ( !m_Hash )
+    // m_findArgument is used to indicate that search is in progress, we reset
+    // it to empty string after iterating over all elements
+    while ( !m_findArgument.empty() )

     {
     {

-        m_Hash = new wxHashTable(wxKEY_STRING);
+        // advance m_findIter before checking the value at the current position
+        // as we need to do it anyhow, whether it matches or not
+        const wxMemoryFSHash::const_iterator current = m_findIter;
+
+        if ( ++m_findIter == m_Hash.end() )
+            m_findArgument.clear();
+
+        if ( current->first.Matches(m_findArgument) )
+            return current->first;

     }
 
     }
 

-    if ( m_Hash->Get(filename) )
+    return wxString();
+}
+
+bool wxMemoryFSHandlerBase::CheckDoesntExist(const wxString& filename)
+{
+    if ( m_Hash.count(filename) )

     {
         wxLogError(_("Memory VFS already contains file '%s'!"), filename);
         return false;
     {
         wxLogError(_("Memory VFS already contains file '%s'!"), filename);
         return false;


@@ -175,10 +193,10 @@ void wxMemoryFSHandlerBase::AddFileWithMimeType(const wxString& filename,
                                                 const void *binarydata, size_t size,
                                                 const wxString& mimetype)
 {
                                                 const void *binarydata, size_t size,
                                                 const wxString& mimetype)
 {

-    if ( !CheckHash(filename) )
+    if ( !CheckDoesntExist(filename) )

         return;
 
         return;
 

-    m_Hash->Put(filename, new MemFSHashObj(binarydata, size, mimetype));
+    m_Hash[filename] = new wxMemoryFSFile(binarydata, size, mimetype);

 }
 
 /*static*/
 }
 
 /*static*/


@@ -200,16 +218,17 @@ void wxMemoryFSHandlerBase::AddFile(const wxString& filename,
 
 /*static*/ void wxMemoryFSHandlerBase::RemoveFile(const wxString& filename)
 {
 
 /*static*/ void wxMemoryFSHandlerBase::RemoveFile(const wxString& filename)
 {

-    if ( !m_Hash || !m_Hash->Get(filename) )
+    wxMemoryFSHash::iterator i = m_Hash.find(filename);
+    if ( i == m_Hash.end() )

     {
         wxLogError(_("Trying to remove file '%s' from memory VFS, "
                      "but it is not loaded!"),
                    filename);
     {
         wxLogError(_("Trying to remove file '%s' from memory VFS, "
                      "but it is not loaded!"),
                    filename);

+        return;

     }
     }

-    else
-    {
-        delete m_Hash->Delete(filename);
-    }
+
+    delete i->second;
+    m_Hash.erase(i);

 }
 
 #endif // wxUSE_BASE
 }
 
 #endif // wxUSE_BASE


@@ -222,21 +241,17 @@ wxMemoryFSHandler::AddFile(const wxString& filename,
                            const wxImage& image,
                            wxBitmapType type)
 {
                            const wxImage& image,
                            wxBitmapType type)
 {

-    if ( !CheckHash(filename) )
+    if ( !CheckDoesntExist(filename) )

         return;
 
     wxMemoryOutputStream mems;
         return;
 
     wxMemoryOutputStream mems;

-    if (image.Ok() && image.SaveFile(mems, type))
+    if ( image.Ok() && image.SaveFile(mems, type) )

     {
     {

-        m_Hash->Put
-                (
-                    filename,
-                    new MemFSHashObj
-                        (
-                            mems,
-                            wxImage::FindHandler(type)->GetMimeType()
-                        )
-                );
+        m_Hash[filename] = new wxMemoryFSFile
+                               (
+                                    mems,
+                                    wxImage::FindHandler(type)->GetMimeType()
+                               );

     }
     else
     {
     }
     else
     {


@@ -249,10 +264,8 @@ wxMemoryFSHandler::AddFile(const wxString& filename,
                            const wxBitmap& bitmap,
                            wxBitmapType type)
 {
                            const wxBitmap& bitmap,
                            wxBitmapType type)
 {

-#if !defined(__WXMSW__) || wxUSE_WXDIB

     wxImage img = bitmap.ConvertToImage();
     AddFile(filename, img, type);
     wxImage img = bitmap.ConvertToImage();
     AddFile(filename, img, type);

-#endif

 }
 
 #endif // wxUSE_IMAGE
 }
 
 #endif // wxUSE_IMAGE