]> git.saurik.com Git - wxWidgets.git/blobdiff - src/msw/ole/automtn.cpp
remove incorrect cast of wxString to char* (unnecessary and breaks wxUSE_STL build)
[wxWidgets.git] / src / msw / ole / automtn.cpp
index 1224bc06276d96ecd34d8a8ab7b6fb33c6921c0d..4580beea95552b75deb00e20c795454ba850a275 100644 (file)
@@ -1,5 +1,5 @@
 /////////////////////////////////////////////////////////////////////////////
-// Name:        automtn.cpp
+// Name:        src/msw/ole/automtn.cpp
 // Purpose:     OLE automation utilities
 // Author:      Julian Smart
 // Modified by:
@@ -9,29 +9,30 @@
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
-#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)
-#pragma implementation "automtn.h"
-#endif
-
 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
 
 #if defined(__BORLANDC__)
-#pragma hdrstop
+    #pragma hdrstop
 #endif
 
-#include "wx/defs.h"
-
-// Watcom C++ gives a linker error if this is compiled in.
 // With Borland C++, all samples crash if this is compiled in.
-#if wxUSE_OLE && !(defined(__BORLANDC__) && (__BORLANDC__ < 0x520)) && !defined(__CYGWIN10__)
+#if (defined(__BORLANDC__) && (__BORLANDC__ < 0x520)) || defined(__CYGWIN10__)
+    #undef wxUSE_OLE_AUTOMATION
+    #define wxUSE_OLE_AUTOMATION 0
+#endif
+
+#if wxUSE_OLE_AUTOMATION
+
+#ifndef WX_PRECOMP
+    #include "wx/log.h"
+    #include "wx/math.h"
+#endif
 
 #define _FORCENAMELESSUNION
-#include "wx/log.h"
 #include "wx/msw/private.h"
 #include "wx/msw/ole/oleutils.h"
 #include "wx/msw/ole/automtn.h"
-#include "wx/math.h"
 
 #ifdef __WXWINCE__
 #include "wx/msw/wince/time.h"
 
 #include <oleauto.h>
 
-// Verifies will fail if the needed buffer size is too large
-#define MAX_TIME_BUFFER_SIZE    128         // matches that in timecore.cpp
-#define MIN_DATE                (-657434L)  // about year 100
-#define MAX_DATE                2958465L    // about year 9999
-
-// Half a second, expressed in days
-#define HALF_SECOND  (1.0/172800.0)
-
-// One-based array of days in year at month start
-static int rgMonthDays[13] =
-    {0, 31, 59, 90, 120, 151, 181, 212, 243, 273, 304, 334, 365};
-
 #if wxUSE_DATETIME
 #include "wx/datetime.h"
-
-static BOOL OleDateFromTm(WORD wYear, WORD wMonth, WORD wDay,
-    WORD wHour, WORD wMinute, WORD wSecond, DATE& dtDest);
-static BOOL TmFromOleDate(DATE dtSrc, struct tm& tmDest);
-#endif // wxUSE_TIMEDATE
+#endif // wxUSE_DATETIME
 
 static void ClearVariant(VARIANTARG *pvarg) ;
 static void ReleaseVariant(VARIANTARG *pvarg) ;
@@ -110,7 +95,7 @@ bool wxAutomationObject::Invoke(const wxString& member, int action,
     {
         // Use dot notation to get the next object
         wxString member2(nonConstMember.Left((size_t) ch));
-        wxString rest(nonConstMember.Right(nonConstMember.Length() - ch - 1));
+        wxString rest(nonConstMember.Right(nonConstMember.length() - ch - 1));
         wxAutomationObject obj;
         if (!GetObject(obj, member2))
             return false;
@@ -174,7 +159,7 @@ bool wxAutomationObject::Invoke(const wxString& member, int action,
     {
         namedArgCount = 1;
         dispIds[1] = DISPID_PROPERTYPUT;
-        vReturnPtr = (VARIANTARG*) NULL;
+        vReturnPtr = NULL;
     }
 
     // Convert the wxVariants to VARIANTARGs
@@ -235,7 +220,7 @@ bool wxAutomationObject::Invoke(const wxString& member, int action,
             // Mustn't release the dispatch pointer
             if (vReturn.vt == VT_DISPATCH)
             {
-                vReturn.pdispVal = (IDispatch*) NULL;
+                vReturn.pdispVal = NULL;
             }
             ReleaseVariant(& vReturn);
         }
@@ -455,7 +440,7 @@ WXIDISPATCH* wxAutomationObject::GetDispatchProperty(const wxString& property, i
         }
     }
 
-    return (WXIDISPATCH*) NULL;
+    return NULL;
 }
 
 // Uses DISPATCH_PROPERTYGET
@@ -473,7 +458,7 @@ WXIDISPATCH* wxAutomationObject::GetDispatchProperty(const wxString& property, i
         }
     }
 
-    return (WXIDISPATCH*) NULL;
+    return NULL;
 }
 
 
@@ -513,7 +498,7 @@ bool wxAutomationObject::GetInstance(const wxString& classId) const
     CLSID clsId;
     IUnknown * pUnk = NULL;
 
-    wxBasicString unicodeName(classId.mb_str());
+    wxBasicString unicodeName(classId);
 
     if (FAILED(CLSIDFromProgID((BSTR) unicodeName, &clsId)))
     {
@@ -545,7 +530,7 @@ bool wxAutomationObject::CreateInstance(const wxString& classId) const
 
     CLSID clsId;
 
-    wxBasicString unicodeName(classId.mb_str());
+    wxBasicString unicodeName(classId);
 
     if (FAILED(CLSIDFromProgID((BSTR) unicodeName, &clsId)))
     {
@@ -553,8 +538,13 @@ bool wxAutomationObject::CreateInstance(const wxString& classId) const
         return false;
     }
 
-    // start a new copy of Excel, grab the IDispatch interface
-    if (FAILED(CoCreateInstance(clsId, NULL, CLSCTX_LOCAL_SERVER, IID_IDispatch, (void**)&m_dispatchPtr)))
+    // get the server IDispatch interface
+    //
+    // NB: using CLSCTX_INPROC_HANDLER results in failure when getting
+    //     Automation interface for Microsoft Office applications so don't use
+    //     CLSCTX_ALL which includes it
+    if (FAILED(CoCreateInstance(clsId, NULL, CLSCTX_SERVER, IID_IDispatch,
+                                (void**)&m_dispatchPtr)))
     {
         wxLogWarning(wxT("Cannot start an instance of this class."));
         return false;
@@ -564,7 +554,7 @@ bool wxAutomationObject::CreateInstance(const wxString& classId) const
 }
 
 
-bool wxConvertVariantToOle(const wxVariant& variant, VARIANTARG& oleVariant)
+WXDLLEXPORT bool wxConvertVariantToOle(const wxVariant& variant, VARIANTARG& oleVariant)
 {
     ClearVariant(&oleVariant);
     if (variant.IsNull())
@@ -616,9 +606,11 @@ bool wxConvertVariantToOle(const wxVariant& variant, VARIANTARG& oleVariant)
         wxDateTime date( variant.GetDateTime() );
         oleVariant.vt = VT_DATE;
 
-        if (!OleDateFromTm((WORD)date.GetYear(), date.GetMonth(), date.GetDay(),
-                date.GetHour(), date.GetMinute(), date.GetSecond(), oleVariant.date))
-            return false;
+        long dosDateTime = date.GetAsDOS();
+        short dosDate = short((dosDateTime & 0xFFFF0000) >> 16);
+        short dosTime = short(dosDateTime & 0xFFFF);
+
+        DosDateTimeToVariantTime(dosDate, dosTime, & oleVariant.date);
     }
 #endif
     else if (type == wxT("void*"))
@@ -684,408 +676,151 @@ bool wxConvertVariantToOle(const wxVariant& variant, VARIANTARG& oleVariant)
 #define VT_TYPEMASK 0xfff
 #endif
 
-bool wxConvertOleToVariant(const VARIANTARG& oleVariant, wxVariant& variant)
+WXDLLEXPORT bool
+wxConvertOleToVariant(const VARIANTARG& oleVariant, wxVariant& variant)
 {
-    switch (oleVariant.vt & VT_TYPEMASK)
+    bool ok = true;
+    if ( oleVariant.vt & VT_ARRAY )
     {
-    case VT_BSTR:
-        {
-            wxString str(wxConvertStringFromOle(oleVariant.bstrVal));
-            variant = str;
-            break;
-        }
-    case VT_DATE:
-        {
-#if wxUSE_DATETIME
-            struct tm tmTemp;
-            if (!TmFromOleDate(oleVariant.date, tmTemp))
-                return false;
-
-            wxDateTime date((wxDateTime::wxDateTime_t) tmTemp.tm_yday,
-                            (wxDateTime::Month) tmTemp.tm_mon,
-                            tmTemp.tm_year,
-                            (wxDateTime::wxDateTime_t) tmTemp.tm_hour,
-                            (wxDateTime::wxDateTime_t) tmTemp.tm_min,
-                            (wxDateTime::wxDateTime_t) tmTemp.tm_sec);
 
-            variant = date;
-#endif
+        // Compute the total number of elements in all array dimensions
+        int cElements = 1;
+        for ( int cDims = 0; cDims < oleVariant.parray->cDims; cDims++ )
+            cElements *= oleVariant.parray->rgsabound[cDims].cElements;
 
-            break;
-        }
-    case VT_I4:
-        {
-            variant = (long) oleVariant.lVal;
-            break;
-        }
-    case VT_I2:
-        {
-            variant = (long) oleVariant.iVal;
-            break;
-        }
+        // Get a pointer to the data
+        void* pvdata;
+        HRESULT hr = SafeArrayAccessData(oleVariant.parray, &pvdata);
+        if ( FAILED(hr) )
+            return false;
 
-    case VT_BOOL:
-        {
-#if (defined(_MSC_VER) && (_MSC_VER <= 1000) && !defined(__MWERKS__) ) //GC
-#ifndef HAVE_BOOL // Can't use bool operator if no native bool type
-            variant = (long) (oleVariant.bool != 0);
-#else
-            variant = (bool) (oleVariant.bool != 0);
-#endif
-#else
-#ifndef HAVE_BOOL // Can't use bool operator if no native bool type
-            variant = (long) (oleVariant.boolVal != 0);
-#else
-            variant = (bool) (oleVariant.boolVal != 0);
-#endif
-#endif
-            break;
-        }
-    case VT_R8:
+        switch (oleVariant.vt & VT_TYPEMASK)
         {
-            variant = oleVariant.dblVal;
-            break;
-        }
-    case VT_ARRAY:
-        {
-            variant.ClearList();
-
-            int cDims, cElements, i;
-            VARIANTARG* pvdata;
-
-            // Iterate the dimensions: number of elements is x*y*z
-            for (cDims = 0, cElements = 1;
-                cDims < oleVariant.parray->cDims; cDims ++)
-                    cElements *= oleVariant.parray->rgsabound[cDims].cElements;
+            case VT_VARIANT:
+                {
+                    variant.ClearList();
+                    VARIANTARG *variant_data=(VARIANTARG*)pvdata;
+                    for ( int i = 0; i < cElements; i++ )
+                    {
+                        VARIANTARG& oleElement = variant_data[i];
+                        wxVariant vElement;
+                        if ( !wxConvertOleToVariant(oleElement, vElement) )
+                        {
+                            ok = false;
+                            variant.ClearList();
+                            break;
+                        }
+
+                        variant.Append(vElement);
+                    }
+                }
+                break;
 
-            // Get a pointer to the data
-            HRESULT hr = SafeArrayAccessData(oleVariant.parray, (void HUGEP* FAR*) & pvdata);
-            if (hr != NOERROR)
-                return false;
-            // Iterate the data.
-            for (i = 0; i < cElements; i++)
-            {
-                VARIANTARG& oleElement = pvdata[i];
-                wxVariant vElement;
-                if (!wxConvertOleToVariant(oleElement, vElement))
-                    return false;
+            case VT_BSTR:
+                {
+                    wxArrayString strings;
+                    BSTR *string_val=(BSTR*)pvdata;
+                    for ( int i = 0; i < cElements; ++i )
+                    {
+                        wxString str=wxConvertStringFromOle(*string_val);
+                        strings.Add(str);
+                        ++string_val;
+                    }
+                    variant=strings;
+                }
+                break;
 
-                variant.Append(vElement);
-            }
-            SafeArrayUnaccessData(oleVariant.parray);
-            break;
-        }
-    case VT_DISPATCH:
-        {
-            variant = (void*) oleVariant.pdispVal;
-            break;
-        }
-    case VT_NULL:
-        {
-            variant.MakeNull();
-            break;
-        }
-    case VT_EMPTY:
-        {
-            break;    // Ignore Empty Variant, used only during destruction of objects
-        }
-    default:
-        {
-            wxLogError(wxT("wxAutomationObject::ConvertOleToVariant: Unknown variant value type"));
-            return false;
+            default:
+                wxLogDebug(_T("unhandled VT_ARRAY type %x in wxConvertOleToVariant"),
+                           oleVariant.vt & VT_TYPEMASK);
+                variant = wxVariant();
+                ok = false;
+                break;
         }
-    }
-    return true;
-}
-
-BSTR wxConvertStringToOle(const wxString& str)
-{
-/*
-    unsigned int len = strlen((const char*) str);
-    unsigned short* s = new unsigned short[len*2+2];
-    unsigned int i;
-    memset(s, 0, len*2+2);
-    for (i=0; i < len; i++)
-        s[i*2] = str[i];
-*/
-    wxBasicString bstr(str.mb_str());
-    return bstr.Get();
-}
-
-wxString wxConvertStringFromOle(BSTR bStr)
-{
-#if wxUSE_UNICODE
-    wxString str(bStr);
-#else
-    int len = SysStringLen(bStr) + 1;
-    char    *buf = new char[len];
-    (void)wcstombs( buf, bStr, len);
-    wxString str(buf);
-    delete[] buf;
-#endif
-    return str;
-}
-
-// ----------------------------------------------------------------------------
-// wxBasicString
-// ----------------------------------------------------------------------------
-
-// ctor takes an ANSI string and transforms it to Unicode
-wxBasicString::wxBasicString(const char *sz)
-{
-    Init(sz);
-}
 
-// ctor takes an ANSI or Unicode string and transforms it to Unicode
-wxBasicString::wxBasicString(const wxString& str)
-{
-#if wxUSE_UNICODE
-    m_wzBuf = new OLECHAR[str.Length() + 1];
-    memcpy(m_wzBuf, str.c_str(), str.Length()*2);
-    m_wzBuf[str.Length()] = L'\0';
-#else
-    Init(str.c_str());
-#endif
-}
-
-// Takes an ANSI string and transforms it to Unicode
-void wxBasicString::Init(const char *sz)
-{
-    // get the size of required buffer
-    UINT lenAnsi = strlen(sz);
-#ifdef __MWERKS__
-    UINT lenWide = lenAnsi * 2 ;
-#else
-    UINT lenWide = mbstowcs(NULL, sz, lenAnsi);
-#endif
-
-    if ( lenWide > 0 ) {
-        m_wzBuf = new OLECHAR[lenWide + 1];
-        mbstowcs(m_wzBuf, sz, lenAnsi);
-        m_wzBuf[lenWide] = L'\0';
+        SafeArrayUnaccessData(oleVariant.parray);
     }
-    else {
-        m_wzBuf = NULL;
-    }
-}
-
-// dtor frees memory
-wxBasicString::~wxBasicString()
-{
-  delete [] m_wzBuf;
-}
-
-/////////////////////////////////////////////////////////////////////////////
-// COleDateTime class HELPERS - implementation
-
-BOOL OleDateFromTm(WORD wYear, WORD wMonth, WORD wDay,
-    WORD wHour, WORD wMinute, WORD wSecond, DATE& dtDest)
-{
-    // Validate year and month (ignore day of week and milliseconds)
-    if (wYear > 9999 || wMonth < 1 || wMonth > 12)
-        return FALSE;
-
-    //  Check for leap year and set the number of days in the month
-    BOOL bLeapYear = ((wYear & 3) == 0) &&
-        ((wYear % 100) != 0 || (wYear % 400) == 0);
-
-    int nDaysInMonth =
-        rgMonthDays[wMonth] - rgMonthDays[wMonth-1] +
-        ((bLeapYear && wDay == 29 && wMonth == 2) ? 1 : 0);
-
-    // Finish validating the date
-    if (wDay < 1 || wDay > nDaysInMonth ||
-        wHour > 23 || wMinute > 59 ||
-        wSecond > 59)
+    else if ( oleVariant.vt & VT_BYREF )
     {
-        return FALSE;
-    }
-
-    // Cache the date in days and time in fractional days
-    long nDate;
-    double dblTime;
-
-    //It is a valid date; make Jan 1, 1AD be 1
-    nDate = wYear*365L + wYear/4 - wYear/100 + wYear/400 +
-        rgMonthDays[wMonth-1] + wDay;
-
-    //  If leap year and it's before March, subtract 1:
-    if (wMonth <= 2 && bLeapYear)
-        --nDate;
-
-    //  Offset so that 12/30/1899 is 0
-    nDate -= 693959L;
-
-    dblTime = (((long)wHour * 3600L) +  // hrs in seconds
-        ((long)wMinute * 60L) +  // mins in seconds
-        ((long)wSecond)) / 86400.;
-
-    dtDest = (double) nDate + ((nDate >= 0) ? dblTime : -dblTime);
-
-    return TRUE;
-}
-
-BOOL TmFromOleDate(DATE dtSrc, struct tm& tmDest)
-{
-    // The legal range does not actually span year 0 to 9999.
-    if (dtSrc > MAX_DATE || dtSrc < MIN_DATE) // about year 100 to about 9999
-        return FALSE;
-
-    long nDaysAbsolute;     // Number of days since 1/1/0
-    long nSecsInDay;        // Time in seconds since midnight
-    long nMinutesInDay;     // Minutes in day
-
-    long n400Years;         // Number of 400 year increments since 1/1/0
-    long n400Century;       // Century within 400 year block (0,1,2 or 3)
-    long n4Years;           // Number of 4 year increments since 1/1/0
-    long n4Day;             // Day within 4 year block
-                            //  (0 is 1/1/yr1, 1460 is 12/31/yr4)
-    long n4Yr;              // Year within 4 year block (0,1,2 or 3)
-    BOOL bLeap4 = TRUE;     // TRUE if 4 year block includes leap year
-
-    double dblDate = dtSrc; // tempory serial date
-
-    // Round to the second
-    dblDate += ((dtSrc > 0.0) ? HALF_SECOND : -HALF_SECOND);
-
-    nDaysAbsolute = (long)dblDate + 693959L; // Add days from 1/1/0 to 12/30/1899
-
-    dblDate = fabs(dblDate);
-    nSecsInDay = (long)((dblDate - floor(dblDate)) * 86400.);
-
-    // Calculate the day of week (sun=1, mon=2...)
-    //   -1 because 1/1/0 is Sat.  +1 because we want 1-based
-    tmDest.tm_wday = (int)((nDaysAbsolute - 1) % 7L) + 1;
-
-    // Leap years every 4 yrs except centuries not multiples of 400.
-    n400Years = (long)(nDaysAbsolute / 146097L);
-
-    // Set nDaysAbsolute to day within 400-year block
-    nDaysAbsolute %= 146097L;
-
-    // -1 because first century has extra day
-    n400Century = (long)((nDaysAbsolute - 1) / 36524L);
-
-    // Non-leap century
-    if (n400Century != 0)
-    {
-        // Set nDaysAbsolute to day within century
-        nDaysAbsolute = (nDaysAbsolute - 1) % 36524L;
-
-        // +1 because 1st 4 year increment has 1460 days
-        n4Years = (long)((nDaysAbsolute + 1) / 1461L);
-
-        if (n4Years != 0)
-            n4Day = (long)((nDaysAbsolute + 1) % 1461L);
-        else
+        switch ( oleVariant.vt & VT_TYPEMASK )
         {
-            bLeap4 = FALSE;
-            n4Day = (long)nDaysAbsolute;
+            case VT_VARIANT:
+                {
+                    VARIANTARG& oleReference = *((LPVARIANT)oleVariant.byref);
+                    if (!wxConvertOleToVariant(oleReference,variant))
+                        return false;
+                    break;
+                }
+
+            default:
+                wxLogError(wxT("wxAutomationObject::ConvertOleToVariant: [as yet] unhandled reference %X"),
+                            oleVariant.vt);
+                return false;
         }
     }
-    else
+    else // simply type (not array or reference)
     {
-        // Leap century - not special case!
-        n4Years = (long)(nDaysAbsolute / 1461L);
-        n4Day = (long)(nDaysAbsolute % 1461L);
-    }
+        switch ( oleVariant.vt & VT_TYPEMASK )
+        {
+            case VT_BSTR:
+                {
+                    wxString str(wxConvertStringFromOle(oleVariant.bstrVal));
+                    variant = str;
+                }
+                break;
 
-    if (bLeap4)
-    {
-        // -1 because first year has 366 days
-        n4Yr = (n4Day - 1) / 365;
+            case VT_DATE:
+#if wxUSE_DATETIME
+                {
+                    unsigned short dosDate = 0;
+                    unsigned short dosTime = 0;
+                    VariantTimeToDosDateTime(oleVariant.date, & dosDate, & dosTime);
+
+                    long dosDateTime = (dosDate << 16) | dosTime;
+                    wxDateTime date;
+                    date.SetFromDOS(dosDateTime);
+                    variant = date;
+                }
+#endif // wxUSE_DATETIME
+                break;
 
-        if (n4Yr != 0)
-            n4Day = (n4Day - 1) % 365;
-    }
-    else
-    {
-        n4Yr = n4Day / 365;
-        n4Day %= 365;
-    }
+            case VT_I4:
+                variant = (long) oleVariant.lVal;
+                break;
 
-    // n4Day is now 0-based day of year. Save 1-based day of year, year number
-    tmDest.tm_yday = (int)n4Day + 1;
-    tmDest.tm_year = n400Years * 400 + n400Century * 100 + n4Years * 4 + n4Yr;
+            case VT_I2:
+                variant = (long) oleVariant.iVal;
+                break;
 
-    // Handle leap year: before, on, and after Feb. 29.
-    if (n4Yr == 0 && bLeap4)
-    {
-        // Leap Year
-        if (n4Day == 59)
-        {
-            /* Feb. 29 */
-            tmDest.tm_mon = 2;
-            tmDest.tm_mday = 29;
-            goto DoTime;
-        }
+            case VT_BOOL:
+                variant = oleVariant.boolVal != 0;
+                break;
 
-        // Pretend it's not a leap year for month/day comp.
-        if (n4Day >= 60)
-            --n4Day;
-    }
+            case VT_R8:
+                variant = oleVariant.dblVal;
+                break;
 
-    // Make n4DaY a 1-based day of non-leap year and compute
-    //  month/day for everything but Feb. 29.
-    ++n4Day;
+            case VT_DISPATCH:
+                variant = (void*) oleVariant.pdispVal;
+                break;
 
-    // Month number always >= n/32, so save some loop time */
-    for (tmDest.tm_mon = (n4Day >> 5) + 1;
-        n4Day > rgMonthDays[tmDest.tm_mon]; tmDest.tm_mon++)
-            ;
+            case VT_NULL:
+                variant.MakeNull();
+                break;
 
-    tmDest.tm_mday = (int)(n4Day - rgMonthDays[tmDest.tm_mon-1]);
+            case VT_EMPTY:
+                break;    // Ignore Empty Variant, used only during destruction of objects
 
-DoTime:
-    if (nSecsInDay == 0)
-        tmDest.tm_hour = tmDest.tm_min = tmDest.tm_sec = 0;
-    else
-    {
-        tmDest.tm_sec = (int)nSecsInDay % 60L;
-        nMinutesInDay = nSecsInDay / 60L;
-        tmDest.tm_min = (int)nMinutesInDay % 60;
-        tmDest.tm_hour = (int)nMinutesInDay / 60;
+            default:
+                wxLogError(wxT("wxAutomationObject::ConvertOleToVariant: Unknown variant value type %X -> %X"),
+                           oleVariant.vt,oleVariant.vt&VT_TYPEMASK);
+                return false;
+        }
     }
 
-    return TRUE;
+    return ok;
 }
 
-// this function is not used
-#if 0
-void TmConvertToStandardFormat(struct tm& tmSrc)
-{
-    // Convert afx internal tm to format expected by runtimes (_tcsftime, etc)
-    tmSrc.tm_year -= 1900;  // year is based on 1900
-    tmSrc.tm_mon -= 1;      // month of year is 0-based
-    tmSrc.tm_wday -= 1;     // day of week is 0-based
-    tmSrc.tm_yday -= 1;     // day of year is 0-based
-}
-
-double DoubleFromDate(DATE dt)
-{
-    // No problem if positive
-    if (dt >= 0)
-        return dt;
-
-    // If negative, must convert since negative dates not continuous
-    // (examples: -1.25 to -.75, -1.50 to -.50, -1.75 to -.25)
-    double temp = ceil(dt);
-    return temp - (dt - temp);
-}
-
-DATE DateFromDouble(double dbl)
-{
-    // No problem if positive
-    if (dbl >= 0)
-        return dbl;
-
-    // If negative, must convert since negative dates not continuous
-    // (examples: -.75 to -1.25, -.50 to -1.50, -.25 to -1.75)
-    double temp = floor(dbl); // dbl is now whole part
-    return temp + (temp - dbl);
-}
-#endif // 0
-
 /*
  *  ClearVariant
  *
@@ -1110,7 +845,7 @@ static void ReleaseVariant(VARIANTARG *pvarg)
 {
     VARTYPE vt;
     VARIANTARG _huge *pvargArray;
-    long lLBound, lUBound, l;
+    LONG lLBound, lUBound, l;
 
     vt = (VARTYPE)(pvarg->vt & 0xfff);        // mask off flags
 
@@ -1162,9 +897,12 @@ static void ReleaseVariant(VARIANTARG *pvarg)
                 break;
 
             case VT_I2:
+            case VT_I4:
             case VT_BOOL:
             case VT_R8:
             case VT_ERROR:        // to avoid erroring on an error return from Excel
+            case VT_EMPTY:
+            case VT_DATE:
                 // no work for these types
                 break;
 
@@ -1231,7 +969,7 @@ void ShowException(LPOLESTR szMember, HRESULT hr, EXCEPINFO *pexcep, unsigned in
             break;
 
         default:
-            wsprintf(szBuf, L"%s: Unknown error occured.", szMember);
+            wsprintf(szBuf, L"%s: Unknown error occurred.", szMember);
             break;
     }
 
@@ -1240,5 +978,4 @@ void ShowException(LPOLESTR szMember, HRESULT hr, EXCEPINFO *pexcep, unsigned in
 
 #endif
 
-#endif // wxUSE_OLE && !(defined(__BORLANDC__) && (__BORLANDC__ < 0x520)) && !defined(__CYGWIN10__)
-
+#endif // wxUSE_OLE_AUTOMATION