From e54266add728b5cf83a4d1b2136ecfca55fdc146 Mon Sep 17 00:00:00 2001
From: Vadim Zeitlin <vadim@wxwidgets.org>
Date: Thu, 8 May 2008 14:48:59 +0000
Subject: [PATCH] fix some unused variables warnings reported by Borland

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@53493 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
---
 src/common/windowid.cpp  | 2 +-
 src/generic/progdlgg.cpp | 6 ------
 src/html/winpars.cpp     | 2 --
 src/msw/ole/automtn.cpp  | 4 ++--
 4 files changed, 3 insertions(+), 11 deletions(-)

diff --git a/src/common/windowid.cpp b/src/common/windowid.cpp
index aace85973c..10e57fa66c 100644
--- a/src/common/windowid.cpp
+++ b/src/common/windowid.cpp
@@ -118,7 +118,7 @@ void DecIdRefCount(wxWindowID id)
     // so it should never be reserved, but test anyway
     if(gs_autoIdsRefCount[id] == ID_RESERVED)
     {
-        wxASSERT_MSG(false, wxT("reserve id being decreased"));
+        wxFAIL_MSG(wxT("reserve id being decreased"));
         gs_autoIdsRefCount[id] = ID_FREE;
     }
     else
diff --git a/src/generic/progdlgg.cpp b/src/generic/progdlgg.cpp
index 6f4fec1fe8..25a42355ae 100644
--- a/src/generic/progdlgg.cpp
+++ b/src/generic/progdlgg.cpp
@@ -182,9 +182,6 @@ wxProgressDialog::wxProgressDialog(const wxString& title,
     m_break = 0;
     m_ctdelay = 0;
 
-    // if we are going to have at least one label, remember it in this var
-    wxStaticText *label = NULL;
-
     // also count how many labels we really have
     size_t nTimeLabels = 0;
 
@@ -194,7 +191,6 @@ wxProgressDialog::wxProgressDialog(const wxString& title,
     {
         nTimeLabels++;
 
-        label =
         m_elapsed = CreateLabel(_("Elapsed time:"), sizerLabels);
     }
 
@@ -202,7 +198,6 @@ wxProgressDialog::wxProgressDialog(const wxString& title,
     {
         nTimeLabels++;
 
-        label =
         m_estimated = CreateLabel(_("Estimated time:"), sizerLabels);
     }
 
@@ -210,7 +205,6 @@ wxProgressDialog::wxProgressDialog(const wxString& title,
     {
         nTimeLabels++;
 
-        label =
         m_remaining = CreateLabel(_("Remaining time:"), sizerLabels);
     }
     sizerTop->Add(sizerLabels, 0, wxALIGN_CENTER_HORIZONTAL | wxTOP, LAYOUT_MARGIN);
diff --git a/src/html/winpars.cpp b/src/html/winpars.cpp
index f21cf9e8fb..ac32fb27bf 100644
--- a/src/html/winpars.cpp
+++ b/src/html/winpars.cpp
@@ -477,7 +477,6 @@ void wxHtmlWinParser::AddPreBlock(const wxString& text)
 
         const wxString::const_iterator end = text.end();
         wxString::const_iterator copyFrom = text.begin();
-        size_t posFrom = 0;
         size_t pos = 0;
         int posColumn = m_posColumn;
         for ( wxString::const_iterator i = copyFrom; i != end; ++i, ++pos )
@@ -493,7 +492,6 @@ void wxHtmlWinParser::AddPreBlock(const wxString& text)
 
                 posColumn += expandTo;
                 copyFrom = i + 1;
-                posFrom = pos + 1;
             }
             else
             {
diff --git a/src/msw/ole/automtn.cpp b/src/msw/ole/automtn.cpp
index c39b3b2e4c..3ca89f3231 100644
--- a/src/msw/ole/automtn.cpp
+++ b/src/msw/ole/automtn.cpp
@@ -674,6 +674,7 @@ WXDLLEXPORT bool wxConvertVariantToOle(const wxVariant& variant, VARIANTARG& ole
 WXDLLEXPORT bool
 wxConvertOleToVariant(const VARIANTARG& oleVariant, wxVariant& variant)
 {
+    bool ok = true;
     if ( oleVariant.vt & VT_ARRAY )
     {
         variant.ClearList();
@@ -689,7 +690,6 @@ wxConvertOleToVariant(const VARIANTARG& oleVariant, wxVariant& variant)
         if ( FAILED(hr) )
             return false;
 
-        bool ok = true;
         for ( int i = 0; i < cElements; i++ )
         {
             VARIANTARG& oleElement = pvdata[i];
@@ -802,7 +802,7 @@ wxConvertOleToVariant(const VARIANTARG& oleVariant, wxVariant& variant)
         }
     }
 
-    return true;
+    return ok;
 }
 
 /*
-- 
2.45.2