From 8703bc01c28e0397764cd28b915caf17a28020ad Mon Sep 17 00:00:00 2001
From: Dimitri Schoolwerth <dimitri.schoolwerth@gmail.com>
Date: Mon, 10 Nov 2003 15:17:46 +0000
Subject: [PATCH] Added wxUnusedVar for Borland and fixed 2 release mode
 warnings for Borland (patch #835707); Slight code cleanup.

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@24508 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
---
 include/wx/defs.h     |  4 ++++
 src/generic/vlbox.cpp |  1 +
 src/html/htmltag.cpp  | 27 ++++++++++++++-------------
 3 files changed, 19 insertions(+), 13 deletions(-)

diff --git a/include/wx/defs.h b/include/wx/defs.h
index 4929dec125..d4c2f50058 100644
--- a/include/wx/defs.h
+++ b/include/wx/defs.h
@@ -457,8 +457,12 @@ typedef int wxWindowID;
 
 // sometimes the value of a variable is *really* not used, to suppress  the
 // resulting warning you may pass it to this function
+#ifdef __BORLANDC__
+#define wxUnusedVar(identifier) identifier
+#else
 template <class T>
 inline void wxUnusedVar(const T& WXUNUSED(t)) { }
+#endif
 
 // ----------------------------------------------------------------------------
 // compiler specific settings
diff --git a/src/generic/vlbox.cpp b/src/generic/vlbox.cpp
index 0b978eaaef..50a484eec6 100644
--- a/src/generic/vlbox.cpp
+++ b/src/generic/vlbox.cpp
@@ -540,6 +540,7 @@ void wxVListBox::OnKeyDown(wxKeyEvent& event)
         default:
             event.Skip();
             current = 0; // just to silent the stupid compiler warnings
+            wxUnusedVar(current);
             return;
     }
 
diff --git a/src/html/htmltag.cpp b/src/html/htmltag.cpp
index 16579f607f..361aaa971a 100644
--- a/src/html/htmltag.cpp
+++ b/src/html/htmltag.cpp
@@ -148,8 +148,9 @@ wxHtmlTagsCache::wxHtmlTagsCache(const wxString& source)
 
                         // found a match
                         if (match_pos == tag_len) {
-                            pos = pos - tag_len - 3;
-                            stpos = pos;
+                            // These 2 lines commented:
+                            //pos = pos - tag_len - 3; pos is not used outside of while loop
+                            //stpos = pos; Value is local to while loop, no need to set it
                             break;
                         }
                         else {
@@ -362,12 +363,12 @@ wxHtmlTag::~wxHtmlTag()
 
 bool wxHtmlTag::HasParam(const wxString& par) const
 {
-    return (m_ParamNames.Index(par, FALSE) != wxNOT_FOUND);
+    return (m_ParamNames.Index(par, false) != wxNOT_FOUND);
 }
 
 wxString wxHtmlTag::GetParam(const wxString& par, bool with_commas) const
 {
-    int index = m_ParamNames.Index(par, FALSE);
+    int index = m_ParamNames.Index(par, false);
     if (index == wxNOT_FOUND)
         return wxEmptyString;
     if (with_commas)
@@ -393,23 +394,23 @@ bool wxHtmlTag::GetParamAsColour(const wxString& par, wxColour *clr) const
 {
     wxString str = GetParam(par);
 
-    if (str.IsEmpty()) return FALSE;
+    if (str.IsEmpty()) return false;
     if (str.GetChar(0) == wxT('#'))
     {
         unsigned long tmp;
         if (ScanParam(par, wxT("#%lX"), &tmp) != 1)
-            return FALSE;
+            return false;
         *clr = wxColour((unsigned char)((tmp & 0xFF0000) >> 16),
-					    (unsigned char)((tmp & 0x00FF00) >> 8),
-					    (unsigned char)(tmp & 0x0000FF));
-        return TRUE;
+                        (unsigned char)((tmp & 0x00FF00) >> 8),
+                        (unsigned char)(tmp & 0x0000FF));
+        return true;
     }
     else
     {
         // Handle colours defined in HTML 4.0:
         #define HTML_COLOUR(name,r,g,b)                 \
-            if (str.IsSameAs(wxT(name), FALSE))         \
-                { *clr = wxColour(r,g,b); return TRUE; }
+            if (str.IsSameAs(wxT(name), false))         \
+                { *clr = wxColour(r,g,b); return true; }
         HTML_COLOUR("black",   0x00,0x00,0x00)
         HTML_COLOUR("silver",  0xC0,0xC0,0xC0)
         HTML_COLOUR("gray",    0x80,0x80,0x80)
@@ -429,12 +430,12 @@ bool wxHtmlTag::GetParamAsColour(const wxString& par, wxColour *clr) const
         #undef HTML_COLOUR
     }
 
-    return FALSE;
+    return false;
 }
 
 bool wxHtmlTag::GetParamAsInt(const wxString& par, int *clr) const
 {
-    if (!HasParam(par)) return FALSE;
+    if (!HasParam(par)) return false;
     long i;
     bool succ = GetParam(par).ToLong(&i);
     *clr = (int)i;