]> git.saurik.com Git - wxWidgets.git/blobdiff - src/common/variant.cpp
projects / wxWidgets.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | inline | side by side
added missing parentheses in a couple of GetHxxxOf() macros
[wxWidgets.git] / src / common / variant.cpp
diff --git a/src/common/variant.cpp b/src/common/variant.cpp
index cab5e3ead12aa3b52b649d6b7bfa701e3674d9d6..b018603350e0fddf2164cf931c459e3d1303385a 100644 (file)
--- a/src/common/variant.cpp
+++ b/src/common/variant.cpp
@@ -1,5 +1,5 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////
-// Name:        variant.cpp
+// Name:        src/common/variant.cpp
 // Purpose:     wxVariant class, container for any type
 // Author:      Julian Smart
 // Modified by:
 // Purpose:     wxVariant class, container for any type
 // Author:      Julian Smart
 // Modified by:
@@ -9,12 +9,6 @@
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
-#ifdef __GNUG__
-#pragma implementation "variant.h"
-#pragma implementation "time.h"
-#pragma implementation "date.h"
-#endif
-
 // For compilers that support precompilation, includes "wx/wx.h".
 #include "wx/wxprec.h"
 
 // For compilers that support precompilation, includes "wx/wx.h".
 #include "wx/wxprec.h"
 
@@ -22,6 +16,19 @@
     #pragma hdrstop
 #endif
 
     #pragma hdrstop
 #endif
 
+#include "wx/variant.h"
+
+#if wxUSE_VARIANT
+
+#ifndef WX_PRECOMP
+    #include "wx/string.h"
+    #include "wx/math.h"
+    #include "wx/crt.h"
+    #if wxUSE_STREAMS
+        #include "wx/stream.h"
+    #endif
+#endif
+
 #if wxUSE_STD_IOSTREAM
     #if wxUSE_IOSTREAMH
         #include <fstream.h>
 #if wxUSE_STD_IOSTREAM
     #if wxUSE_IOSTREAMH
         #include <fstream.h>
@@ -30,290 +37,169 @@
     #endif
 #endif
 
     #endif
 #endif
 
+#if defined(__MWERKS__) && __MSL__ >= 0x6000
+namespace std {}
+using namespace std ;
+#endif
+
 #if wxUSE_STREAMS
 #if wxUSE_STREAMS
-#include "wx/stream.h"
-#include "wx/txtstrm.h"
+    #include "wx/txtstrm.h"
 #endif
 
 #include "wx/string.h"
 #endif
 
 #include "wx/string.h"
-#include "wx/variant.h"
-
-#if wxUSE_TIMEDATE
-IMPLEMENT_DYNAMIC_CLASS(wxDate, wxObject)
-IMPLEMENT_DYNAMIC_CLASS(wxTime, wxObject)
+#include "wx/tokenzr.h"
 
 
-wxTime::tFormat    wxTime::ms_Format    = wxTime::wx12h;
-wxTime::tPrecision wxTime::ms_Precision  = wxTime::wxStdMinSec;
-wxChar             wxTime::ms_bufTime[128];
-#endif
+wxVariant WXDLLIMPEXP_BASE wxNullVariant;
 
 
-IMPLEMENT_ABSTRACT_CLASS(wxVariantData, wxObject)
 
 
-wxVariant WXDLLEXPORT wxNullVariant;
+#include "wx/listimpl.cpp"
+WX_DEFINE_LIST(wxVariantList)
 
 /*
 
 /*
- * wxVariantDataList
+ * wxVariant
  */
 
  */
 
-class WXDLLEXPORT wxVariantDataList: public wxVariantData
-{
-DECLARE_DYNAMIC_CLASS(wxVariantDataList)
-public:
-    wxVariantDataList() {}
-    wxVariantDataList(const wxList& list);
-    ~wxVariantDataList();
-
-    wxList& GetValue() const { return (wxList&) m_value; }
-    void SetValue(const wxList& value) ;
-
-    virtual void Copy(wxVariantData& data);
-    virtual bool Eq(wxVariantData& data) const;
-#if wxUSE_STD_IOSTREAM
-    virtual bool Write(wxSTD ostream& str) const;
-#endif
-    virtual bool Write(wxString& str) const;
-#if wxUSE_STD_IOSTREAM
-    virtual bool Read(wxSTD istream& str);
-#endif
-    virtual bool Read(wxString& str);
-    virtual wxString GetType() const { return wxT("list"); };
-
-    void Clear();
-
-protected:
-    wxList  m_value;
-};
-
-IMPLEMENT_DYNAMIC_CLASS(wxVariantDataList, wxVariantData)
+IMPLEMENT_DYNAMIC_CLASS(wxVariant, wxObject)
 
 
-wxVariantDataList::wxVariantDataList(const wxList& list)
+wxVariant::wxVariant()
+    : wxObject()
 {
 {
-    SetValue(list);
 }
 
 }
 
-wxVariantDataList::~wxVariantDataList()
+bool wxVariant::IsNull() const
 {
 {
-    Clear();
+     return (m_refData == NULL);
 }
 
 }
 
-void wxVariantDataList::SetValue(const wxList& value)
+void wxVariant::MakeNull()
 {
 {
-    Clear();
-    wxNode* node = value.First();
-    while (node)
-    {
-        wxVariant* var = (wxVariant*) node->Data();
-        m_value.Append(new wxVariant(*var));
-        node = node->Next();
-    }
+    UnRef();
 }
 
 }
 
-void wxVariantDataList::Clear()
+void wxVariant::Clear()
 {
 {
-    wxNode* node = m_value.First();
-    while (node)
-    {
-        wxVariant* var = (wxVariant*) node->Data();
-        delete var;
-        node = node->Next();
-    }
-    m_value.Clear();
+    m_name = wxEmptyString;
 }
 
 }
 
-void wxVariantDataList::Copy(wxVariantData& data)
+wxVariant::wxVariant(const wxVariant& variant)
+    : wxObject()
 {
 {
-    wxASSERT_MSG( (data.GetType() == wxT("list")), wxT("wxVariantDataList::Copy: Can't copy to this type of data") );
-
-    wxVariantDataList& listData = (wxVariantDataList&) data;
+    if (!variant.IsNull())
+        Ref(variant);
 
 
-    listData.Clear();
-    wxNode* node = m_value.First();
-    while (node)
-    {
-        wxVariant* var = (wxVariant*) node->Data();
-        listData.m_value.Append(new wxVariant(*var));
-        node = node->Next();
-    }
+    m_name = variant.m_name;
 }
 
 }
 
-bool wxVariantDataList::Eq(wxVariantData& data) const
+wxVariant::wxVariant(wxVariantData* data, const wxString& name) // User-defined data
+    : wxObject()
 {
 {
-    wxASSERT_MSG( (data.GetType() == wxT("list")), wxT("wxVariantDataList::Eq: argument mismatch") );
-
-    wxVariantDataList& listData = (wxVariantDataList&) data;
-    wxNode* node1 = m_value.First();
-    wxNode* node2 = listData.GetValue().First();
-    while (node1 && node2)
-    {
-        wxVariant* var1 = (wxVariant*) node1->Data();
-        wxVariant* var2 = (wxVariant*) node2->Data();
-        if ((*var1) != (*var2))
-            return FALSE;
-        node1 = node1->Next();
-        node2 = node2->Next();
-    }
-    if (node1 || node2) return FALSE;
-    return TRUE;
+    m_refData = data;
+    m_name = name;
 }
 
 }
 
-#if wxUSE_STD_IOSTREAM
-bool wxVariantDataList::Write(wxSTD ostream& str) const
+wxVariant::~wxVariant()
 {
 {
-    wxString s;
-    Write(s);
-    str << (const char*) s.mb_str();
-    return TRUE;
 }
 }
-#endif
 
 
-bool wxVariantDataList::Write(wxString& str) const
+wxObjectRefData *wxVariant::CreateRefData() const
 {
 {
-    str = wxT("");
-    wxNode* node = m_value.First();
-    while (node)
-    {
-        wxVariant* var = (wxVariant*) node->Data();
-        if (node != m_value.First())
-          str += wxT(" ");
-        wxString str1;
-        str += var->MakeString();
-        node = node->Next();
-    }
-
-    return TRUE;
+    // We cannot create any particular wxVariantData.
+    wxFAIL_MSG("wxVariant::CreateRefData() cannot be implemented");
+    return NULL;
 }
 
 }
 
-#if wxUSE_STD_IOSTREAM
-bool wxVariantDataList::Read(wxSTD istream& WXUNUSED(str))
+wxObjectRefData *wxVariant::CloneRefData(const wxObjectRefData *data) const
 {
 {
-    wxFAIL_MSG(wxT("Unimplemented"));
-    // TODO
-    return FALSE;
+    return ((wxVariantData*) data)->Clone();
 }
 }
-#endif
 
 
-bool wxVariantDataList::Read(wxString& WXUNUSED(str))
+// Assignment
+void wxVariant::operator= (const wxVariant& variant)
 {
 {
-    wxFAIL_MSG(wxT("Unimplemented"));
-    // TODO
-    return FALSE;
+    Ref(variant);
+    m_name = variant.m_name;
 }
 
 }
 
-/*
- * wxVariantDataStringList
- */
-
-class WXDLLEXPORT wxVariantDataStringList: public wxVariantData
-{
-DECLARE_DYNAMIC_CLASS(wxVariantDataStringList)
-public:
-    wxVariantDataStringList() {}
-    wxVariantDataStringList(const wxStringList& list) { m_value = list; }
-
-    wxStringList& GetValue() const { return (wxStringList&) m_value; }
-    void SetValue(const wxStringList& value);
-
-    virtual void Copy(wxVariantData& data);
-    virtual bool Eq(wxVariantData& data) const;
-#if wxUSE_STD_IOSTREAM
-    virtual bool Write(wxSTD ostream& str) const;
-#endif
-    virtual bool Write(wxString& str) const;
-#if wxUSE_STD_IOSTREAM
-    virtual bool Read(wxSTD istream& str);
-#endif
-    virtual bool Read(wxString& str);
-    virtual wxString GetType() const { return wxT("stringlist"); };
-
-protected:
-    wxStringList  m_value;
-};
-
-IMPLEMENT_DYNAMIC_CLASS(wxVariantDataStringList, wxVariantData)
-
-void wxVariantDataStringList::SetValue(const wxStringList& value)
+// myVariant = new wxStringVariantData("hello")
+void wxVariant::operator= (wxVariantData* variantData)
 {
 {
-    m_value = value;
+    UnRef();
+    m_refData = variantData;
 }
 
 }
 
-void wxVariantDataStringList::Copy(wxVariantData& data)
+bool wxVariant::operator== (const wxVariant& variant) const
 {
 {
-    wxASSERT_MSG( (data.GetType() == wxT("stringlist")), wxT("wxVariantDataStringList::Copy: Can't copy to this type of data") );
+    if (IsNull() || variant.IsNull())
+        return (IsNull() == variant.IsNull());
 
 
-    wxVariantDataStringList& listData = (wxVariantDataStringList&) data;
+    if (GetType() != variant.GetType())
+        return false;
 
 
-    listData.m_value = m_value ;
+    return (GetData()->Eq(* variant.GetData()));
 }
 
 }
 
-bool wxVariantDataStringList::Eq(wxVariantData& data) const
+bool wxVariant::operator!= (const wxVariant& variant) const
 {
 {
-    wxASSERT_MSG( (data.GetType() == wxT("stringlist")), wxT("wxVariantDataStringList::Eq: argument mismatch") );
+    return (!(*this == variant));
+}
 
 
-    wxVariantDataStringList& listData = (wxVariantDataStringList&) data;
-    wxNode* node1 = m_value.First();
-    wxNode* node2 = listData.GetValue().First();
-    while (node1 && node2)
+wxString wxVariant::MakeString() const
+{
+    if (!IsNull())
     {
     {
-        wxString str1 ((wxChar*) node1->Data());
-        wxString str2 ((wxChar*) node2->Data());
-        if (str1 != str2)
-            return FALSE;
-        node1 = node1->Next();
-        node2 = node2->Next();
+        wxString str;
+        if (GetData()->Write(str))
+            return str;
     }
     }
-    if (node1 || node2) return FALSE;
-    return TRUE;
+    return wxEmptyString;
 }
 
 }
 
-#if wxUSE_STD_IOSTREAM
-bool wxVariantDataStringList::Write(wxSTD ostream& str) const
+void wxVariant::SetData(wxVariantData* data)
 {
 {
-    wxString s;
-    Write(s);
-    str << (const char*) s.mb_str();
-    return TRUE;
+    UnRef();
+    m_refData = data;
 }
 }
-#endif
 
 
-bool wxVariantDataStringList::Write(wxString& str) const
+bool wxVariant::Unshare()
 {
 {
-    str = wxT("");
-    wxNode* node = m_value.First();
-    while (node)
-    {
-        wxChar* s = (wxChar*) node->Data();
-        if (node != m_value.First())
-          str += wxT(" ");
-        str += s;
-        node = node->Next();
-    }
+    if ( !m_refData || m_refData->GetRefCount() == 1 )
+        return true;
 
 
-    return TRUE;
+    wxObject::UnShare();
+
+    return (m_refData && m_refData->GetRefCount() == 1);
 }
 
 }
 
-#if wxUSE_STD_IOSTREAM
-bool wxVariantDataStringList::Read(wxSTD istream& WXUNUSED(str))
+
+// Returns a string representing the type of the variant,
+// e.g. "string", "bool", "list", "double", "long"
+wxString wxVariant::GetType() const
 {
 {
-    wxFAIL_MSG(wxT("Unimplemented"));
-    // TODO
-    return FALSE;
+    if (IsNull())
+        return wxString(wxT("null"));
+    else
+        return GetData()->GetType();
 }
 }
-#endif
 
 
-bool wxVariantDataStringList::Read(wxString& WXUNUSED(str))
+
+bool wxVariant::IsType(const wxString& type) const
 {
 {
-    wxFAIL_MSG(wxT("Unimplemented"));
-    // TODO
-    return FALSE;
+    return (GetType() == type);
 }
 
 }
 
-/*
- * wxVariantDataLong
- */
+bool wxVariant::IsValueKindOf(const wxClassInfo* type) const
+{
+    wxClassInfo* info=GetData()->GetValueClassInfo();
+    return info ? info->IsKindOf(type) : false ;
+}
+
+
+// -----------------------------------------------------------------
+// wxVariantDataLong
+// -----------------------------------------------------------------
 
 
-class WXDLLEXPORT wxVariantDataLong: public wxVariantData
+class WXDLLIMPEXP_BASE wxVariantDataLong: public wxVariantData
 {
 {
-DECLARE_DYNAMIC_CLASS(wxVariantDataLong)
 public:
     wxVariantDataLong() { m_value = 0; }
     wxVariantDataLong(long value) { m_value = value; }
 public:
     wxVariantDataLong() { m_value = 0; }
     wxVariantDataLong(long value) { m_value = value; }
@@ -321,7 +207,6 @@ public:
     inline long GetValue() const { return m_value; }
     inline void SetValue(long value) { m_value = value; }
 
     inline long GetValue() const { return m_value; }
     inline void SetValue(long value) { m_value = value; }
 
-    virtual void Copy(wxVariantData& data);
     virtual bool Eq(wxVariantData& data) const;
 
     virtual bool Read(wxString& str);
     virtual bool Eq(wxVariantData& data) const;
 
     virtual bool Read(wxString& str);
@@ -335,23 +220,14 @@ public:
     virtual bool Write(wxOutputStream &str) const;
 #endif // wxUSE_STREAMS
 
     virtual bool Write(wxOutputStream &str) const;
 #endif // wxUSE_STREAMS
 
-    virtual wxString GetType() const { return wxT("long"); };
+    wxVariantData* Clone() const { return new wxVariantDataLong(m_value); }
+
+    virtual wxString GetType() const { return wxT("long"); }
 
 protected:
     long m_value;
 };
 
 
 protected:
     long m_value;
 };
 
-IMPLEMENT_DYNAMIC_CLASS(wxVariantDataLong, wxVariantData)
-
-void wxVariantDataLong::Copy(wxVariantData& data)
-{
-    wxASSERT_MSG( (data.GetType() == wxT("long")), wxT("wxVariantDataLong::Copy: Can't copy to this type of data") );
-
-    wxVariantDataLong& otherData = (wxVariantDataLong&) data;
-
-    otherData.m_value = m_value;
-}
-
 bool wxVariantDataLong::Eq(wxVariantData& data) const
 {
     wxASSERT_MSG( (data.GetType() == wxT("long")), wxT("wxVariantDataLong::Eq: argument mismatch") );
 bool wxVariantDataLong::Eq(wxVariantData& data) const
 {
     wxASSERT_MSG( (data.GetType() == wxT("long")), wxT("wxVariantDataLong::Eq: argument mismatch") );
@@ -367,21 +243,21 @@ bool wxVariantDataLong::Write(wxSTD ostream& str) const
     wxString s;
     Write(s);
     str << (const char*) s.mb_str();
     wxString s;
     Write(s);
     str << (const char*) s.mb_str();
-    return TRUE;
+    return true;
 }
 #endif
 
 bool wxVariantDataLong::Write(wxString& str) const
 {
     str.Printf(wxT("%ld"), m_value);
 }
 #endif
 
 bool wxVariantDataLong::Write(wxString& str) const
 {
     str.Printf(wxT("%ld"), m_value);
-    return TRUE;
+    return true;
 }
 
 #if wxUSE_STD_IOSTREAM
 bool wxVariantDataLong::Read(wxSTD istream& str)
 {
     str >> m_value;
 }
 
 #if wxUSE_STD_IOSTREAM
 bool wxVariantDataLong::Read(wxSTD istream& str)
 {
     str >> m_value;
-    return TRUE;
+    return true;
 }
 #endif
 
 }
 #endif
 
@@ -391,38 +267,96 @@ bool wxVariantDataLong::Write(wxOutputStream& str) const
     wxTextOutputStream s(str);
 
     s.Write32((size_t)m_value);
     wxTextOutputStream s(str);
 
     s.Write32((size_t)m_value);
-    return TRUE;
+    return true;
 }
 
 bool wxVariantDataLong::Read(wxInputStream& str)
 {
    wxTextInputStream s(str);
    m_value = s.Read32();
 }
 
 bool wxVariantDataLong::Read(wxInputStream& str)
 {
    wxTextInputStream s(str);
    m_value = s.Read32();
-   return TRUE;
+   return true;
 }
 #endif // wxUSE_STREAMS
 
 bool wxVariantDataLong::Read(wxString& str)
 {
 }
 #endif // wxUSE_STREAMS
 
 bool wxVariantDataLong::Read(wxString& str)
 {
-    m_value = wxAtol((const wxChar*) str);
-    return TRUE;
+    m_value = wxAtol(str);
+    return true;
 }
 
 }
 
-/*
- * wxVariantDataReal
- */
+// wxVariant
+
+wxVariant::wxVariant(long val, const wxString& name)
+{
+    m_refData = new wxVariantDataLong(val);
+    m_name = name;
+}
+
+wxVariant::wxVariant(int val, const wxString& name)
+{
+    m_refData = new wxVariantDataLong((long)val);
+    m_name = name;
+}
+
+wxVariant::wxVariant(short val, const wxString& name)
+{
+    m_refData = new wxVariantDataLong((long)val);
+    m_name = name;
+}
+
+bool wxVariant::operator== (long value) const
+{
+    long thisValue;
+    if (!Convert(&thisValue))
+        return false;
+    else
+        return (value == thisValue);
+}
+
+bool wxVariant::operator!= (long value) const
+{
+    return (!((*this) == value));
+}
+
+void wxVariant::operator= (long value)
+{
+    if (GetType() == wxT("long") &&
+        m_refData->GetRefCount() == 1)
+    {
+        ((wxVariantDataLong*)GetData())->SetValue(value);
+    }
+    else
+    {
+        UnRef();
+        m_refData = new wxVariantDataLong(value);
+    }
+}
+
+long wxVariant::GetLong() const
+{
+    long value;
+    if (Convert(& value))
+        return value;
+    else
+    {
+        wxFAIL_MSG(wxT("Could not convert to a long"));
+        return 0;
+    }
+}
 
 
-class WXDLLEXPORT wxVariantDataReal: public wxVariantData
+// -----------------------------------------------------------------
+// wxVariantDoubleData
+// -----------------------------------------------------------------
+
+class WXDLLIMPEXP_BASE wxVariantDoubleData: public wxVariantData
 {
 {
-DECLARE_DYNAMIC_CLASS(wxVariantDataReal)
 public:
 public:
-    wxVariantDataReal() { m_value = 0.0; }
-    wxVariantDataReal(double value) { m_value = value; }
+    wxVariantDoubleData() { m_value = 0.0; }
+    wxVariantDoubleData(double value) { m_value = value; }
 
     inline double GetValue() const { return m_value; }
     inline void SetValue(double value) { m_value = value; }
 
 
     inline double GetValue() const { return m_value; }
     inline void SetValue(double value) { m_value = value; }
 
-    virtual void Copy(wxVariantData& data);
     virtual bool Eq(wxVariantData& data) const;
     virtual bool Read(wxString& str);
 #if wxUSE_STD_IOSTREAM
     virtual bool Eq(wxVariantData& data) const;
     virtual bool Read(wxString& str);
 #if wxUSE_STD_IOSTREAM
@@ -436,123 +370,148 @@ public:
     virtual bool Read(wxInputStream& str);
     virtual bool Write(wxOutputStream &str) const;
 #endif // wxUSE_STREAMS
     virtual bool Read(wxInputStream& str);
     virtual bool Write(wxOutputStream &str) const;
 #endif // wxUSE_STREAMS
-    virtual wxString GetType() const { return wxT("double"); };
+    virtual wxString GetType() const { return wxT("double"); }
 
 
+    wxVariantData* Clone() const { return new wxVariantDoubleData(m_value); }
 protected:
     double m_value;
 };
 
 protected:
     double m_value;
 };
 
-IMPLEMENT_DYNAMIC_CLASS(wxVariantDataReal, wxVariantData)
-
-void wxVariantDataReal::Copy(wxVariantData& data)
-{
-    wxASSERT_MSG( (data.GetType() == wxT("double")), wxT("wxVariantDataReal::Copy: Can't copy to this type of data") );
-
-    wxVariantDataReal& otherData = (wxVariantDataReal&) data;
-
-    otherData.m_value = m_value;
-}
-
-bool wxVariantDataReal::Eq(wxVariantData& data) const
+bool wxVariantDoubleData::Eq(wxVariantData& data) const
 {
 {
-    wxASSERT_MSG( (data.GetType() == wxT("double")), wxT("wxVariantDataReal::Eq: argument mismatch") );
+    wxASSERT_MSG( (data.GetType() == wxT("double")), wxT("wxVariantDoubleData::Eq: argument mismatch") );
 
 
-    wxVariantDataReal& otherData = (wxVariantDataReal&) data;
+    wxVariantDoubleData& otherData = (wxVariantDoubleData&) data;
 
 
-    return (otherData.m_value == m_value);
+    return wxIsSameDouble(otherData.m_value, m_value);
 }
 
 #if wxUSE_STD_IOSTREAM
 }
 
 #if wxUSE_STD_IOSTREAM
-bool wxVariantDataReal::Write(wxSTD ostream& str) const
+bool wxVariantDoubleData::Write(wxSTD ostream& str) const
 {
     wxString s;
     Write(s);
     str << (const char*) s.mb_str();
 {
     wxString s;
     Write(s);
     str << (const char*) s.mb_str();
-    return TRUE;
+    return true;
 }
 #endif
 
 }
 #endif
 
-bool wxVariantDataReal::Write(wxString& str) const
+bool wxVariantDoubleData::Write(wxString& str) const
 {
 {
-    str.Printf(wxT("%.4f"), m_value);
-    return TRUE;
+    str.Printf(wxT("%.14g"), m_value);
+    return true;
 }
 
 #if wxUSE_STD_IOSTREAM
 }
 
 #if wxUSE_STD_IOSTREAM
-bool wxVariantDataReal::Read(wxSTD istream& str)
+bool wxVariantDoubleData::Read(wxSTD istream& str)
 {
     str >> m_value;
 {
     str >> m_value;
-    return TRUE;
+    return true;
 }
 #endif
 
 #if wxUSE_STREAMS
 }
 #endif
 
 #if wxUSE_STREAMS
-bool wxVariantDataReal::Write(wxOutputStream& str) const
+bool wxVariantDoubleData::Write(wxOutputStream& str) const
 {
     wxTextOutputStream s(str);
     s.WriteDouble((double)m_value);
 {
     wxTextOutputStream s(str);
     s.WriteDouble((double)m_value);
-    return TRUE;
+    return true;
 }
 
 }
 
-bool wxVariantDataReal::Read(wxInputStream& str)
+bool wxVariantDoubleData::Read(wxInputStream& str)
 {
     wxTextInputStream s(str);
     m_value = (float)s.ReadDouble();
 {
     wxTextInputStream s(str);
     m_value = (float)s.ReadDouble();
-    return TRUE;
+    return true;
 }
 #endif // wxUSE_STREAMS
 
 }
 #endif // wxUSE_STREAMS
 
-bool wxVariantDataReal::Read(wxString& str)
+bool wxVariantDoubleData::Read(wxString& str)
 {
 {
-    m_value = wxAtof((const wxChar*) str);
-    return TRUE;
+    m_value = wxAtof(str);
+    return true;
 }
 
 }
 
-#ifdef HAVE_BOOL
-/*
- * wxVariantDataBool
- */
+//  wxVariant double code
 
 
-class WXDLLEXPORT wxVariantDataBool: public wxVariantData
+wxVariant::wxVariant(double val, const wxString& name)
 {
 {
-DECLARE_DYNAMIC_CLASS(wxVariantDataBool)
-public:
-    wxVariantDataBool() { m_value = 0; }
-    wxVariantDataBool(bool value) { m_value = value; }
+    m_refData = new wxVariantDoubleData(val);
+    m_name = name;
+}
 
 
-    inline bool GetValue() const { return m_value; }
-    inline void SetValue(bool value) { m_value = value; }
+bool wxVariant::operator== (double value) const
+{
+    double thisValue;
+    if (!Convert(&thisValue))
+        return false;
 
 
-    virtual void Copy(wxVariantData& data);
-    virtual bool Eq(wxVariantData& data) const;
-#if wxUSE_STD_IOSTREAM
-    virtual bool Write(wxSTD ostream& str) const;
-#endif
-    virtual bool Write(wxString& str) const;
-    virtual bool Read(wxString& str);
-#if wxUSE_STD_IOSTREAM
-    virtual bool Read(wxSTD istream& str);
-#endif
-#if wxUSE_STREAMS
-    virtual bool Read(wxInputStream& str);
-    virtual bool Write(wxOutputStream& str) const;
-#endif // wxUSE_STREAMS
-    virtual wxString GetType() const { return wxT("bool"); };
+    return wxIsSameDouble(value, thisValue);
+}
 
 
-protected:
-    bool m_value;
-};
+bool wxVariant::operator!= (double value) const
+{
+    return (!((*this) == value));
+}
 
 
-IMPLEMENT_DYNAMIC_CLASS(wxVariantDataBool, wxVariantData)
+void wxVariant::operator= (double value)
+{
+    if (GetType() == wxT("double") &&
+        m_refData->GetRefCount() == 1)
+    {
+        ((wxVariantDoubleData*)GetData())->SetValue(value);
+    }
+    else
+    {
+        UnRef();
+        m_refData = new wxVariantDoubleData(value);
+    }
+}
 
 
-void wxVariantDataBool::Copy(wxVariantData& data)
+double wxVariant::GetDouble() const
 {
 {
-    wxASSERT_MSG( (data.GetType() == wxT("bool")), wxT("wxVariantDataBool::Copy: Can't copy to this type of data") );
+    double value;
+    if (Convert(& value))
+        return value;
+    else
+    {
+        wxFAIL_MSG(wxT("Could not convert to a double number"));
+        return 0.0;
+    }
+}
 
 
-    wxVariantDataBool& otherData = (wxVariantDataBool&) data;
+// -----------------------------------------------------------------
+// wxVariantBoolData
+// -----------------------------------------------------------------
 
 
-    otherData.m_value = m_value;
-}
+class WXDLLIMPEXP_BASE wxVariantDataBool: public wxVariantData
+{
+public:
+    wxVariantDataBool() { m_value = 0; }
+    wxVariantDataBool(bool value) { m_value = value; }
+
+    inline bool GetValue() const { return m_value; }
+    inline void SetValue(bool value) { m_value = value; }
+
+    virtual bool Eq(wxVariantData& data) const;
+#if wxUSE_STD_IOSTREAM
+    virtual bool Write(wxSTD ostream& str) const;
+#endif
+    virtual bool Write(wxString& str) const;
+    virtual bool Read(wxString& str);
+#if wxUSE_STD_IOSTREAM
+    virtual bool Read(wxSTD istream& str);
+#endif
+#if wxUSE_STREAMS
+    virtual bool Read(wxInputStream& str);
+    virtual bool Write(wxOutputStream& str) const;
+#endif // wxUSE_STREAMS
+    virtual wxString GetType() const { return wxT("bool"); }
+
+    wxVariantData* Clone() const { return new wxVariantDataBool(m_value); }
+protected:
+    bool m_value;
+};
 
 bool wxVariantDataBool::Eq(wxVariantData& data) const
 {
 
 bool wxVariantDataBool::Eq(wxVariantData& data) const
 {
@@ -569,14 +528,14 @@ bool wxVariantDataBool::Write(wxSTD ostream& str) const
     wxString s;
     Write(s);
     str << (const char*) s.mb_str();
     wxString s;
     Write(s);
     str << (const char*) s.mb_str();
-    return TRUE;
+    return true;
 }
 #endif
 
 bool wxVariantDataBool::Write(wxString& str) const
 {
     str.Printf(wxT("%d"), (int) m_value);
 }
 #endif
 
 bool wxVariantDataBool::Write(wxString& str) const
 {
     str.Printf(wxT("%d"), (int) m_value);
-    return TRUE;
+    return true;
 }
 
 #if wxUSE_STD_IOSTREAM
 }
 
 #if wxUSE_STD_IOSTREAM
@@ -584,7 +543,7 @@ bool wxVariantDataBool::Read(wxSTD istream& WXUNUSED(str))
 {
     wxFAIL_MSG(wxT("Unimplemented"));
 //    str >> (long) m_value;
 {
     wxFAIL_MSG(wxT("Unimplemented"));
 //    str >> (long) m_value;
-    return FALSE;
+    return false;
 }
 #endif
 
 }
 #endif
 
@@ -593,8 +552,8 @@ bool wxVariantDataBool::Write(wxOutputStream& str) const
 {
     wxTextOutputStream s(str);
 
 {
     wxTextOutputStream s(str);
 
-    s.Write8(m_value); 
-    return TRUE;
+    s.Write8(m_value);
+    return true;
 }
 
 bool wxVariantDataBool::Read(wxInputStream& str)
 }
 
 bool wxVariantDataBool::Read(wxInputStream& str)
@@ -602,32 +561,77 @@ bool wxVariantDataBool::Read(wxInputStream& str)
     wxTextInputStream s(str);
 
     m_value = s.Read8() != 0;
     wxTextInputStream s(str);
 
     m_value = s.Read8() != 0;
-    return TRUE;
+    return true;
 }
 #endif // wxUSE_STREAMS
 
 bool wxVariantDataBool::Read(wxString& str)
 {
 }
 #endif // wxUSE_STREAMS
 
 bool wxVariantDataBool::Read(wxString& str)
 {
-    m_value = (wxAtol((const wxChar*) str) != 0);
-    return TRUE;
+    m_value = (wxAtol(str) != 0);
+    return true;
 }
 }
-#endif // HAVE_BOOL
 
 
-/*
- * wxVariantDataChar
- */
+// wxVariant ****
+
+wxVariant::wxVariant(bool val, const wxString& name)
+{
+    m_refData = new wxVariantDataBool(val);
+    m_name = name;
+}
+
+bool wxVariant::operator== (bool value) const
+{
+    bool thisValue;
+    if (!Convert(&thisValue))
+        return false;
+    else
+        return (value == thisValue);
+}
+
+bool wxVariant::operator!= (bool value) const
+{
+    return (!((*this) == value));
+}
+
+void wxVariant::operator= (bool value)
+{
+    if (GetType() == wxT("bool") &&
+        m_refData->GetRefCount() == 1)
+    {
+        ((wxVariantDataBool*)GetData())->SetValue(value);
+    }
+    else
+    {
+        UnRef();
+        m_refData = new wxVariantDataBool(value);
+    }
+}
+
+bool wxVariant::GetBool() const
+{
+    bool value;
+    if (Convert(& value))
+        return value;
+    else
+    {
+        wxFAIL_MSG(wxT("Could not convert to a bool"));
+        return 0;
+    }
+}
+
+// -----------------------------------------------------------------
+// wxVariantDataChar
+// -----------------------------------------------------------------
 
 
-class WXDLLEXPORT wxVariantDataChar: public wxVariantData
+class WXDLLIMPEXP_BASE wxVariantDataChar: public wxVariantData
 {
 {
-DECLARE_DYNAMIC_CLASS(wxVariantDataChar)
 public:
     wxVariantDataChar() { m_value = 0; }
 public:
     wxVariantDataChar() { m_value = 0; }
-    wxVariantDataChar(char value) { m_value = value; }
+    wxVariantDataChar(const wxUniChar& value) { m_value = value; }
 
 
-    inline char GetValue() const { return m_value; }
-    inline void SetValue(char value) { m_value = value; }
+    inline wxUniChar GetValue() const { return m_value; }
+    inline void SetValue(const wxUniChar& value) { m_value = value; }
 
 
-    virtual void Copy(wxVariantData& data);
     virtual bool Eq(wxVariantData& data) const;
 #if wxUSE_STD_IOSTREAM
     virtual bool Read(wxSTD istream& str);
     virtual bool Eq(wxVariantData& data) const;
 #if wxUSE_STD_IOSTREAM
     virtual bool Read(wxSTD istream& str);
@@ -639,23 +643,13 @@ public:
     virtual bool Read(wxInputStream& str);
     virtual bool Write(wxOutputStream& str) const;
 #endif // wxUSE_STREAMS
     virtual bool Read(wxInputStream& str);
     virtual bool Write(wxOutputStream& str) const;
 #endif // wxUSE_STREAMS
-    virtual wxString GetType() const { return wxT("char"); };
+    virtual wxString GetType() const { return wxT("char"); }
+    wxVariantData* Clone() const { return new wxVariantDataChar(m_value); }
 
 protected:
 
 protected:
-    char m_value;
+    wxUniChar m_value;
 };
 
 };
 
-IMPLEMENT_DYNAMIC_CLASS(wxVariantDataChar, wxVariantData)
-
-void wxVariantDataChar::Copy(wxVariantData& data)
-{
-    wxASSERT_MSG( (data.GetType() == wxT("char")), wxT("wxVariantDataChar::Copy: Can't copy to this type of data") );
-
-    wxVariantDataChar& otherData = (wxVariantDataChar&) data;
-
-    otherData.m_value = m_value;
-}
-
 bool wxVariantDataChar::Eq(wxVariantData& data) const
 {
     wxASSERT_MSG( (data.GetType() == wxT("char")), wxT("wxVariantDataChar::Eq: argument mismatch") );
 bool wxVariantDataChar::Eq(wxVariantData& data) const
 {
     wxASSERT_MSG( (data.GetType() == wxT("char")), wxT("wxVariantDataChar::Eq: argument mismatch") );
@@ -668,25 +662,23 @@ bool wxVariantDataChar::Eq(wxVariantData& data) const
 #if wxUSE_STD_IOSTREAM
 bool wxVariantDataChar::Write(wxSTD ostream& str) const
 {
 #if wxUSE_STD_IOSTREAM
 bool wxVariantDataChar::Write(wxSTD ostream& str) const
 {
-    wxString s;
-    Write(s);
-    str << (const char*) s.mb_str();
-    return TRUE;
+    str << wxString(m_value);
+    return true;
 }
 #endif
 
 bool wxVariantDataChar::Write(wxString& str) const
 {
 }
 #endif
 
 bool wxVariantDataChar::Write(wxString& str) const
 {
-    str.Printf(wxT("%c"), m_value);
-    return TRUE;
+    str = m_value;
+    return true;
 }
 
 #if wxUSE_STD_IOSTREAM
 bool wxVariantDataChar::Read(wxSTD istream& WXUNUSED(str))
 {
     wxFAIL_MSG(wxT("Unimplemented"));
 }
 
 #if wxUSE_STD_IOSTREAM
 bool wxVariantDataChar::Read(wxSTD istream& WXUNUSED(str))
 {
     wxFAIL_MSG(wxT("Unimplemented"));
-//    str >> m_value;
-    return FALSE;
+
+    return false;
 }
 #endif
 
 }
 #endif
 
@@ -695,41 +687,94 @@ bool wxVariantDataChar::Write(wxOutputStream& str) const
 {
     wxTextOutputStream s(str);
 
 {
     wxTextOutputStream s(str);
 
-    s.Write8(m_value);
-    return TRUE;
+    // FIXME-UTF8: this should be just "s << m_value;" after removal of
+    //             ANSI build and addition of wxUniChar to wxTextOutputStream:
+    s << (wxChar)m_value;
+
+    return true;
 }
 
 bool wxVariantDataChar::Read(wxInputStream& str)
 {
     wxTextInputStream s(str);
 
 }
 
 bool wxVariantDataChar::Read(wxInputStream& str)
 {
     wxTextInputStream s(str);
 
-    m_value = s.Read8();
-    return TRUE;
+    // FIXME-UTF8: this should be just "s >> m_value;" after removal of
+    //             ANSI build and addition of wxUniChar to wxTextInputStream:
+    wxChar ch;
+    s >> ch;
+    m_value = ch;
+
+    return true;
 }
 #endif // wxUSE_STREAMS
 
 bool wxVariantDataChar::Read(wxString& str)
 {
 }
 #endif // wxUSE_STREAMS
 
 bool wxVariantDataChar::Read(wxString& str)
 {
-    m_value = str[(size_t)0];
-    return TRUE;
+    m_value = str[0u];
+    return true;
 }
 
 }
 
-/*
- * wxVariantDataString
- */
+wxVariant::wxVariant(const wxUniChar& val, const wxString& name)
+{
+    m_refData = new wxVariantDataChar(val);
+    m_name = name;
+}
 
 
-#if defined(__BORLANDC__) && defined(__WIN16__)
-// Change name because of truncation
-#define wxVariantDataString wxVariantStringData
-#endif
+wxVariant::wxVariant(char val, const wxString& name)
+{
+    m_refData = new wxVariantDataChar(val);
+    m_name = name;
+}
 
 
-class WXDLLEXPORT wxVariantDataString: public wxVariantData
+wxVariant::wxVariant(wchar_t val, const wxString& name)
+{
+    m_refData = new wxVariantDataChar(val);
+    m_name = name;
+}
+
+bool wxVariant::operator==(const wxUniChar& value) const
+{
+    wxUniChar thisValue;
+    if (!Convert(&thisValue))
+        return false;
+    else
+        return (value == thisValue);
+}
+
+wxVariant& wxVariant::operator=(const wxUniChar& value)
+{
+    if (GetType() == wxT("char") &&
+        m_refData->GetRefCount() == 1)
+    {
+        ((wxVariantDataChar*)GetData())->SetValue(value);
+    }
+    else
+    {
+        UnRef();
+        m_refData = new wxVariantDataChar(value);
+    }
+
+    return *this;
+}
+
+wxUniChar wxVariant::GetChar() const
+{
+    wxUniChar value;
+    if (Convert(& value))
+        return value;
+    else
+    {
+        wxFAIL_MSG(wxT("Could not convert to a char"));
+        return wxUniChar(0);
+    }
+}
+
+// ----------------------------------------------------------------------------
+// wxVariantDataString
+// ----------------------------------------------------------------------------
+
+class WXDLLIMPEXP_BASE wxVariantDataString: public wxVariantData
 {
 {
-#if defined(__BORLANDC__) && defined(__WIN16__)
-DECLARE_DYNAMIC_CLASS(wxVariantStringData)
-#else
-DECLARE_DYNAMIC_CLASS(wxVariantDataString)
-#endif
 public:
     wxVariantDataString() { }
     wxVariantDataString(const wxString& value) { m_value = value; }
 public:
     wxVariantDataString() { }
     wxVariantDataString(const wxString& value) { m_value = value; }
@@ -737,7 +782,6 @@ public:
     inline wxString GetValue() const { return m_value; }
     inline void SetValue(const wxString& value) { m_value = value; }
 
     inline wxString GetValue() const { return m_value; }
     inline void SetValue(const wxString& value) { m_value = value; }
 
-    virtual void Copy(wxVariantData& data);
     virtual bool Eq(wxVariantData& data) const;
 #if wxUSE_STD_IOSTREAM
     virtual bool Write(wxSTD ostream& str) const;
     virtual bool Eq(wxVariantData& data) const;
 #if wxUSE_STD_IOSTREAM
     virtual bool Write(wxSTD ostream& str) const;
@@ -745,27 +789,19 @@ public:
     virtual bool Read(wxString& str);
     virtual bool Write(wxString& str) const;
 #if wxUSE_STD_IOSTREAM
     virtual bool Read(wxString& str);
     virtual bool Write(wxString& str) const;
 #if wxUSE_STD_IOSTREAM
-    virtual bool Read(wxSTD istream& str);
+    virtual bool Read(wxSTD istream& WXUNUSED(str)) { return false; }
 #endif
 #if wxUSE_STREAMS
     virtual bool Read(wxInputStream& str);
     virtual bool Write(wxOutputStream& str) const;
 #endif // wxUSE_STREAMS
 #endif
 #if wxUSE_STREAMS
     virtual bool Read(wxInputStream& str);
     virtual bool Write(wxOutputStream& str) const;
 #endif // wxUSE_STREAMS
-    virtual wxString GetType() const { return wxT("string"); };
+    virtual wxString GetType() const { return wxT("string"); }
+    wxVariantData* Clone() const { return new wxVariantDataString(m_value); }
 
 protected:
     wxString m_value;
 };
 
 
 protected:
     wxString m_value;
 };
 
-void wxVariantDataString::Copy(wxVariantData& data)
-{
-    wxASSERT_MSG( (data.GetType() == wxT("string")), wxT("wxVariantDataString::Copy: Can't copy to this type of data") );
-
-    wxVariantDataString& otherData = (wxVariantDataString&) data;
-
-    otherData.m_value = m_value;
-}
-
 bool wxVariantDataString::Eq(wxVariantData& data) const
 {
     wxASSERT_MSG( (data.GetType() == wxT("string")), wxT("wxVariantDataString::Eq: argument mismatch") );
 bool wxVariantDataString::Eq(wxVariantData& data) const
 {
     wxASSERT_MSG( (data.GetType() == wxT("string")), wxT("wxVariantDataString::Eq: argument mismatch") );
@@ -779,23 +815,15 @@ bool wxVariantDataString::Eq(wxVariantData& data) const
 bool wxVariantDataString::Write(wxSTD ostream& str) const
 {
     str << (const char*) m_value.mb_str();
 bool wxVariantDataString::Write(wxSTD ostream& str) const
 {
     str << (const char*) m_value.mb_str();
-    return TRUE;
+    return true;
 }
 #endif
 
 bool wxVariantDataString::Write(wxString& str) const
 {
     str = m_value;
 }
 #endif
 
 bool wxVariantDataString::Write(wxString& str) const
 {
     str = m_value;
-    return TRUE;
-}
-
-#if wxUSE_STD_IOSTREAM
-bool wxVariantDataString::Read(wxSTD istream& str)
-{
-    str >> m_value;
-    return TRUE;
+    return true;
 }
 }
-#endif
 
 #if wxUSE_STREAMS
 bool wxVariantDataString::Write(wxOutputStream& str) const
 
 #if wxUSE_STREAMS
 bool wxVariantDataString::Write(wxOutputStream& str) const
@@ -803,130 +831,115 @@ bool wxVariantDataString::Write(wxOutputStream& str) const
   // why doesn't wxOutputStream::operator<< take "const wxString&"
     wxTextOutputStream s(str);
     s.WriteString(m_value);
   // why doesn't wxOutputStream::operator<< take "const wxString&"
     wxTextOutputStream s(str);
     s.WriteString(m_value);
-    return TRUE;
+    return true;
 }
 
 bool wxVariantDataString::Read(wxInputStream& str)
 {
     wxTextInputStream s(str);
 
 }
 
 bool wxVariantDataString::Read(wxInputStream& str)
 {
     wxTextInputStream s(str);
 
-    m_value = s.ReadString();
-    return TRUE;
+    m_value = s.ReadLine();
+    return true;
 }
 #endif // wxUSE_STREAMS
 
 bool wxVariantDataString::Read(wxString& str)
 {
     m_value = str;
 }
 #endif // wxUSE_STREAMS
 
 bool wxVariantDataString::Read(wxString& str)
 {
     m_value = str;
-    return TRUE;
+    return true;
 }
 
 }
 
-#if defined(__BORLANDC__) && defined(__WIN16__)
-IMPLEMENT_DYNAMIC_CLASS(wxVariantStringData, wxVariantData)
-#else
-IMPLEMENT_DYNAMIC_CLASS(wxVariantDataString, wxVariantData)
-#endif
-
-/*
- * wxVariantDataTime
- */
-
-// For some reason, Watcom C++ can't link variant.cpp with time/date classes compiled
-#if wxUSE_TIMEDATE && !defined(__WATCOMC__)
+// wxVariant ****
 
 
-class wxVariantDataTime: public wxVariantData
+wxVariant::wxVariant(const wxString& val, const wxString& name)
 {
 {
- DECLARE_DYNAMIC_CLASS(wxVariantDataTime)
-public:
-    wxVariantDataTime() { }
-    wxVariantDataTime(const wxTime& value) { m_value = value; }
-
-    inline wxTime GetValue() const { return m_value; }
-    inline void SetValue(const wxTime& value) { m_value = value; }
-
-    virtual void Copy(wxVariantData& data);
-    virtual bool Eq(wxVariantData& data) const;
-#if wxUSE_STD_IOSTREAM
-    virtual bool Write(wxSTD ostream& str) const;
-#endif
-    virtual bool Write(wxString& str) const;
-#if wxUSE_STD_IOSTREAM
-    virtual bool Read(wxSTD istream& str);
-#endif
-    virtual bool Read(wxString& str);
-    virtual wxString GetType() const { return wxT("time"); };
-       virtual wxVariantData* Clone() { return new wxVariantDataTime; }
-
-protected:
-    wxTime m_value;
-};
-
-IMPLEMENT_DYNAMIC_CLASS(wxVariantDataTime, wxVariantData)
+    m_refData = new wxVariantDataString(val);
+    m_name = name;
+}
 
 
-void wxVariantDataTime::Copy(wxVariantData& data)
+wxVariant::wxVariant(const char* val, const wxString& name)
 {
 {
-    wxASSERT_MSG( (data.GetType() == wxT("time")), wxT("wxVariantDataTime::Copy: Can't copy to this type of data") );
-
-    wxVariantDataTime& otherData = (wxVariantDataTime&) data;
+    m_refData = new wxVariantDataString(wxString(val));
+    m_name = name;
+}
 
 
-    otherData.m_value = m_value;
+wxVariant::wxVariant(const wchar_t* val, const wxString& name)
+{
+    m_refData = new wxVariantDataString(wxString(val));
+    m_name = name;
 }
 
 }
 
-bool wxVariantDataTime::Eq(wxVariantData& data) const
+wxVariant::wxVariant(const wxCStrData& val, const wxString& name)
 {
 {
-    wxASSERT_MSG( (data.GetType() == wxT("time")), wxT("wxVariantDataTime::Eq: argument mismatch") );
+    m_refData = new wxVariantDataString(val.AsString());
+    m_name = name;
+}
 
 
-    wxVariantDataTime& otherData = (wxVariantDataTime&) data;
+wxVariant::wxVariant(const wxScopedCharBuffer& val, const wxString& name)
+{
+    m_refData = new wxVariantDataString(wxString(val));
+    m_name = name;
+}
 
 
-    return (otherData.m_value == m_value);
+wxVariant::wxVariant(const wxScopedWCharBuffer& val, const wxString& name)
+{
+    m_refData = new wxVariantDataString(wxString(val));
+    m_name = name;
 }
 
 }
 
-#if wxUSE_STD_IOSTREAM
-bool wxVariantDataTime::Write(wxSTD ostream& str) const
+bool wxVariant::operator== (const wxString& value) const
 {
 {
-    wxString s;
-    Write(s);
-    str << (const char*) s.mb_str();
-    return TRUE;
+    wxString thisValue;
+    if (!Convert(&thisValue))
+        return false;
+
+    return value == thisValue;
 }
 }
-#endif
 
 
-bool wxVariantDataTime::Write(wxString& str) const
+bool wxVariant::operator!= (const wxString& value) const
 {
 {
-    wxChar*s = m_value.FormatTime();
-    str = s;
-    return TRUE;
+    return (!((*this) == value));
 }
 
 }
 
-#if wxUSE_STD_IOSTREAM
-bool wxVariantDataTime::Read(wxSTD istream& WXUNUSED(str))
+wxVariant& wxVariant::operator= (const wxString& value)
 {
 {
-    // Not implemented
-    return FALSE;
+    if (GetType() == wxT("string") &&
+        m_refData->GetRefCount() == 1)
+    {
+        ((wxVariantDataString*)GetData())->SetValue(value);
+    }
+    else
+    {
+        UnRef();
+        m_refData = new wxVariantDataString(value);
+    }
+    return *this;
 }
 }
-#endif
 
 
-bool wxVariantDataTime::Read(wxString& WXUNUSED(str))
+wxString wxVariant::GetString() const
 {
 {
-    // Not implemented
-    return FALSE;
+    wxString value;
+    if (!Convert(& value))
+    {
+        wxFAIL_MSG(wxT("Could not convert to a string"));
+    }
+
+    return value;
 }
 
 }
 
-/*
- * wxVariantDataDate
- */
+// ----------------------------------------------------------------------------
+// wxVariantDataWxObjectPtr
+// ----------------------------------------------------------------------------
 
 
-class wxVariantDataDate: public wxVariantData
+class wxVariantDataWxObjectPtr: public wxVariantData
 {
 {
- DECLARE_DYNAMIC_CLASS(wxVariantDataDate)
 public:
 public:
-    wxVariantDataDate() { }
-    wxVariantDataDate(const wxDate& value) { m_value = value; }
+    wxVariantDataWxObjectPtr() { }
+    wxVariantDataWxObjectPtr(wxObject* value) { m_value = value; }
 
 
-    inline wxDate GetValue() const { return m_value; }
-    inline void SetValue(const wxDate& value) { m_value = value; }
+    inline wxObject* GetValue() const { return m_value; }
+    inline void SetValue(wxObject* value) { m_value = value; }
 
 
-    virtual void Copy(wxVariantData& data);
     virtual bool Eq(wxVariantData& data) const;
 #if wxUSE_STD_IOSTREAM
     virtual bool Write(wxSTD ostream& str) const;
     virtual bool Eq(wxVariantData& data) const;
 #if wxUSE_STD_IOSTREAM
     virtual bool Write(wxSTD ostream& str) const;
@@ -936,724 +949,312 @@ public:
     virtual bool Read(wxSTD istream& str);
 #endif
     virtual bool Read(wxString& str);
     virtual bool Read(wxSTD istream& str);
 #endif
     virtual bool Read(wxString& str);
-    virtual wxString GetType() const { return wxT("date"); };
-       virtual wxVariantData* Clone() { return new wxVariantDataDate; }
+    virtual wxString GetType() const ;
+    virtual wxVariantData* Clone() const { return new wxVariantDataWxObjectPtr(m_value); }
+
+    virtual wxClassInfo* GetValueClassInfo();
 
 protected:
 
 protected:
-    wxDate m_value;
+    wxObject* m_value;
 };
 
 };
 
-IMPLEMENT_DYNAMIC_CLASS(wxVariantDataDate, wxVariantData)
+bool wxVariantDataWxObjectPtr::Eq(wxVariantData& data) const
+{
+    wxASSERT_MSG( data.GetType() == GetType(), wxT("wxVariantDataWxObjectPtr::Eq: argument mismatch") );
+
+    wxVariantDataWxObjectPtr& otherData = (wxVariantDataWxObjectPtr&) data;
+
+    return (otherData.m_value == m_value);
+}
 
 
-void wxVariantDataDate::Copy(wxVariantData& data)
+wxString wxVariantDataWxObjectPtr::GetType() const
 {
 {
-    wxASSERT_MSG( (data.GetType() == wxT("date")), wxT("wxVariantDataDate::Copy: Can't copy to this type of data") );
+    wxString returnVal(wxT("wxObject*"));
 
 
-    wxVariantDataDate& otherData = (wxVariantDataDate&) data;
+    if (m_value)
+    {
+        returnVal = m_value->GetClassInfo()->GetClassName();
+        returnVal += wxT("*");
+    }
 
 
-    otherData.m_value = m_value;
+    return returnVal;
 }
 
 }
 
-bool wxVariantDataDate::Eq(wxVariantData& data) const
+wxClassInfo* wxVariantDataWxObjectPtr::GetValueClassInfo()
 {
 {
-    wxASSERT_MSG( (data.GetType() == wxT("date")), wxT("wxVariantDataDate::Eq: argument mismatch") );
+    wxClassInfo* returnVal=NULL;
 
 
-    wxVariantDataDate& otherData = (wxVariantDataDate&) data;
+    if (m_value) returnVal = m_value->GetClassInfo();
 
 
-    return (otherData.m_value == m_value);
+    return returnVal;
 }
 
 #if wxUSE_STD_IOSTREAM
 }
 
 #if wxUSE_STD_IOSTREAM
-bool wxVariantDataDate::Write(wxSTD ostream& str) const
+bool wxVariantDataWxObjectPtr::Write(wxSTD ostream& str) const
 {
     wxString s;
     Write(s);
     str << (const char*) s.mb_str();
 {
     wxString s;
     Write(s);
     str << (const char*) s.mb_str();
-    return TRUE;
+    return true;
 }
 #endif
 
 }
 #endif
 
-bool wxVariantDataDate::Write(wxString& str) const
+bool wxVariantDataWxObjectPtr::Write(wxString& str) const
 {
 {
-    str = m_value.FormatDate();
-    return TRUE;
+    str.Printf(wxT("%s(%p)"), GetType().c_str(), static_cast<void*>(m_value));
+    return true;
 }
 
 #if wxUSE_STD_IOSTREAM
 }
 
 #if wxUSE_STD_IOSTREAM
-bool wxVariantDataDate::Read(wxSTD istream& WXUNUSED(str))
+bool wxVariantDataWxObjectPtr::Read(wxSTD istream& WXUNUSED(str))
 {
     // Not implemented
 {
     // Not implemented
-    return FALSE;
+    return false;
 }
 #endif
 
 }
 #endif
 
-bool wxVariantDataDate::Read(wxString& WXUNUSED(str))
+bool wxVariantDataWxObjectPtr::Read(wxString& WXUNUSED(str))
 {
     // Not implemented
 {
     // Not implemented
-    return FALSE;
+    return false;
 }
 }
-#endif
-  // wxUSE_TIMEDATE
 
 
-/*
- * wxVariantDataVoidPtr
- */
+// wxVariant
 
 
-class wxVariantDataVoidPtr: public wxVariantData
+wxVariant::wxVariant( wxObject* val, const wxString& name)
 {
 {
-DECLARE_DYNAMIC_CLASS(wxVariantDataVoidPtr)
-public:
-    wxVariantDataVoidPtr() { }
-    wxVariantDataVoidPtr(void* value) { m_value = value; }
+    m_refData = new wxVariantDataWxObjectPtr(val);
+    m_name = name;
+}
 
 
-    inline void* GetValue() const { return m_value; }
-    inline void SetValue(void* value) { m_value = value; }
+bool wxVariant::operator== (wxObject* value) const
+{
+    return (value == ((wxVariantDataWxObjectPtr*)GetData())->GetValue());
+}
 
 
-    virtual void Copy(wxVariantData& data);
-    virtual bool Eq(wxVariantData& data) const;
-#if wxUSE_STD_IOSTREAM
-    virtual bool Write(wxSTD ostream& str) const;
-#endif
-    virtual bool Write(wxString& str) const;
-#if wxUSE_STD_IOSTREAM
-    virtual bool Read(wxSTD istream& str);
-#endif
-    virtual bool Read(wxString& str);
-    virtual wxString GetType() const { return wxT("void*"); };
-       virtual wxVariantData* Clone() { return new wxVariantDataVoidPtr; }
-
-protected:
-    void* m_value;
-};
-
-IMPLEMENT_DYNAMIC_CLASS(wxVariantDataVoidPtr, wxVariantData)
-
-void wxVariantDataVoidPtr::Copy(wxVariantData& data)
+bool wxVariant::operator!= (wxObject* value) const
 {
 {
-    wxASSERT_MSG( (data.GetType() == wxT("void*")), wxT("wxVariantDataVoidPtr::Copy: Can't copy to this type of data") );
-
-    wxVariantDataVoidPtr& otherData = (wxVariantDataVoidPtr&) data;
-
-    otherData.m_value = m_value;
+    return (!((*this) == (wxObject*) value));
 }
 
 }
 
-bool wxVariantDataVoidPtr::Eq(wxVariantData& data) const
+void wxVariant::operator= (wxObject* value)
 {
 {
-    wxASSERT_MSG( (data.GetType() == wxT("void*")), wxT("wxVariantDataVoidPtr::Eq: argument mismatch") );
-
-    wxVariantDataVoidPtr& otherData = (wxVariantDataVoidPtr&) data;
-
-    return (otherData.m_value == m_value);
+    UnRef();
+    m_refData = new wxVariantDataWxObjectPtr(value);
 }
 
 }
 
-#if wxUSE_STD_IOSTREAM
-bool wxVariantDataVoidPtr::Write(wxSTD ostream& str) const
+wxObject* wxVariant::GetWxObjectPtr() const
 {
 {
-    wxString s;
-    Write(s);
-    str << (const char*) s.mb_str();
-    return TRUE;
+    return (wxObject*) ((wxVariantDataWxObjectPtr*) m_refData)->GetValue();
 }
 }
-#endif
 
 
-bool wxVariantDataVoidPtr::Write(wxString& str) const
-{
-    str.Printf(wxT("%ld"), (long) m_value);
-    return TRUE;
-}
+// ----------------------------------------------------------------------------
+// wxVariantDataVoidPtr
+// ----------------------------------------------------------------------------
 
 
-#if wxUSE_STD_IOSTREAM
-bool wxVariantDataVoidPtr::Read(wxSTD istream& WXUNUSED(str))
-{
-    // Not implemented
-    return FALSE;
-}
-#endif
-
-bool wxVariantDataVoidPtr::Read(wxString& WXUNUSED(str))
-{
-    // Not implemented
-    return FALSE;
-}
-
-/*
- * wxVariantDataDateTime
- */
-
-class wxVariantDataDateTime: public wxVariantData
+class wxVariantDataVoidPtr: public wxVariantData
 {
 {
-    DECLARE_DYNAMIC_CLASS(wxVariantDataDateTime)
-
 public:
 public:
-    wxVariantDataDateTime() { }
-    wxVariantDataDateTime(const wxDateTime& value) { m_value = value; }
-#if wxUSE_ODBC
-    wxVariantDataDateTime(const TIME_STRUCT* valptr) 
-        { m_value = wxDateTime(valptr->hour, valptr->minute, valptr->second); }
-    wxVariantDataDateTime(const DATE_STRUCT* valptr) 
-        { m_value = wxDateTime(valptr->day, (wxDateTime::Month) (valptr->month - 1),valptr->year); }
-    wxVariantDataDateTime(const TIMESTAMP_STRUCT* valptr) 
-        { m_value = wxDateTime(valptr->day, (wxDateTime::Month) (valptr->month - 1), valptr->year,
-                        valptr->hour, valptr->minute, valptr->second, valptr->fraction ); }
-#endif //ODBC
+    wxVariantDataVoidPtr() { }
+    wxVariantDataVoidPtr(void* value) { m_value = value; }
 
 
-    inline wxDateTime GetValue() const { return m_value; }
-    inline void SetValue(const wxDateTime& value) { m_value = value; }
+    inline void* GetValue() const { return m_value; }
+    inline void SetValue(void* value) { m_value = value; }
 
 
-    virtual void Copy(wxVariantData& data);
     virtual bool Eq(wxVariantData& data) const;
 #if wxUSE_STD_IOSTREAM
     virtual bool Eq(wxVariantData& data) const;
 #if wxUSE_STD_IOSTREAM
-    virtual bool Write(ostream& str) const;
+    virtual bool Write(wxSTD ostream& str) const;
 #endif
     virtual bool Write(wxString& str) const;
 #if wxUSE_STD_IOSTREAM
 #endif
     virtual bool Write(wxString& str) const;
 #if wxUSE_STD_IOSTREAM
-    virtual bool Read(istream& str);
+    virtual bool Read(wxSTD istream& str);
 #endif
     virtual bool Read(wxString& str);
 #endif
     virtual bool Read(wxString& str);
-    virtual wxString GetType() const { return wxT("datetime"); };
-    virtual wxVariantData* Clone() { return new wxVariantDataDateTime; }
+    virtual wxString GetType() const { return wxT("void*"); }
+    virtual wxVariantData* Clone() const { return new wxVariantDataVoidPtr(m_value); }
 
 protected:
 
 protected:
-    wxDateTime m_value;
+    void* m_value;
 };
 
 };
 
-
-IMPLEMENT_DYNAMIC_CLASS(wxVariantDataDateTime, wxVariantData)
-
-void wxVariantDataDateTime::Copy(wxVariantData& data)
-{
-    wxASSERT_MSG( (data.GetType() == wxT("datetime")), wxT("wxVariantDataDateTime::Copy: Can't copy to this type of data") );
-
-    wxVariantDataDateTime& otherData = (wxVariantDataDateTime&) data;
-
-    otherData.m_value = m_value;
-}
-
-
-bool wxVariantDataDateTime::Eq(wxVariantData& data) const
+bool wxVariantDataVoidPtr::Eq(wxVariantData& data) const
 {
 {
-    wxASSERT_MSG( (data.GetType() == wxT("datetime")), wxT("wxVariantDataDateTime::Eq: argument mismatch") );
+    wxASSERT_MSG( data.GetType() == wxT("void*"), wxT("wxVariantDataVoidPtr::Eq: argument mismatch") );
 
 
-    wxVariantDataDateTime& otherData = (wxVariantDataDateTime&) data;
+    wxVariantDataVoidPtr& otherData = (wxVariantDataVoidPtr&) data;
 
     return (otherData.m_value == m_value);
 }
 
 
     return (otherData.m_value == m_value);
 }
 
-
 #if wxUSE_STD_IOSTREAM
 #if wxUSE_STD_IOSTREAM
-bool wxVariantDataDateTime::Write(ostream& str) const
+bool wxVariantDataVoidPtr::Write(wxSTD ostream& str) const
 {
 {
-    // Not implemented
-    return FALSE;
+    wxString s;
+    Write(s);
+    str << (const char*) s.mb_str();
+    return true;
 }
 #endif
 
 }
 #endif
 
-
-bool wxVariantDataDateTime::Write(wxString& str) const
+bool wxVariantDataVoidPtr::Write(wxString& str) const
 {
 {
-    str = m_value.Format();
-    return TRUE;
+    str.Printf(wxT("%p"), m_value);
+    return true;
 }
 
 }
 
-
 #if wxUSE_STD_IOSTREAM
 #if wxUSE_STD_IOSTREAM
-bool wxVariantDataDateTime::Read(istream& WXUNUSED(str))
+bool wxVariantDataVoidPtr::Read(wxSTD istream& WXUNUSED(str))
 {
     // Not implemented
 {
     // Not implemented
-    return FALSE;
-}
-#endif
-
-
-bool wxVariantDataDateTime::Read(wxString& str)
-{
-    if(! m_value.ParseDateTime(str))
-        return FALSE;
-    return TRUE;
-}
-
-
-/*
- * wxVariant
- */
-
-IMPLEMENT_DYNAMIC_CLASS(wxVariant, wxObject)
-
-// Construction & destruction
-wxVariant::wxVariant()
-{
-    m_data = (wxVariantData*) NULL;
-}
-
-wxVariant::wxVariant(double val, const wxString& name)
-{
-    m_data = new wxVariantDataReal(val);
-    m_name = name;
-}
-
-wxVariant::wxVariant(long val, const wxString& name)
-{
-    m_data = new wxVariantDataLong(val);
-    m_name = name;
-}
-
-#ifdef HAVE_BOOL
-wxVariant::wxVariant(bool val, const wxString& name)
-{
-    m_data = new wxVariantDataBool(val);
-    m_name = name;
+    return false;
 }
 #endif
 
 }
 #endif
 
-wxVariant::wxVariant(char val, const wxString& name)
-{
-    m_data = new wxVariantDataChar(val);
-    m_name = name;
-}
-
-wxVariant::wxVariant(const wxString& val, const wxString& name)
-{
-    m_data = new wxVariantDataString(val);
-    m_name = name;
-}
-
-wxVariant::wxVariant(const wxChar* val, const wxString& name)
-{
-    m_data = new wxVariantDataString(wxString(val));
-    m_name = name;
-}
-
-wxVariant::wxVariant(const wxStringList& val, const wxString& name)
-{
-    m_data = new wxVariantDataStringList(val);
-    m_name = name;
-}
-
-wxVariant::wxVariant(const wxList& val, const wxString& name) // List of variants
-{
-    m_data = new wxVariantDataList(val);
-    m_name = name;
-}
-
-// For some reason, Watcom C++ can't link variant.cpp with time/date classes compiled
-#if wxUSE_TIMEDATE && !defined(__WATCOMC__)
-wxVariant::wxVariant(const wxTime& val, const wxString& name) // Time
-{
-    m_data = new wxVariantDataTime(val);
-    m_name = name;
-}
-
-wxVariant::wxVariant(const wxDate& val, const wxString& name) // Date
-{
-    m_data = new wxVariantDataDate(val);
-    m_name = name;
-}
-#endif
-
-wxVariant::wxVariant(void* val, const wxString& name) // Void ptr
-{
-    m_data = new wxVariantDataVoidPtr(val);
-    m_name = name;
-}
-
-wxVariant::wxVariant(const wxDateTime& val, const wxString& name) // Date
-{
-    m_data = new wxVariantDataDateTime(val);
-    m_name = name;
-}
-
-#if wxUSE_ODBC
-wxVariant::wxVariant(const TIME_STRUCT* valptr, const wxString& name) // Date
-{
-    m_data = new wxVariantDataDateTime(valptr);
-    m_name = name;
-}
-
-wxVariant::wxVariant(const TIMESTAMP_STRUCT* valptr, const wxString& name) // Date
-{
-    m_data = new wxVariantDataDateTime(valptr);
-    m_name = name;
-}
-
-wxVariant::wxVariant(const DATE_STRUCT* valptr, const wxString& name) // Date
+bool wxVariantDataVoidPtr::Read(wxString& WXUNUSED(str))
 {
 {
-    m_data = new wxVariantDataDateTime(valptr);
-    m_name = name;
+    // Not implemented
+    return false;
 }
 }
-#endif
 
 
-wxVariant::wxVariant(const wxVariant& variant)
-{
-    if (!variant.IsNull())
-    {
-        m_data = (wxVariantData*) variant.GetData()->GetClassInfo()->CreateObject();
-        variant.m_data->Copy(*m_data);
-    }
-    else
-        m_data = (wxVariantData*) NULL;
-    m_name = variant.m_name;
-}
+// wxVariant
 
 
-wxVariant::wxVariant(wxVariantData* data, const wxString& name) // User-defined data
+wxVariant::wxVariant( void* val, const wxString& name)
 {
 {
-    m_data = data;
+    m_refData = new wxVariantDataVoidPtr(val);
     m_name = name;
 }
 
     m_name = name;
 }
 
-wxVariant::~wxVariant()
-{
-    delete m_data;
-}
-
-
-// Make NULL (i.e. delete the data)
-void wxVariant::MakeNull()
-{
-    delete m_data;
-    m_data = NULL;
-}
-
-// Generic operators
-// Assignment
-void wxVariant::operator= (const wxVariant& variant)
-{
-    if (variant.IsNull())
-    {
-        MakeNull();
-        return;
-    }
-
-    if (IsNull() || (GetType() != variant.GetType()))
-    {
-        if (m_data)
-            delete m_data;
-        m_data = (wxVariantData*) variant.GetData()->GetClassInfo()->CreateObject();
-    }
-//    GetData()->Copy(* variant.GetData());
-    variant.GetData()->Copy(* GetData());
-}
-
-// Assignment using data, e.g.
-// myVariant = new wxStringVariantData("hello")
-void wxVariant::operator= (wxVariantData* variantData)
-{
-    MakeNull();
-    m_data = variantData;
-}
-
-bool wxVariant::operator== (const wxVariant& variant) const
-{
-    if (IsNull() || variant.IsNull())
-        return (IsNull() == variant.IsNull());
-
-    return (GetData()->Eq(* variant.GetData()));
-}
-
-bool wxVariant::operator!= (const wxVariant& variant) const
-{
-    return (!(*this == variant));
-}
-
-
-// Specific operators
-bool wxVariant::operator== (double value) const
-{
-    double thisValue;
-    if (!Convert(&thisValue))
-        return FALSE;
-    else
-        return (value == thisValue);
-}
-
-bool wxVariant::operator!= (double value) const
-{
-    return (!((*this) == value));
-}
-
-void wxVariant::operator= (double value)
-{
-    if (GetType() == wxT("double"))
-    {
-        ((wxVariantDataReal*)GetData())->SetValue(value);
-    }
-    else
-    {
-        if (m_data)
-            delete m_data;
-        m_data = new wxVariantDataReal(value);
-    }
-}
-
-bool wxVariant::operator== (long value) const
+bool wxVariant::operator== (void* value) const
 {
 {
-    long thisValue;
-    if (!Convert(&thisValue))
-        return FALSE;
-    else
-        return (value == thisValue);
+    return (value == ((wxVariantDataVoidPtr*)GetData())->GetValue());
 }
 
 }
 
-bool wxVariant::operator!= (long value) const
+bool wxVariant::operator!= (void* value) const
 {
 {
-    return (!((*this) == value));
+    return (!((*this) == (void*) value));
 }
 
 }
 
-void wxVariant::operator= (long value)
+void wxVariant::operator= (void* value)
 {
 {
-    if (GetType() == wxT("long"))
+    if (GetType() == wxT("void*") && (m_refData->GetRefCount() == 1))
     {
     {
-        ((wxVariantDataLong*)GetData())->SetValue(value);
+        ((wxVariantDataVoidPtr*)GetData())->SetValue(value);
     }
     else
     {
     }
     else
     {
-        if (m_data)
-            delete m_data;
-        m_data = new wxVariantDataLong(value);
+        UnRef();
+        m_refData = new wxVariantDataVoidPtr(value);
     }
 }
 
     }
 }
 
-bool wxVariant::operator== (char value) const
-{
-    char thisValue;
-    if (!Convert(&thisValue))
-        return FALSE;
-    else
-        return (value == thisValue);
-}
-
-bool wxVariant::operator!= (char value) const
+void* wxVariant::GetVoidPtr() const
 {
 {
-    return (!((*this) == value));
-}
+    // handling this specially is convenient when working with COM, see #9873
+    if ( IsNull() )
+        return NULL;
 
 
-void wxVariant::operator= (char value)
-{
-    if (GetType() == wxT("char"))
-    {
-        ((wxVariantDataChar*)GetData())->SetValue(value);
-    }
-    else
-    {
-        if (m_data)
-            delete m_data;
-        m_data = new wxVariantDataChar(value);
-    }
-}
+    wxASSERT( GetType() == wxT("void*") );
 
 
-#ifdef HAVE_BOOL
-bool wxVariant::operator== (bool value) const
-{
-    bool thisValue;
-    if (!Convert(&thisValue))
-        return FALSE;
-    else
-        return (value == thisValue);
+    return (void*) ((wxVariantDataVoidPtr*) m_refData)->GetValue();
 }
 
 }
 
-bool wxVariant::operator!= (bool value) const
-{
-    return (!((*this) == value));
-}
+// ----------------------------------------------------------------------------
+// wxVariantDataDateTime
+// ----------------------------------------------------------------------------
 
 
-void wxVariant::operator= (bool value)
-{
-    if (GetType() == wxT("bool"))
-    {
-        ((wxVariantDataBool*)GetData())->SetValue(value);
-    }
-    else
-    {
-        if (m_data)
-            delete m_data;
-        m_data = new wxVariantDataBool(value);
-    }
-}
-#endif // HAVE_BOOL
+#if wxUSE_DATETIME
 
 
-bool wxVariant::operator== (const wxString& value) const
+class wxVariantDataDateTime: public wxVariantData
 {
 {
-    wxString thisValue;
-    if (!Convert(&thisValue))
-        return FALSE;
+public:
+    wxVariantDataDateTime() { }
+    wxVariantDataDateTime(const wxDateTime& value) { m_value = value; }
 
 
-    return value == thisValue;
-}
+    inline wxDateTime GetValue() const { return m_value; }
+    inline void SetValue(const wxDateTime& value) { m_value = value; }
 
 
-bool wxVariant::operator!= (const wxString& value) const
-{
-    return (!((*this) == value));
-}
+    virtual bool Eq(wxVariantData& data) const;
+#if wxUSE_STD_IOSTREAM
+    virtual bool Write(wxSTD ostream& str) const;
+#endif
+    virtual bool Write(wxString& str) const;
+#if wxUSE_STD_IOSTREAM
+    virtual bool Read(wxSTD istream& str);
+#endif
+    virtual bool Read(wxString& str);
+    virtual wxString GetType() const { return wxT("datetime"); }
+    virtual wxVariantData* Clone() const { return new wxVariantDataDateTime(m_value); }
 
 
-void wxVariant::operator= (const wxString& value)
-{
-    if (GetType() == wxT("string"))
-    {
-        ((wxVariantDataString*)GetData())->SetValue(value);
-    }
-    else
-    {
-        if (m_data)
-            delete m_data;
-        m_data = new wxVariantDataString(value);
-    }
-}
+protected:
+    wxDateTime m_value;
+};
 
 
-void wxVariant::operator= (const wxChar* value)
-{
-    if (GetType() == wxT("string"))
-    {
-        ((wxVariantDataString*)GetData())->SetValue(wxString(value));
-    }
-    else
-    {
-        if (m_data)
-            delete m_data;
-        m_data = new wxVariantDataString(wxString(value));
-    }
-}
 
 
-bool wxVariant::operator== (const wxStringList& value) const
+bool wxVariantDataDateTime::Eq(wxVariantData& data) const
 {
 {
-    wxASSERT_MSG( (GetType() == wxT("stringlist")), wxT("Invalid type for == operator") );
+    wxASSERT_MSG( (data.GetType() == wxT("datetime")), wxT("wxVariantDataDateTime::Eq: argument mismatch") );
 
 
-    wxVariantDataStringList other(value);
-    return (m_data->Eq(other));
-}
+    wxVariantDataDateTime& otherData = (wxVariantDataDateTime&) data;
 
 
-bool wxVariant::operator!= (const wxStringList& value) const
-{
-    return (!((*this) == value));
+    return (otherData.m_value == m_value);
 }
 
 }
 
-void wxVariant::operator= (const wxStringList& value)
-{
-    if (GetType() == wxT("stringlist"))
-    {
-        ((wxVariantDataStringList*)GetData())->SetValue(value);
-    }
-    else
-    {
-        if (m_data)
-            delete m_data;
-        m_data = new wxVariantDataStringList(value);
-    }
-}
 
 
-bool wxVariant::operator== (const wxList& value) const
+#if wxUSE_STD_IOSTREAM
+bool wxVariantDataDateTime::Write(wxSTD ostream& str) const
 {
 {
-    wxASSERT_MSG( (GetType() == wxT("list")), wxT("Invalid type for == operator") );
-
-    wxVariantDataList other(value);
-    return (m_data->Eq(other));
+    wxString value;
+    Write( value );
+    str << value.c_str();
+    return true;
 }
 }
+#endif
 
 
-bool wxVariant::operator!= (const wxList& value) const
-{
-    return (!((*this) == value));
-}
 
 
-void wxVariant::operator= (const wxList& value)
+bool wxVariantDataDateTime::Write(wxString& str) const
 {
 {
-    if (GetType() == wxT("list"))
-    {
-        ((wxVariantDataList*)GetData())->SetValue(value);
-    }
+    if ( m_value.IsValid() )
+        str = m_value.Format();
     else
     else
-    {
-        if (m_data)
-            delete m_data;
-        m_data = new wxVariantDataList(value);
-    }
+        str = wxS("Invalid");
+    return true;
 }
 
 }
 
-// For some reason, Watcom C++ can't link variant.cpp with time/date classes compiled
-#if wxUSE_TIMEDATE && !defined(__WATCOMC__)
-bool wxVariant::operator== (const wxTime& value) const
-{
-    wxTime thisValue;
-    if (!Convert(&thisValue))
-        return FALSE;
-
-    return value == thisValue;
-}
-
-bool wxVariant::operator!= (const wxTime& value) const
-{
-    return (!((*this) == value));
-}
 
 
-void wxVariant::operator= (const wxTime& value)
+#if wxUSE_STD_IOSTREAM
+bool wxVariantDataDateTime::Read(wxSTD istream& WXUNUSED(str))
 {
 {
-    if (GetType() == wxT("time"))
-    {
-        ((wxVariantDataTime*)GetData())->SetValue(value);
-    }
-    else
-    {
-        if (m_data)
-            delete m_data;
-        m_data = new wxVariantDataTime(value);
-    }
+    // Not implemented
+    return false;
 }
 }
+#endif
 
 
-bool wxVariant::operator== (const wxDate& value) const
-{
-    wxDate thisValue;
-    if (!Convert(&thisValue))
-        return FALSE;
-
-    return (value == thisValue);
-}
 
 
-bool wxVariant::operator!= (const wxDate& value) const
-{
-    return (!((*this) == value));
-}
-
-void wxVariant::operator= (const wxDate& value)
+bool wxVariantDataDateTime::Read(wxString& str)
 {
 {
-    if (GetType() == wxT("date"))
-    {
-        ((wxVariantDataTime*)GetData())->SetValue(value);
-    }
-    else
+    if ( str == wxS("Invalid") )
     {
     {
-        if (m_data)
-            delete m_data;
-        m_data = new wxVariantDataDate(value);
+        m_value = wxInvalidDateTime;
+        return true;
     }
     }
-}
-#endif
 
 
-bool wxVariant::operator== (void* value) const
-{
-    return (value == ((wxVariantDataVoidPtr*)GetData())->GetValue());
+    wxString::const_iterator end;
+    return m_value.ParseDateTime(str, &end) && end == str.end();
 }
 
 }
 
-bool wxVariant::operator!= (void* value) const
-{
-    return (!((*this) == (void*) value));
-}
-
-void wxVariant::operator= (void* value)
-{
-    if (GetType() == wxT("void*"))
-    {
-        ((wxVariantDataVoidPtr*)GetData())->SetValue(value);
-    }
-    else
-    {
-        if (m_data)
-            delete m_data;
-        m_data = new wxVariantDataVoidPtr(value);
-    }
+// wxVariant
+
+wxVariant::wxVariant(const wxDateTime& val, const wxString& name) // Date
+{
+    m_refData = new wxVariantDataDateTime(val);
+    m_name = name;
 }
 
 bool wxVariant::operator== (const wxDateTime& value) const
 {
     wxDateTime thisValue;
     if (!Convert(&thisValue))
 }
 
 bool wxVariant::operator== (const wxDateTime& value) const
 {
     wxDateTime thisValue;
     if (!Convert(&thisValue))
-        return FALSE;
+        return false;
 
     return value.IsEqualTo(thisValue);
 }
 
     return value.IsEqualTo(thisValue);
 }
@@ -1665,256 +1266,335 @@ bool wxVariant::operator!= (const wxDateTime& value) const
 
 void wxVariant::operator= (const wxDateTime& value)
 {
 
 void wxVariant::operator= (const wxDateTime& value)
 {
-    if (GetType() == wxT("datetime"))
+    if (GetType() == wxT("datetime") &&
+        m_refData->GetRefCount() == 1)
     {
         ((wxVariantDataDateTime*)GetData())->SetValue(value);
     }
     else
     {
     {
         ((wxVariantDataDateTime*)GetData())->SetValue(value);
     }
     else
     {
-        if (m_data)
-            delete m_data;
-        m_data = new wxVariantDataDateTime(value);
+        UnRef();
+        m_refData = new wxVariantDataDateTime(value);
     }
 }
 
     }
 }
 
-
-#if wxUSE_ODBC
-void wxVariant::operator= (const DATE_STRUCT* value)
+wxDateTime wxVariant::GetDateTime() const
 {
 {
-    if (m_data)
-        delete m_data;
-    m_data = new wxVariantDataDateTime(value);
+    wxDateTime value;
+    if (!Convert(& value))
+    {
+        wxFAIL_MSG(wxT("Could not convert to a datetime"));
+    }
+
+    return value;
 }
 
 }
 
+#endif // wxUSE_DATETIME
+
+// ----------------------------------------------------------------------------
+// wxVariantDataArrayString
+// ----------------------------------------------------------------------------
 
 
-void wxVariant::operator= (const TIME_STRUCT* value)
+class wxVariantDataArrayString: public wxVariantData
 {
 {
-    if (m_data)
-        delete m_data;
-    m_data = new wxVariantDataDateTime(value);
-}
+public:
+    wxVariantDataArrayString() { }
+    wxVariantDataArrayString(const wxArrayString& value) { m_value = value; }
 
 
+    wxArrayString GetValue() const { return m_value; }
+    void SetValue(const wxArrayString& value) { m_value = value; }
 
 
-void wxVariant::operator= (const TIMESTAMP_STRUCT* value)
+    virtual bool Eq(wxVariantData& data) const;
+#if wxUSE_STD_IOSTREAM
+    virtual bool Write(wxSTD ostream& str) const;
+#endif
+    virtual bool Write(wxString& str) const;
+#if wxUSE_STD_IOSTREAM
+    virtual bool Read(wxSTD istream& str);
+#endif
+    virtual bool Read(wxString& str);
+    virtual wxString GetType() const { return wxT("arrstring"); }
+    virtual wxVariantData* Clone() const { return new wxVariantDataArrayString(m_value); }
+
+protected:
+    wxArrayString m_value;
+};
+
+bool wxVariantDataArrayString::Eq(wxVariantData& data) const
 {
 {
-    if (m_data)
-        delete m_data;
-    m_data = new wxVariantDataDateTime(value);
+    wxASSERT_MSG( data.GetType() == GetType(), wxT("wxVariantDataArrayString::Eq: argument mismatch") );
+
+    wxVariantDataArrayString& otherData = (wxVariantDataArrayString&) data;
+
+    return otherData.m_value == m_value;
 }
 
 }
 
+#if wxUSE_STD_IOSTREAM
+bool wxVariantDataArrayString::Write(wxSTD ostream& WXUNUSED(str)) const
+{
+    // Not implemented
+    return false;
+}
 #endif
 
 #endif
 
-// Treat a list variant as an array
-wxVariant wxVariant::operator[] (size_t idx) const
+bool wxVariantDataArrayString::Write(wxString& str) const
 {
 {
-    wxASSERT_MSG( (GetType() == wxT("list") || GetType() == wxT("stringlist")), wxT("Invalid type for array operator") );
-
-    if (GetType() == wxT("list"))
-    {
-        wxVariantDataList* data = (wxVariantDataList*) m_data;
-        wxASSERT_MSG( (idx < (size_t) data->GetValue().Number()), wxT("Invalid index for array") );
-        return * (wxVariant*) (data->GetValue().Nth(idx)->Data());
-    }
-    else if (GetType() == wxT("stringlist"))
+    size_t count = m_value.GetCount();
+    for ( size_t n = 0; n < count; n++ )
     {
     {
-        wxVariantDataStringList* data = (wxVariantDataStringList*) m_data;
-        wxASSERT_MSG( (idx < (size_t) data->GetValue().Number()), wxT("Invalid index for array") );
+        if ( n )
+            str += _T(';');
 
 
-        wxVariant variant( wxString( (wxChar*) (data->GetValue().Nth(idx)->Data()) ));
-        return variant;
+        str += m_value[n];
     }
     }
-    return wxNullVariant;
+
+    return true;
 }
 
 }
 
-wxVariant& wxVariant::operator[] (size_t idx)
+
+#if wxUSE_STD_IOSTREAM
+bool wxVariantDataArrayString::Read(wxSTD istream& WXUNUSED(str))
 {
 {
-    // We can't return a reference to a variant for a string list, since the string
-    // is actually stored as a char*, not a variant.
+    // Not implemented
+    return false;
+}
+#endif
 
 
-    wxASSERT_MSG( (GetType() == wxT("list")), wxT("Invalid type for array operator") );
 
 
-    wxVariantDataList* data = (wxVariantDataList*) m_data;
-    wxASSERT_MSG( (idx < (size_t) data->GetValue().Number()), wxT("Invalid index for array") );
+bool wxVariantDataArrayString::Read(wxString& str)
+{
+    wxStringTokenizer tk(str, _T(";"));
+    while ( tk.HasMoreTokens() )
+    {
+        m_value.Add(tk.GetNextToken());
+    }
 
 
-    return * (wxVariant*) (data->GetValue().Nth(idx)->Data());
+    return true;
 }
 
 }
 
-// Return the number of elements in a list
-int wxVariant::GetCount() const
+// wxVariant
+
+wxVariant::wxVariant(const wxArrayString& val, const wxString& name) // Strings
 {
 {
-    wxASSERT_MSG( (GetType() == wxT("list") || GetType() == wxT("stringlist")), wxT("Invalid type for GetCount()") );
+    m_refData = new wxVariantDataArrayString(val);
+    m_name = name;
+}
 
 
-    if (GetType() == wxT("list"))
+bool wxVariant::operator==(const wxArrayString& WXUNUSED(value)) const
+{
+    wxFAIL_MSG( _T("TODO") );
+
+    return false;
+}
+
+bool wxVariant::operator!=(const wxArrayString& value) const
+{
+    return !(*this == value);
+}
+
+void wxVariant::operator=(const wxArrayString& value)
+{
+    if (GetType() == wxT("arrstring") &&
+        m_refData->GetRefCount() == 1)
     {
     {
-        wxVariantDataList* data = (wxVariantDataList*) m_data;
-        return data->GetValue().Number();
+        ((wxVariantDataArrayString *)GetData())->SetValue(value);
     }
     }
-    else if (GetType() == wxT("stringlist"))
+    else
     {
     {
-        wxVariantDataStringList* data = (wxVariantDataStringList*) m_data;
-        return data->GetValue().Number();
+        UnRef();
+        m_refData = new wxVariantDataArrayString(value);
     }
     }
-    return 0;
 }
 
 }
 
-wxString wxVariant::MakeString() const
+wxArrayString wxVariant::GetArrayString() const
 {
 {
-    if (!IsNull())
-    {
-        wxString str;
-        if (GetData()->Write(str))
-            return str;
-    }
-    return wxString(wxT(""));
+    if ( GetType() == wxT("arrstring") )
+        return ((wxVariantDataArrayString *)GetData())->GetValue();
+
+    return wxArrayString();
 }
 
 }
 
-// Accessors
+// ----------------------------------------------------------------------------
+// wxVariantDataList
+// ----------------------------------------------------------------------------
 
 
-void wxVariant::SetData(wxVariantData* data)
+class WXDLLIMPEXP_BASE wxVariantDataList: public wxVariantData
 {
 {
-    if (m_data) delete m_data;
-    m_data = data;
-}
+public:
+    wxVariantDataList() {}
+    wxVariantDataList(const wxVariantList& list);
+    virtual ~wxVariantDataList();
 
 
+    wxVariantList& GetValue() { return m_value; }
+    void SetValue(const wxVariantList& value) ;
 
 
-// Returns a string representing the type of the variant,
-// e.g. "string", "bool", "stringlist", "list", "double", "long"
-wxString wxVariant::GetType() const
-{
-    if (IsNull())
-        return wxString(wxT("null"));
-    else
-        return m_data->GetType();
-}
+    virtual bool Eq(wxVariantData& data) const;
+#if wxUSE_STD_IOSTREAM
+    virtual bool Write(wxSTD ostream& str) const;
+#endif
+    virtual bool Write(wxString& str) const;
+#if wxUSE_STD_IOSTREAM
+    virtual bool Read(wxSTD istream& str);
+#endif
+    virtual bool Read(wxString& str);
+    virtual wxString GetType() const { return wxT("list"); }
 
 
+    void Clear();
 
 
-bool wxVariant::IsType(const wxString& type) const
+    wxVariantData* Clone() const { return new wxVariantDataList(m_value); }
+protected:
+    wxVariantList  m_value;
+};
+
+wxVariantDataList::wxVariantDataList(const wxVariantList& list)
 {
 {
-    return (GetType() == type);
+    SetValue(list);
 }
 
 }
 
+wxVariantDataList::~wxVariantDataList()
+{
+    Clear();
+}
 
 
-// Value accessors
-double wxVariant::GetReal() const
+void wxVariantDataList::SetValue(const wxVariantList& value)
 {
 {
-    double value;
-    if (Convert(& value))
-        return value;
-    else
+    Clear();
+    wxVariantList::compatibility_iterator node = value.GetFirst();
+    while (node)
     {
     {
-        wxFAIL_MSG(wxT("Could not convert to a real number"));
-        return 0.0;
+        wxVariant* var = node->GetData();
+        m_value.Append(new wxVariant(*var));
+        node = node->GetNext();
     }
 }
 
     }
 }
 
-long wxVariant::GetInteger() const
+void wxVariantDataList::Clear()
 {
 {
-    long value;
-    if (Convert(& value))
-        return value;
-    else
+    wxVariantList::compatibility_iterator node = m_value.GetFirst();
+    while (node)
     {
     {
-        wxFAIL_MSG(wxT("Could not convert to an integer"));
-        return 0;
+        wxVariant* var = node->GetData();
+        delete var;
+        node = node->GetNext();
     }
     }
+    m_value.Clear();
 }
 
 }
 
-char wxVariant::GetChar() const
+bool wxVariantDataList::Eq(wxVariantData& data) const
 {
 {
-    char value;
-    if (Convert(& value))
-        return value;
-    else
+    wxASSERT_MSG( (data.GetType() == wxT("list")), wxT("wxVariantDataList::Eq: argument mismatch") );
+
+    wxVariantDataList& listData = (wxVariantDataList&) data;
+    wxVariantList::compatibility_iterator node1 = m_value.GetFirst();
+    wxVariantList::compatibility_iterator node2 = listData.GetValue().GetFirst();
+    while (node1 && node2)
     {
     {
-        wxFAIL_MSG(wxT("Could not convert to a char"));
-        return 0;
+        wxVariant* var1 = node1->GetData();
+        wxVariant* var2 = node2->GetData();
+        if ((*var1) != (*var2))
+            return false;
+        node1 = node1->GetNext();
+        node2 = node2->GetNext();
     }
     }
+    if (node1 || node2) return false;
+    return true;
 }
 
 }
 
-bool wxVariant::GetBool() const
+#if wxUSE_STD_IOSTREAM
+bool wxVariantDataList::Write(wxSTD ostream& str) const
 {
 {
-    bool value;
-    if (Convert(& value))
-        return value;
-    else
-    {
-        wxFAIL_MSG(wxT("Could not convert to a bool"));
-        return 0;
-    }
+    wxString s;
+    Write(s);
+    str << (const char*) s.mb_str();
+    return true;
 }
 }
+#endif
 
 
-wxString wxVariant::GetString() const
+bool wxVariantDataList::Write(wxString& str) const
 {
 {
-    wxString value;
-    if (!Convert(& value))
+    str = wxEmptyString;
+    wxVariantList::compatibility_iterator node = m_value.GetFirst();
+    while (node)
     {
     {
-        wxFAIL_MSG(wxT("Could not convert to a string"));
+        wxVariant* var = node->GetData();
+        if (node != m_value.GetFirst())
+          str += wxT(" ");
+        wxString str1;
+        str += var->MakeString();
+        node = node->GetNext();
     }
 
     }
 
-    return value;
+    return true;
 }
 
 }
 
-// For some reason, Watcom C++ can't link variant.cpp with time/date classes compiled
-#if wxUSE_TIMEDATE && !defined(__WATCOMC__)
-wxTime wxVariant::GetTime() const
+#if wxUSE_STD_IOSTREAM
+bool wxVariantDataList::Read(wxSTD istream& WXUNUSED(str))
 {
 {
-    wxTime value;
-    if (!Convert(& value))
-    {
-        wxFAIL_MSG(wxT("Could not convert to a time"));
-    }
-
-    return value;
+    wxFAIL_MSG(wxT("Unimplemented"));
+    // TODO
+    return false;
 }
 }
+#endif
 
 
-wxDate wxVariant::GetDate() const
+bool wxVariantDataList::Read(wxString& WXUNUSED(str))
 {
 {
-    wxDate value;
-    if (!Convert(& value))
-    {
-        wxFAIL_MSG(wxT("Could not convert to a date"));
-    }
+    wxFAIL_MSG(wxT("Unimplemented"));
+    // TODO
+    return false;
+}
 
 
-    return value;
+// wxVariant
+
+wxVariant::wxVariant(const wxVariantList& val, const wxString& name) // List of variants
+{
+    m_refData = new wxVariantDataList(val);
+    m_name = name;
 }
 }
-#endif // wxUSE_TIMEDATE
 
 
-void* wxVariant::GetVoidPtr() const
+bool wxVariant::operator== (const wxVariantList& value) const
 {
 {
-    wxASSERT( (GetType() == wxT("void*")) );
+    wxASSERT_MSG( (GetType() == wxT("list")), wxT("Invalid type for == operator") );
 
 
-    return (void*) ((wxVariantDataVoidPtr*) m_data)->GetValue();
+    wxVariantDataList other(value);
+    return (GetData()->Eq(other));
 }
 
 }
 
-wxDateTime wxVariant::GetDateTime() const
+bool wxVariant::operator!= (const wxVariantList& value) const
 {
 {
-    wxDateTime value;
-    if (!Convert(& value))
+    return (!((*this) == value));
+}
+
+void wxVariant::operator= (const wxVariantList& value)
+{
+    if (GetType() == wxT("list") &&
+        m_refData->GetRefCount() == 1)
     {
     {
-        wxFAIL_MSG(wxT("Could not convert to a datetime"));
+        ((wxVariantDataList*)GetData())->SetValue(value);
+    }
+    else
+    {
+        UnRef();
+        m_refData = new wxVariantDataList(value);
     }
     }
-
-    return value;
 }
 
 }
 
-wxList& wxVariant::GetList() const
+wxVariantList& wxVariant::GetList() const
 {
     wxASSERT( (GetType() == wxT("list")) );
 
 {
     wxASSERT( (GetType() == wxT("list")) );
 
-    return (wxList&) ((wxVariantDataList*) m_data)->GetValue();
+    return (wxVariantList&) ((wxVariantDataList*) m_refData)->GetValue();
 }
 
 }
 
-wxStringList& wxVariant::GetStringList() const
+// Make empty list
+void wxVariant::NullList()
 {
 {
-    wxASSERT( (GetType() == wxT("stringlist")) );
-
-    return (wxStringList&) ((wxVariantDataStringList*) m_data)->GetValue();
+    SetData(new wxVariantDataList());
 }
 
 // Append to list
 void wxVariant::Append(const wxVariant& value)
 {
 }
 
 // Append to list
 void wxVariant::Append(const wxVariant& value)
 {
-    wxList& list = GetList();
+    wxVariantList& list = GetList();
 
     list.Append(new wxVariant(value));
 }
 
     list.Append(new wxVariant(value));
 }
@@ -1922,38 +1602,38 @@ void wxVariant::Append(const wxVariant& value)
 // Insert at front of list
 void wxVariant::Insert(const wxVariant& value)
 {
 // Insert at front of list
 void wxVariant::Insert(const wxVariant& value)
 {
-    wxList& list = GetList();
+    wxVariantList& list = GetList();
 
     list.Insert(new wxVariant(value));
 }
 
 
     list.Insert(new wxVariant(value));
 }
 
-// Returns TRUE if the variant is a member of the list
+// Returns true if the variant is a member of the list
 bool wxVariant::Member(const wxVariant& value) const
 {
 bool wxVariant::Member(const wxVariant& value) const
 {
-    wxList& list = GetList();
+    wxVariantList& list = GetList();
 
 
-    wxNode* node = list.First();
+    wxVariantList::compatibility_iterator node = list.GetFirst();
     while (node)
     {
     while (node)
     {
-        wxVariant* other = (wxVariant*) node->Data();
+        wxVariant* other = node->GetData();
         if (value == *other)
         if (value == *other)
-            return TRUE;
-        node = node->Next();
+            return true;
+        node = node->GetNext();
     }
     }
-    return FALSE;
+    return false;
 }
 
 // Deletes the nth element of the list
 }
 
 // Deletes the nth element of the list
-bool wxVariant::Delete(int item)
+bool wxVariant::Delete(size_t item)
 {
 {
-    wxList& list = GetList();
+    wxVariantList& list = GetList();
 
 
-    wxASSERT_MSG( (item < list.Number()), wxT("Invalid index to Delete") );
-    wxNode* node = list.Nth(item);
-    wxVariant* variant = (wxVariant*) node->Data();
+    wxASSERT_MSG( (item < list.GetCount()), wxT("Invalid index to Delete") );
+    wxVariantList::compatibility_iterator node = list.Item(item);
+    wxVariant* variant = node->GetData();
     delete variant;
     delete variant;
-    delete node;
-    return TRUE;
+    list.Erase(node);
+    return true;
 }
 
 // Clear list
 }
 
 // Clear list
@@ -1961,143 +1641,189 @@ void wxVariant::ClearList()
 {
     if (!IsNull() && (GetType() == wxT("list")))
     {
 {
     if (!IsNull() && (GetType() == wxT("list")))
     {
-        ((wxVariantDataList*) m_data)->Clear();
+        ((wxVariantDataList*) m_refData)->Clear();
     }
     else
     {
     }
     else
     {
-        if (GetType() != wxT("list"))
-        {
-            delete m_data;
-            m_data = NULL;
-        }
-        m_data = new wxVariantDataList;
+        if (!GetType().IsSameAs(wxT("list")))
+            UnRef();
+
+        m_refData = new wxVariantDataList;
+    }
+}
+
+// Treat a list variant as an array
+wxVariant wxVariant::operator[] (size_t idx) const
+{
+    wxASSERT_MSG( GetType() == wxT("list"), wxT("Invalid type for array operator") );
+
+    if (GetType() == wxT("list"))
+    {
+        wxVariantDataList* data = (wxVariantDataList*) m_refData;
+        wxASSERT_MSG( (idx < data->GetValue().GetCount()), wxT("Invalid index for array") );
+        return *(data->GetValue().Item(idx)->GetData());
+    }
+    return wxNullVariant;
+}
+
+wxVariant& wxVariant::operator[] (size_t idx)
+{
+    // We can't return a reference to a variant for a string list, since the string
+    // is actually stored as a char*, not a variant.
+
+    wxASSERT_MSG( (GetType() == wxT("list")), wxT("Invalid type for array operator") );
+
+    wxVariantDataList* data = (wxVariantDataList*) m_refData;
+    wxASSERT_MSG( (idx < data->GetValue().GetCount()), wxT("Invalid index for array") );
+
+    return * (data->GetValue().Item(idx)->GetData());
+}
+
+// Return the number of elements in a list
+size_t wxVariant::GetCount() const
+{
+    wxASSERT_MSG( GetType() == wxT("list"), wxT("Invalid type for GetCount()") );
+
+    if (GetType() == wxT("list"))
+    {
+        wxVariantDataList* data = (wxVariantDataList*) m_refData;
+        return data->GetValue().GetCount();
     }
     }
+    return 0;
 }
 
 }
 
+// ----------------------------------------------------------------------------
 // Type conversion
 // Type conversion
+// ----------------------------------------------------------------------------
+
 bool wxVariant::Convert(long* value) const
 {
     wxString type(GetType());
     if (type == wxT("double"))
 bool wxVariant::Convert(long* value) const
 {
     wxString type(GetType());
     if (type == wxT("double"))
-        *value = (long) (((wxVariantDataReal*)GetData())->GetValue());
+        *value = (long) (((wxVariantDoubleData*)GetData())->GetValue());
     else if (type == wxT("long"))
         *value = ((wxVariantDataLong*)GetData())->GetValue();
     else if (type == wxT("long"))
         *value = ((wxVariantDataLong*)GetData())->GetValue();
-#ifdef HAVE_BOOL
     else if (type == wxT("bool"))
         *value = (long) (((wxVariantDataBool*)GetData())->GetValue());
     else if (type == wxT("bool"))
         *value = (long) (((wxVariantDataBool*)GetData())->GetValue());
-#endif
     else if (type == wxT("string"))
     else if (type == wxT("string"))
-        *value = wxAtol((const wxChar*) ((wxVariantDataString*)GetData())->GetValue());
+        *value = wxAtol(((wxVariantDataString*)GetData())->GetValue());
     else
     else
-        return FALSE;
+        return false;
 
 
-    return TRUE;
+    return true;
 }
 
 bool wxVariant::Convert(bool* value) const
 {
     wxString type(GetType());
     if (type == wxT("double"))
 }
 
 bool wxVariant::Convert(bool* value) const
 {
     wxString type(GetType());
     if (type == wxT("double"))
-        *value = ((int) (((wxVariantDataReal*)GetData())->GetValue()) != 0);
+        *value = ((int) (((wxVariantDoubleData*)GetData())->GetValue()) != 0);
     else if (type == wxT("long"))
         *value = (((wxVariantDataLong*)GetData())->GetValue() != 0);
     else if (type == wxT("long"))
         *value = (((wxVariantDataLong*)GetData())->GetValue() != 0);
-#ifdef HAVE_BOOL
     else if (type == wxT("bool"))
         *value = ((wxVariantDataBool*)GetData())->GetValue();
     else if (type == wxT("bool"))
         *value = ((wxVariantDataBool*)GetData())->GetValue();
-#endif
     else if (type == wxT("string"))
     {
         wxString val(((wxVariantDataString*)GetData())->GetValue());
         val.MakeLower();
     else if (type == wxT("string"))
     {
         wxString val(((wxVariantDataString*)GetData())->GetValue());
         val.MakeLower();
-        if (val == wxT("true") || val == wxT("yes"))
-            *value = TRUE;
-        else if (val == wxT("false") || val == wxT("no"))
-            *value = FALSE;
+        if (val == wxT("true") || val == wxT("yes") || val == wxT('1') )
+            *value = true;
+        else if (val == wxT("false") || val == wxT("no") || val == wxT('0') )
+            *value = false;
         else
         else
-            return FALSE;
+            return false;
     }
     else
     }
     else
-        return FALSE;
+        return false;
 
 
-    return TRUE;
+    return true;
 }
 
 bool wxVariant::Convert(double* value) const
 {
     wxString type(GetType());
     if (type == wxT("double"))
 }
 
 bool wxVariant::Convert(double* value) const
 {
     wxString type(GetType());
     if (type == wxT("double"))
-        *value = ((wxVariantDataReal*)GetData())->GetValue();
+        *value = ((wxVariantDoubleData*)GetData())->GetValue();
     else if (type == wxT("long"))
         *value = (double) (((wxVariantDataLong*)GetData())->GetValue());
     else if (type == wxT("long"))
         *value = (double) (((wxVariantDataLong*)GetData())->GetValue());
-#ifdef HAVE_BOOL
     else if (type == wxT("bool"))
         *value = (double) (((wxVariantDataBool*)GetData())->GetValue());
     else if (type == wxT("bool"))
         *value = (double) (((wxVariantDataBool*)GetData())->GetValue());
-#endif
     else if (type == wxT("string"))
     else if (type == wxT("string"))
-        *value = (double) wxAtof((const wxChar*) ((wxVariantDataString*)GetData())->GetValue());
+        *value = (double) wxAtof(((wxVariantDataString*)GetData())->GetValue());
     else
     else
-        return FALSE;
+        return false;
 
 
-    return TRUE;
+    return true;
 }
 
 }
 
-bool wxVariant::Convert(char* value) const
+bool wxVariant::Convert(wxUniChar* value) const
 {
     wxString type(GetType());
     if (type == wxT("char"))
         *value = ((wxVariantDataChar*)GetData())->GetValue();
     else if (type == wxT("long"))
         *value = (char) (((wxVariantDataLong*)GetData())->GetValue());
 {
     wxString type(GetType());
     if (type == wxT("char"))
         *value = ((wxVariantDataChar*)GetData())->GetValue();
     else if (type == wxT("long"))
         *value = (char) (((wxVariantDataLong*)GetData())->GetValue());
-#ifdef HAVE_BOOL
     else if (type == wxT("bool"))
         *value = (char) (((wxVariantDataBool*)GetData())->GetValue());
     else if (type == wxT("bool"))
         *value = (char) (((wxVariantDataBool*)GetData())->GetValue());
-#endif
     else
     else
-        return FALSE;
+        return false;
 
 
-    return TRUE;
+    return true;
 }
 
 }
 
-bool wxVariant::Convert(wxString* value) const
+bool wxVariant::Convert(char* value) const
 {
 {
-    *value = MakeString();
-    return TRUE;
+    wxUniChar ch;
+    if ( !Convert(&ch) )
+        return false;
+    *value = ch;
+    return true;
 }
 
 }
 
-// For some reason, Watcom C++ can't link variant.cpp with time/date classes compiled
-#if wxUSE_TIMEDATE && !defined(__WATCOMC__)
-bool wxVariant::Convert(wxTime* value) const
+bool wxVariant::Convert(wchar_t* value) const
 {
 {
-    wxString type(GetType());
-    if (type == wxT("time"))
-        *value = ((wxVariantDataTime*)GetData())->GetValue();
-    else if (type == wxT("date"))
-        *value = wxTime(((wxVariantDataDate*)GetData())->GetValue());
-    else
-        return FALSE;
-
-    return TRUE;
+    wxUniChar ch;
+    if ( !Convert(&ch) )
+        return false;
+    *value = ch;
+    return true;
 }
 
 }
 
-bool wxVariant::Convert(wxDate* value) const
+bool wxVariant::Convert(wxString* value) const
 {
 {
-    wxString type(GetType());
-    if (type == wxT("date"))
-        *value = ((wxVariantDataDate*)GetData())->GetValue();
-    else
-        return FALSE;
-
-    return TRUE;
+    *value = MakeString();
+    return true;
 }
 }
-#endif // wxUSE_TIMEDATE
 
 
+#if wxUSE_DATETIME
 bool wxVariant::Convert(wxDateTime* value) const
 {
     wxString type(GetType());
     if (type == wxT("datetime"))
 bool wxVariant::Convert(wxDateTime* value) const
 {
     wxString type(GetType());
     if (type == wxT("datetime"))
+    {
         *value = ((wxVariantDataDateTime*)GetData())->GetValue();
         *value = ((wxVariantDataDateTime*)GetData())->GetValue();
-    else
-        return FALSE;
+        return true;
+    }
+
+    // Fallback to string conversion
+    wxString val;
+    if ( !Convert(&val) )
+        return false;
 
 
-    return TRUE;
+    // Try to parse this as either date and time, only date or only time
+    // checking that the entire string was parsed
+    wxString::const_iterator end;
+    if ( value->ParseDateTime(val, &end) && end == val.end() )
+        return true;
+
+    if ( value->ParseDate(val, &end) && end == val.end() )
+        return true;
+
+    if ( value->ParseTime(val, &end) && end == val.end() )
+        return true;
+
+    return false;
 }
 }
+#endif // wxUSE_DATETIME
+
+#endif // wxUSE_VARIANT
wxWidgets (Impactor)