Don’t use void for functions without arguments

include/wx/generic/helpext.h

@@ -44,15 +44,15 @@ public:
 
     virtual bool Initialize(const wxString& dir) wxOVERRIDE;
     virtual bool LoadFile(const wxString& file = wxEmptyString) wxOVERRIDE;
-    virtual bool DisplayContents(void) wxOVERRIDE;
+    virtual bool DisplayContents() wxOVERRIDE;
     virtual bool DisplaySection(int sectionNo) wxOVERRIDE;
     virtual bool DisplaySection(const wxString& section) wxOVERRIDE;
     virtual bool DisplayBlock(long blockNo) wxOVERRIDE;
     virtual bool KeywordSearch(const wxString& k,
                                 wxHelpSearchMode mode = wxHELP_SEARCH_ALL) wxOVERRIDE;
 
-    virtual bool Quit(void) wxOVERRIDE;
+    virtual bool Quit() wxOVERRIDE;
-    virtual void OnQuit(void) wxOVERRIDE;
+    virtual void OnQuit() wxOVERRIDE;
 
     virtual bool DisplayHelp(const wxString &) ;
 
@@ -88,7 +88,7 @@ private:
     bool ParseMapFileLine(const wxString& line);
 
     // Deletes the list and all objects.
-    void DeleteList(void);
+    void DeleteList();
 
 
     // How to call the html viewer.

include/wx/helpbase.h

@@ -50,7 +50,7 @@ public:
     virtual bool LoadFile(const wxString& file = wxEmptyString) = 0;
 
     // Displays the contents
-    virtual bool DisplayContents(void) = 0;
+    virtual bool DisplayContents() = 0;
 
     // Display the given section
     virtual bool DisplaySection(int sectionNo) = 0;

include/wx/image.h

@@ -360,7 +360,7 @@ public:
     // Convert to greyscale image. Uses the luminance component (Y) of the image.
     // The luma value (YUV) is calculated using (R * weight_r) + (G * weight_g) + (B * weight_b), defaults to ITU-T BT.601
     wxImage ConvertToGreyscale(double weight_r, double weight_g, double weight_b) const;
-    wxImage ConvertToGreyscale(void) const;
+    wxImage ConvertToGreyscale() const;
 
     // convert to monochrome image (<r,g,b> will be replaced by white,
     // everything else by black)

include/wx/matrix.h

@@ -34,7 +34,7 @@
 class WXDLLIMPEXP_CORE wxTransformMatrix: public wxObject
 {
 public:
-    wxTransformMatrix(void);
+    wxTransformMatrix();
     wxTransformMatrix(const wxTransformMatrix& mat);
 
     //get the value in the matrix at col,row
@@ -85,18 +85,18 @@ public:
     double operator()(int col, int row) const;
 
     // Invert matrix
-    bool Invert(void);
+    bool Invert();
 
     // Make into identity matrix
-    bool Identity(void);
+    bool Identity();
 
     // Is the matrix the identity matrix?
     // Only returns a flag, which is set whenever an operation
     // is done.
-    inline bool IsIdentity(void) const { return m_isIdentity; }
+    inline bool IsIdentity() const { return m_isIdentity; }
 
     // This does an actual check.
-    inline bool IsIdentity1(void) const ;
+    inline bool IsIdentity1() const ;
 
     //Scale by scale (isotropic scaling i.e. the same in x and y):
     //!ex:
@@ -211,7 +211,7 @@ inline double wxTransformMatrix::TransformY(double y) const
 
 // Is the matrix the identity matrix?
 // Each operation checks whether the result is still the identity matrix and sets a flag.
-inline bool wxTransformMatrix::IsIdentity1(void) const
+inline bool wxTransformMatrix::IsIdentity1() const
 {
     return
     ( wxIsSameDouble(m_matrix[0][0], 1.0) &&

include/wx/msw/joystick.h

@@ -29,54 +29,54 @@ public:
   // Attributes
   ////////////////////////////////////////////////////////////////////////////
 
-  wxPoint GetPosition(void) const;
+  wxPoint GetPosition() const;
   int GetPosition(unsigned axis) const;
   bool GetButtonState(unsigned button) const;
-  int GetZPosition(void) const;
+  int GetZPosition() const;
-  int GetButtonState(void) const;
+  int GetButtonState() const;
-  int GetPOVPosition(void) const;
+  int GetPOVPosition() const;
-  int GetPOVCTSPosition(void) const;
+  int GetPOVCTSPosition() const;
-  int GetRudderPosition(void) const;
+  int GetRudderPosition() const;
-  int GetUPosition(void) const;
+  int GetUPosition() const;
-  int GetVPosition(void) const;
+  int GetVPosition() const;
-  int GetMovementThreshold(void) const;
+  int GetMovementThreshold() const;
   void SetMovementThreshold(int threshold) ;
 
   // Capabilities
   ////////////////////////////////////////////////////////////////////////////
 
-  static int GetNumberJoysticks(void);
+  static int GetNumberJoysticks();
 
-  bool IsOk(void) const; // Checks that the joystick is functioning
+  bool IsOk() const; // Checks that the joystick is functioning
-  int GetManufacturerId(void) const ;
+  int GetManufacturerId() const ;
-  int GetProductId(void) const ;
+  int GetProductId() const ;
-  wxString GetProductName(void) const ;
+  wxString GetProductName() const ;
-  int GetXMin(void) const;
+  int GetXMin() const;
-  int GetYMin(void) const;
+  int GetYMin() const;
-  int GetZMin(void) const;
+  int GetZMin() const;
-  int GetXMax(void) const;
+  int GetXMax() const;
-  int GetYMax(void) const;
+  int GetYMax() const;
-  int GetZMax(void) const;
+  int GetZMax() const;
-  int GetNumberButtons(void) const;
+  int GetNumberButtons() const;
-  int GetNumberAxes(void) const;
+  int GetNumberAxes() const;
-  int GetMaxButtons(void) const;
+  int GetMaxButtons() const;
-  int GetMaxAxes(void) const;
+  int GetMaxAxes() const;
-  int GetPollingMin(void) const;
+  int GetPollingMin() const;
-  int GetPollingMax(void) const;
+  int GetPollingMax() const;
-  int GetRudderMin(void) const;
+  int GetRudderMin() const;
-  int GetRudderMax(void) const;
+  int GetRudderMax() const;
-  int GetUMin(void) const;
+  int GetUMin() const;
-  int GetUMax(void) const;
+  int GetUMax() const;
-  int GetVMin(void) const;
+  int GetVMin() const;
-  int GetVMax(void) const;
+  int GetVMax() const;
 
-  bool HasRudder(void) const;
+  bool HasRudder() const;
-  bool HasZ(void) const;
+  bool HasZ() const;
-  bool HasU(void) const;
+  bool HasU() const;
-  bool HasV(void) const;
+  bool HasV() const;
-  bool HasPOV(void) const;
+  bool HasPOV() const;
-  bool HasPOV4Dir(void) const;
+  bool HasPOV4Dir() const;
-  bool HasPOVCTS(void) const;
+  bool HasPOVCTS() const;
 
   // Operations
   ////////////////////////////////////////////////////////////////////////////
@@ -84,7 +84,7 @@ public:
   // pollingFreq = 0 means that movement events are sent when above the threshold.
   // If pollingFreq > 0, events are received every this many milliseconds.
   bool SetCapture(wxWindow* win, int pollingFreq = 0);
-  bool ReleaseCapture(void);
+  bool ReleaseCapture();
 
 protected:
   int                   m_joystick;

include/wx/msw/taskbar.h

@@ -29,7 +29,7 @@ public:
 
     // Operations
     bool SetIcon(const wxIcon& icon, const wxString& tooltip = wxEmptyString) wxOVERRIDE;
-    bool RemoveIcon(void) wxOVERRIDE;
+    bool RemoveIcon() wxOVERRIDE;
     bool PopupMenu(wxMenu *menu) wxOVERRIDE;
 
     // MSW-specific class methods

include/wx/sckipc.h

@@ -71,7 +71,7 @@ public:
                                 wxIPCFormat format = wxIPC_TEXT) wxOVERRIDE;
     virtual bool StartAdvise(const wxString& item) wxOVERRIDE;
     virtual bool StopAdvise(const wxString& item) wxOVERRIDE;
-    virtual bool Disconnect(void) wxOVERRIDE;
+    virtual bool Disconnect() wxOVERRIDE;
 
     // Will be used in the future to enable the compression but does nothing
     // for now.

interface/wx/object.h

@@ -726,7 +726,7 @@ public:
     @code
     wxIMPLEMENT_ABSTRACT_CLASS(wxCommand, wxObject);
 
-    wxCommand::wxCommand(void)
+    wxCommand::wxCommand()
     {
         ...
     }
@@ -754,7 +754,7 @@ public:
     @code
     wxIMPLEMENT_DYNAMIC_CLASS(wxFrame, wxWindow);
 
-    wxFrame::wxFrame(void)
+    wxFrame::wxFrame()
     {
         ...
     }

interface/wx/thread.h

@@ -1782,7 +1782,7 @@ bool wxIsMainThread();
     Typically, these functions are used like this:
 
     @code
-    void MyThread::Foo(void)
+    void MyThread::Foo()
     {
         // before doing any GUI calls we must ensure that
         // this thread is the only one doing it!

src/common/dcgraph.cpp

@@ -1266,7 +1266,7 @@ bool wxGCDCImpl::DoGetPartialTextExtents(const wxString& text, wxArrayInt& width
     return true;
 }
 
-wxCoord wxGCDCImpl::GetCharWidth(void) const
+wxCoord wxGCDCImpl::GetCharWidth() const
 {
     wxCoord width = 0;
     DoGetTextExtent( wxT("g") , &width , NULL , NULL , NULL , NULL );
@@ -1274,7 +1274,7 @@ wxCoord wxGCDCImpl::GetCharWidth(void) const
     return width;
 }
 
-wxCoord wxGCDCImpl::GetCharHeight(void) const
+wxCoord wxGCDCImpl::GetCharHeight() const
 {
     wxCoord height = 0;
     DoGetTextExtent( wxT("g") , NULL , &height , NULL , NULL , NULL );
@@ -1282,7 +1282,7 @@ wxCoord wxGCDCImpl::GetCharHeight(void) const
     return height;
 }
 
-void wxGCDCImpl::Clear(void)
+void wxGCDCImpl::Clear()
 {
     wxCHECK_RET( IsOk(), wxT("wxGCDC(cg)::Clear - invalid DC") );
 

src/common/ftp.cpp

@@ -732,7 +732,7 @@ public:
     {
     }
 
-    virtual ~wxOutputFTPStream(void)
+    virtual ~wxOutputFTPStream()
     {
         if ( IsOk() )
         {

src/common/http.cpp

@@ -430,7 +430,7 @@ bool wxHTTP::BuildRequest(const wxString& path, const wxString& method)
     return ret_value;
 }
 
-bool wxHTTP::Abort(void)
+bool wxHTTP::Abort()
 {
     return wxSocketClient::Close();
 }
@@ -454,7 +454,7 @@ public:
     }
 
     size_t GetSize() const wxOVERRIDE { return m_httpsize; }
-    virtual ~wxHTTPStream(void) { m_http->Abort(); }
+    virtual ~wxHTTPStream() { m_http->Abort(); }
 
 protected:
     size_t OnSysRead(void *buffer, size_t bufsize) wxOVERRIDE;

src/common/longlong.cpp

@@ -1123,7 +1123,7 @@ wxULongLongWx wxULongLongWx::operator%(const wxULongLongWx& ll) const
 // ----------------------------------------------------------------------------
 
 // temporary - just for testing
-void *wxLongLongWx::asArray(void) const
+void *wxLongLongWx::asArray() const
 {
     static unsigned char temp[8];
 
@@ -1139,7 +1139,7 @@ void *wxLongLongWx::asArray(void) const
     return temp;
 }
 
-void *wxULongLongWx::asArray(void) const
+void *wxULongLongWx::asArray() const
 {
     static unsigned char temp[8];
 

src/common/matrix.cpp

@@ -26,7 +26,7 @@
 
 static const double pi = M_PI;
 
-wxTransformMatrix::wxTransformMatrix(void)
+wxTransformMatrix::wxTransformMatrix()
 {
     m_isIdentity = false;
 
@@ -108,7 +108,7 @@ double wxTransformMatrix::operator()(int col, int row) const
 }
 
 // Invert matrix
-bool wxTransformMatrix::Invert(void)
+bool wxTransformMatrix::Invert()
 {
     double inverseMatrix[3][3];
 
@@ -146,7 +146,7 @@ bool wxTransformMatrix::Invert(void)
 }
 
 // Make into identity matrix
-bool wxTransformMatrix::Identity(void)
+bool wxTransformMatrix::Identity()
 {
     m_matrix[0][0] = m_matrix[1][1] = m_matrix[2][2] = 1.0;
     m_matrix[1][0] = m_matrix[2][0] = m_matrix[0][1] = m_matrix[2][1] = m_matrix[0][2] = m_matrix[1][2] = 0.0;

src/common/memory.cpp

@@ -455,11 +455,11 @@ int wxDebugContext::m_balignmask = (int)((char *)&markerCalc[1] - (char*)&marker
 // Pointer to global function to call at shutdown
 wxShutdownNotifyFunction wxDebugContext::sm_shutdownFn;
 
-wxDebugContext::wxDebugContext(void)
+wxDebugContext::wxDebugContext()
 {
 }
 
-wxDebugContext::~wxDebugContext(void)
+wxDebugContext::~wxDebugContext()
 {
 }
 
@@ -573,14 +573,14 @@ void wxDebugContext::TraverseList (PmSFV func, wxMemStruct *from)
 /*
   Print out the list.
   */
-bool wxDebugContext::PrintList (void)
+bool wxDebugContext::PrintList()
 {
   TraverseList ((PmSFV)&wxMemStruct::PrintNode, (checkPoint ? checkPoint->m_next : NULL));
 
   return true;
 }
 
-bool wxDebugContext::Dump(void)
+bool wxDebugContext::Dump()
 {
   {
     const wxChar* appName = wxT("application");
@@ -720,7 +720,7 @@ bool wxDebugContext::PrintStatistics(bool detailed)
   return true;
 }
 
-bool wxDebugContext::PrintClasses(void)
+bool wxDebugContext::PrintClasses()
 {
   {
     const wxChar* appName = wxT("application");

src/common/time.cpp

@@ -328,7 +328,7 @@ wxLongLong wxGetLocalTimeMillis()
 
 #else // !wxUSE_LONGLONG
 
-double wxGetLocalTimeMillis(void)
+double wxGetLocalTimeMillis()
 {
     return (double(clock()) / double(CLOCKS_PER_SEC)) * 1000.0;
 }

src/common/valgen.cpp

@@ -128,7 +128,7 @@ bool wxGenericValidator::Copy(const wxGenericValidator& val)
 }
 
 // Called to transfer data to the window
-bool wxGenericValidator::TransferToWindow(void)
+bool wxGenericValidator::TransferToWindow()
 {
     if ( !m_validatorWindow )
         return false;
@@ -414,7 +414,7 @@ bool wxGenericValidator::TransferToWindow(void)
 }
 
 // Called to transfer data from the window
-bool wxGenericValidator::TransferFromWindow(void)
+bool wxGenericValidator::TransferFromWindow()
 {
     if ( !m_validatorWindow )
         return false;

src/generic/dcpsg.cpp

@@ -1643,7 +1643,7 @@ void wxPostScriptDCImpl::DoGetSizeMM(int *width, int *height) const
 }
 
 // Resolution in pixels per logical inch
-wxSize wxPostScriptDCImpl::GetPPI(void) const
+wxSize wxPostScriptDCImpl::GetPPI() const
 {
     return wxSize( DPI, DPI );
 }

src/msw/gdiobj.cpp

@@ -29,7 +29,7 @@
 #define M_GDIDATA static_cast<wxGDIRefData*>(m_refData)
 
 /*
-void wxGDIObject::IncrementResourceUsage(void)
+void wxGDIObject::IncrementResourceUsage()
 {
     if ( !M_GDIDATA )
         return;
@@ -38,7 +38,7 @@ void wxGDIObject::IncrementResourceUsage(void)
   M_GDIDATA->m_usageCount ++;
 };
 
-void wxGDIObject::DecrementResourceUsage(void)
+void wxGDIObject::DecrementResourceUsage()
 {
     if ( !M_GDIDATA )
         return;

src/msw/helpwin.cpp

@@ -56,7 +56,7 @@ bool wxWinHelpController::LoadFile(const wxString& file)
     return true;
 }
 
-bool wxWinHelpController::DisplayContents(void)
+bool wxWinHelpController::DisplayContents()
 {
     if (m_helpFile.empty()) return false;
 
@@ -102,7 +102,7 @@ bool wxWinHelpController::KeywordSearch(const wxString& k,
 }
 
 // Can't close the help window explicitly in WinHelp
-bool wxWinHelpController::Quit(void)
+bool wxWinHelpController::Quit()
 {
     return WinHelp(GetSuitableHWND(this), 0, HELP_QUIT, 0) != 0;
 }

src/msw/scrolbar.cpp

@@ -44,7 +44,7 @@ bool wxScrollBar::Create(wxWindow *parent, wxWindowID id,
     return true;
 }
 
-wxScrollBar::~wxScrollBar(void)
+wxScrollBar::~wxScrollBar()
 {
 }
 
@@ -153,7 +153,7 @@ void wxScrollBar::SetThumbPosition(int viewStart)
     ::SetScrollInfo((HWND) GetHWND(), SB_CTL, &info, TRUE);
 }
 
-int wxScrollBar::GetThumbPosition(void) const
+int wxScrollBar::GetThumbPosition() const
 {
     WinStruct<SCROLLINFO> scrollInfo;
     scrollInfo.fMask = SIF_POS;