]> git.saurik.com Git - wxWidgets.git/commitdiff
Warning fixes found under hardest mode of OpenWatcom. Seems clean in Borland, MinGW...
authorWłodzimierz Skiba <abx@abx.art.pl>
Mon, 11 Oct 2004 16:30:43 +0000 (16:30 +0000)
committerWłodzimierz Skiba <abx@abx.art.pl>
Mon, 11 Oct 2004 16:30:43 +0000 (16:30 +0000)
git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@29784 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

19 files changed:
include/wx/confbase.h
include/wx/datetime.h
include/wx/html/helpdata.h
include/wx/msw/regconf.h
include/wx/textbuf.h
src/common/config.cpp
src/common/datetime.cpp
src/common/sizer.cpp
src/common/textbuf.cpp
src/generic/choicbkg.cpp
src/generic/grid.cpp
src/generic/listbkg.cpp
src/generic/sashwin.cpp
src/html/helpdata.cpp
src/html/helpfrm.cpp
src/html/htmltag.cpp
src/msw/listbox.cpp
src/msw/regconf.cpp
src/xml/xml.cpp

index 0e23972e4cc6c21e844082a07fff316ffb586526..15c49198960277310e730b1b171557bc56f7da81 100644 (file)
@@ -117,7 +117,7 @@ public:
                long style = 0);
 
     // empty but ensures that dtor of all derived classes is virtual
-  virtual ~wxConfigBase();
+  virtual ~wxConfigBase(){};
 
   // path management
     // set current path: if the first character is '/', it's the absolute path,
index 2ed9826fea509faf0f330f252786840759f44573..6303d40250c7ce2a13a65f09227274744238737b 100644 (file)
@@ -1327,7 +1327,7 @@ public:
     static void AddAuthority(wxDateTimeHolidayAuthority *auth);
 
     // the base class must have a virtual dtor
-    virtual ~wxDateTimeHolidayAuthority();
+    virtual ~wxDateTimeHolidayAuthority(){};
 
 protected:
     // this function is called to determine whether a given day is a holiday
index ff2feb6ab542b9451957ae15da05a01aab839d93..a59c6f76d633f560711c4dc3f1fe744799ba6d53 100644 (file)
@@ -84,7 +84,7 @@ struct WXDLLIMPEXP_HTML wxHtmlHelpDataItem
 {
     wxHtmlHelpDataItem() : level(0), parent(NULL), id(wxID_ANY), book(NULL) {}
 
-    short int level;
+    int level;
     wxHtmlHelpDataItem *parent;
     int id;
     wxString name;
@@ -110,7 +110,7 @@ struct wxHtmlContentsItem
     wxHtmlContentsItem& operator=(const wxHtmlContentsItem& d);
     ~wxHtmlContentsItem();
 
-    short int m_Level;
+    int m_Level;
     int m_ID;
     wxChar *m_Name;
     wxChar *m_Page;
index 166d4ac623f13307556c3c5853c8f8efea3ed191..d0d0172bd58d499f4f0620b392910c033c3a9163 100644 (file)
@@ -39,7 +39,7 @@ public:
               long style = wxCONFIG_USE_GLOBAL_FILE);
 
     // dtor will save unsaved data
-  virtual ~wxRegConfig();
+  virtual ~wxRegConfig(){};
 
   // implement inherited pure virtual functions
   // ------------------------------------------
index 8da8b294548f7f962d27bd1031a3ae6176ebbdd5..ac8b14a3b3dc1c1de344206f739e63e25a79385e 100644 (file)
@@ -165,7 +165,7 @@ public:
                wxMBConv& conv = wxConvUTF8);
 
     // dtor
-    virtual ~wxTextBuffer();
+    virtual ~wxTextBuffer(){};
 
 protected:
     // ctors
index 80b43e4b9a5c5a38cfdf2278c727aabc370193c2..00f2daa6b70616c7e298da6b8ee559921de15b85 100644 (file)
@@ -73,10 +73,6 @@ wxConfigBase::wxConfigBase(const wxString& appName,
     m_bRecordDefaults = false;
 }
 
-wxConfigBase::~wxConfigBase()
-{
-}
-
 wxConfigBase *wxConfigBase::Set(wxConfigBase *pConfig)
 {
   wxConfigBase *pOld = ms_pConfig;
index e2194fb5743d971f078198279bb2a0eb61da6922..1d85ebbbd81c3946803d88f400a6c7d950f6c97e 100644 (file)
@@ -1974,7 +1974,7 @@ wxDateTime& wxDateTime::SetToYearDay(wxDateTime::wxDateTime_t yday)
         // yday lies in December then
         if ( (mon == Dec) || (yday <= gs_cumulatedDays[isLeap][mon + 1]) )
         {
-            Set(yday - gs_cumulatedDays[isLeap][mon], mon, year);
+            Set((wxDateTime::wxDateTime_t)(yday - gs_cumulatedDays[isLeap][mon]), mon, year);
 
             break;
         }
@@ -2565,7 +2565,7 @@ const wxChar *wxDateTime::ParseRfc822Date(const wxChar* date)
         return (wxChar *)NULL;
     }
 
-    wxDateTime_t hour = *p++ - _T('0');
+    wxDateTime_t hour = (wxDateTime_t)(*p++ - _T('0'));
 
     if ( !wxIsdigit(*p) )
     {
@@ -2573,7 +2573,7 @@ const wxChar *wxDateTime::ParseRfc822Date(const wxChar* date)
     }
 
     hour *= 10;
-    hour += *p++ - _T('0');
+    hour = (wxDateTime_t)(hour + (*p++ - _T('0')));
 
     if ( *p++ != _T(':') )
     {
@@ -2585,7 +2585,7 @@ const wxChar *wxDateTime::ParseRfc822Date(const wxChar* date)
         return (wxChar *)NULL;
     }
 
-    wxDateTime_t min = *p++ - _T('0');
+    wxDateTime_t min = (wxDateTime_t)(*p++ - _T('0'));
 
     if ( !wxIsdigit(*p) )
     {
@@ -2593,7 +2593,7 @@ const wxChar *wxDateTime::ParseRfc822Date(const wxChar* date)
     }
 
     min *= 10;
-    min += *p++ - _T('0');
+    min = (wxDateTime_t)(min + *p++ - _T('0'));
 
     wxDateTime_t sec = 0;
     if ( *p++ == _T(':') )
@@ -2603,7 +2603,7 @@ const wxChar *wxDateTime::ParseRfc822Date(const wxChar* date)
             return (wxChar *)NULL;
         }
 
-        sec = *p++ - _T('0');
+        sec = (wxDateTime_t)(*p++ - _T('0'));
 
         if ( !wxIsdigit(*p) )
         {
@@ -2611,7 +2611,7 @@ const wxChar *wxDateTime::ParseRfc822Date(const wxChar* date)
         }
 
         sec *= 10;
-        sec += *p++ - _T('0');
+        sec = (wxDateTime_t)(sec + *p++ - _T('0'));
     }
 
     if ( *p++ != _T(' ') )
@@ -3463,9 +3463,11 @@ const wxChar *wxDateTime::ParseDate(const wxChar *date)
                 }
                 else // may be either day or year
                 {
-                    wxDateTime_t maxDays = haveMon
+                    wxDateTime_t maxDays = (wxDateTime_t)(
+                        haveMon
                         ? GetNumOfDaysInMonth(haveYear ? year : Inv_Year, mon)
-                        : 31;
+                        : 31
+                    );
 
                     // can it be day?
                     if ( (val == 0) || (val > (unsigned long)maxDays) )
@@ -3519,7 +3521,7 @@ const wxChar *wxDateTime::ParseDate(const wxChar *date)
                     {
                         // no need to check in month range as always < 12, but
                         // the days are counted from 1 unlike the months
-                        day = (wxDateTime_t)mon + 1;
+                        day = (wxDateTime_t)(mon + 1);
                         haveDay = true;
                     }
                     else
@@ -3641,7 +3643,7 @@ const wxChar *wxDateTime::ParseDate(const wxChar *date)
                 // we're in the current year then
                 if ( (year > 0) && (year <= (int)GetNumOfDaysInMonth(Inv_Year, mon)) )
                 {
-                    day = year;
+                    day = (wxDateTime_t)year;
 
                     haveMon = true;
                     haveYear = false;
@@ -4028,11 +4030,6 @@ void wxDateTimeHolidayAuthority::AddAuthority(wxDateTimeHolidayAuthority *auth)
     ms_authorities.push_back(auth);
 }
 
-wxDateTimeHolidayAuthority::~wxDateTimeHolidayAuthority()
-{
-    // nothing to do here
-}
-
 // ----------------------------------------------------------------------------
 // wxDateTimeWorkDays
 // ----------------------------------------------------------------------------
index 475867e17b734abb37d4d022441df79b9869d0cc..9f73688a00f897e2287d6c6343207ca280f67e48 100644 (file)
@@ -1351,7 +1351,7 @@ void wxBoxSizer::RecalcSizes()
                 }
 
                 wxPoint child_pos( pt );
-                wxSize  child_size( wxSize( size.x, height) );
+                wxSize  child_size( size.x, height );
 
                 if (item->GetFlag() & (wxEXPAND | wxSHAPED))
                     child_size.x = m_size.x;
@@ -1377,7 +1377,7 @@ void wxBoxSizer::RecalcSizes()
                 }
 
                 wxPoint child_pos( pt );
-                wxSize  child_size( wxSize(width, size.y) );
+                wxSize  child_size( width, size.y );
 
                 if (item->GetFlag() & (wxEXPAND | wxSHAPED))
                     child_size.y = m_size.y;
index 6158973ff1e470075a9ce36c6e913590359f34b1..42268b62ed4b0a171248017722f3feb12248a190 100644 (file)
@@ -146,10 +146,6 @@ wxTextBuffer::wxTextBuffer(const wxString& strBufferName)
     m_isOpened = false;
 }
 
-wxTextBuffer::~wxTextBuffer()
-{
-}
-
 // ----------------------------------------------------------------------------
 // buffer operations
 // ----------------------------------------------------------------------------
index 9c3062d7421fa13535326004ee212cbe66562810..15390adec2d934564303f01f12244ac58f8c3a46 100644 (file)
@@ -144,7 +144,8 @@ wxRect wxChoicebook::GetPageRect() const
 {
     const wxSize sizeChoice = m_choice->GetSize();
 
-    wxRect rectPage(wxPoint(0, 0), GetClientSize());
+    wxPoint pt(0, 0);
+    wxRect rectPage(pt, GetClientSize());
     switch ( GetWindowStyle() & wxCHB_ALIGN_MASK )
     {
         default:
index ac9b77b5e049049e178d3b600ba705ebec5368b1..64a0c40a1dfde0c2ed6f07c93f5e227c8c010217 100644 (file)
@@ -1072,7 +1072,8 @@ void wxGridCellFloatEditor::StartingKey(wxKeyEvent& event)
     char tmpbuf[2];
     tmpbuf[0] = (char) keycode;
     tmpbuf[1] = '\0';
-    bool is_decimal_point = ( wxString(tmpbuf, *wxConvCurrent) ==
+    wxString strbuf(tmpbuf, *wxConvCurrent);
+    bool is_decimal_point = ( strbuf ==
       wxLocale::GetInfo(wxLOCALE_DECIMAL_POINT, wxLOCALE_CAT_NUMBER) );
         if ( wxIsdigit(keycode) || keycode == '+' || keycode == '-'
             || is_decimal_point
@@ -8909,7 +8910,7 @@ void wxGrid::SetDefaultEditor(wxGridCellEditor *editor)
 {
     RegisterDataType(wxGRID_VALUE_STRING,
                      GetDefaultRendererForType(wxGRID_VALUE_STRING),
-                     editor);                     
+                     editor);
 }
 
 // ----------------------------------------------------------------------------
@@ -9891,7 +9892,7 @@ wxSize wxGrid::DoGetBestSize() const
 
     if (!width) width=100;
     if (!height) height=80;
-    
+
     // Round up to a multiple the scroll rate NOTE: this still doesn't get rid
     // of the scrollbars, is there any magic incantaion for that?
     int xpu, ypu;
@@ -9900,16 +9901,16 @@ wxSize wxGrid::DoGetBestSize() const
         width  += 1 + xpu - (width  % xpu);
     if (ypu)
         height += 1 + ypu - (height % ypu);
-    
+
     // limit to 1/4 of the screen size
     int maxwidth, maxheight;
     wxDisplaySize( & maxwidth, & maxheight );
     maxwidth /= 2;
-    maxheight /= 2;    
+    maxheight /= 2;
     if ( width > maxwidth ) width = maxwidth;
     if ( height > maxheight ) height = maxheight;
 
-    
+
     wxSize best(width, height);
     // NOTE: This size should be cached, but first we need to add calls to
     // InvalidateBestSize everywhere that could change the results of this
index 34d726efe714225d9d7a4c1a1cacb69f35731267..3b10666dcb6a19e797a3796450e03b817ed09067 100644 (file)
@@ -172,7 +172,8 @@ wxRect wxListbook::GetPageRect() const
 {
     const wxSize sizeList = m_list->GetSize();
 
-    wxRect rectPage(wxPoint(0, 0), GetClientSize());
+    wxPoint pt(0, 0);
+    wxRect rectPage(pt, GetClientSize());
     switch ( GetWindowStyle() & wxLB_ALIGN_MASK )
     {
         default:
@@ -401,7 +402,7 @@ wxListbook::InsertPage(size_t n,
     // index of the selected page
     if ( int(n) <= m_selection )
     {
-        // one extra page added 
+        // one extra page added
         m_selection++;
         m_list->Select(m_selection);
         m_list->Focus(m_selection);
index ad855afdbc82af2428752c880ed4ebd195a9661b..e6469b11ee1988ed23ec820bc0213da0f00e9a56 100644 (file)
@@ -483,8 +483,10 @@ void wxSashWindow::DrawSash(wxSashEdgePosition edge, wxDC& dc)
     wxPen darkShadowPen(m_darkShadowColour, 1, wxSOLID);
     wxPen lightShadowPen(m_lightShadowColour, 1, wxSOLID);
     wxPen hilightPen(m_hilightColour, 1, wxSOLID);
-    wxPen blackPen(wxColour(0, 0, 0), 1, wxSOLID);
-    wxPen whitePen(wxColour(255, 255, 255), 1, wxSOLID);
+    wxColour blackClr(0, 0, 0);
+    wxColour whiteClr(255, 255, 255);
+    wxPen blackPen(blackClr, 1, wxSOLID);
+    wxPen whitePen(whiteClr, 1, wxSOLID);
 
     if ( edge == wxSASH_LEFT || edge == wxSASH_RIGHT )
     {
index 327cc5fa4e9d6ef3627f87a4cd91a75231eea133..9d5e70638d25f088ad42b2cc55609d58069969f6 100644 (file)
@@ -82,7 +82,7 @@ wxHtmlHelpIndexCompareFunc(wxHtmlHelpDataItem **a, wxHtmlHelpDataItem **b)
         return -1;
     if (ib == NULL)
         return 1;
-    
+
     if (ia->parent == ib->parent)
     {
         return ia->name.CmpNoCase(ib->name);
@@ -686,7 +686,7 @@ bool wxHtmlHelpData::AddBook(const wxString& book)
         lineptr = ReadLine(lineptr, linebuf, 300);
 
         for (wxChar *ch = linebuf; *ch != wxT('\0') && *ch != wxT('='); ch++)
-           *ch = tolower(*ch);
+           *ch = (wxChar)wxTolower(*ch);
 
         if (wxStrstr(linebuf, _T("title=")) == linebuf)
             title = linebuf + wxStrlen(_T("title="));
index bf3223b784e74f65f3b47c57b57789f4878148ff..5858f92b082f7ac12e54e26b4eaf95206fccbd77 100644 (file)
@@ -1674,7 +1674,7 @@ void wxHtmlHelpFrame::OnIndexFind(wxCommandEvent& event)
                 // other items, show them as well, because they are refinements
                 // of the displayed index entry (i.e. it is implicitly contained
                 // in them: "foo" with parent "bar" reads as "bar, foo"):
-                short int level = index[i].items[0]->level;
+                int level = index[i].items[0]->level;
                 i++;
                 while (i < cnt && index[i].items[0]->level > level)
                 {
index a8e30676ec9ecd9cb9c045d2e94e09d76c085d56..1b9b7bd784006de5e7bc9337c34b547afdd5af39 100644 (file)
@@ -90,7 +90,7 @@ wxHtmlTagsCache::wxHtmlTagsCache(const wxString& source)
                   src[pos] != wxT('>') && !wxIsspace(src[pos]);
                   i++, pos++ )
             {
-                tagBuffer[i] = wxToupper(src[pos]);
+                tagBuffer[i] = (wxChar)wxToupper(src[pos]);
             }
             tagBuffer[i] = _T('\0');
 
index df49c91cc6fb93438145134e397dea4baedf70e0..1fc6d08142d39e81e4752ae2159ffa2ad8c348ea 100644 (file)
@@ -513,7 +513,9 @@ wxString wxListBox::GetString(int N) const
 
     // +1 for terminating NUL
     wxString result;
-    ListBox_GetText(GetHwnd(), N, wxStringBuffer(result, len + 1));
+    wxChar* buffer = result.GetWriteBuf(len + 1);
+    ListBox_GetText(GetHwnd(), N, buffer);
+    result.UngetWriteBuf();
 
     return result;
 }
@@ -805,8 +807,9 @@ bool wxListBox::MSWOnDraw(WXDRAWITEMSTRUCT *item)
     wxListBoxItem *pItem = (wxListBoxItem *)data;
 
     wxDCTemp dc((WXHDC)pStruct->hDC);
-    wxRect rect(wxPoint(pStruct->rcItem.left, pStruct->rcItem.top),
-                wxPoint(pStruct->rcItem.right, pStruct->rcItem.bottom));
+    wxPoint pt1(pStruct->rcItem.left, pStruct->rcItem.top);
+    wxPoint pt2(pStruct->rcItem.right, pStruct->rcItem.bottom);
+    wxRect rect(pt1, pt2);
 
     return pItem->OnDrawItem(dc, rect,
                              (wxOwnerDrawn::wxODAction)pStruct->itemAction,
index 9647ae12176b74f0d7ff35ab82fb7a38712af6e3..13f59406cdc2c24412043b283ecee7eb678dd270 100644 (file)
@@ -151,11 +151,6 @@ wxRegConfig::wxRegConfig(const wxString& appName, const wxString& vendorName,
   }
 }
 
-wxRegConfig::~wxRegConfig()
-{
-  // nothing to do - key will be closed in their dtors
-}
-
 // ----------------------------------------------------------------------------
 // path management
 // ----------------------------------------------------------------------------
index 80c76829fceb212a123e9cbd1aeffd425d8a457b..f55ad6f7a1ead4267637837e5c6d404e57556588 100644 (file)
@@ -482,7 +482,8 @@ static int UnknownEncodingHnd(void * WXUNUSED(encodingHandlerData),
     // We must build conversion table for expat. The easiest way to do so
     // is to let wxCSConv convert as string containing all characters to
     // wide character representation:
-    wxCSConv conv(wxString(name, wxConvLibc));
+    wxString str(name, wxConvLibc);
+    wxCSConv conv(str);
     char mbBuf[2];
     wchar_t wcBuf[10];
     size_t i;
@@ -499,7 +500,7 @@ static int UnknownEncodingHnd(void * WXUNUSED(encodingHandlerData),
         }
         info->map[i+1] = (int)wcBuf[0];
     }
-    
+
     info->data = NULL;
     info->convert = NULL;
     info->release = NULL;