diff --git a/include/wx/qt/dcscreen.h b/include/wx/qt/dcscreen.h index 371ac49586..c6c19b4848 100644 --- a/include/wx/qt/dcscreen.h +++ b/include/wx/qt/dcscreen.h @@ -21,7 +21,7 @@ protected: virtual void DoGetSize(int *width, int *height) const wxOVERRIDE; virtual bool DoGetPixel(wxCoord x, wxCoord y, wxColour *col) const; - QImage *GetQImage(); + virtual QImage *GetQImage(); wxDECLARE_ABSTRACT_CLASS(wxScreenDCImpl); }; diff --git a/src/qt/bitmap.cpp b/src/qt/bitmap.cpp index 3b86572e7a..bcbde2ea5e 100644 --- a/src/qt/bitmap.cpp +++ b/src/qt/bitmap.cpp @@ -264,7 +264,7 @@ int wxBitmap::GetDepth() const wxImage wxBitmap::ConvertToImage() const { QPixmap pixmap(M_PIXDATA); - if(M_MASK && M_MASK->GetHandle()) + if ( M_MASK && M_MASK->GetHandle() ) pixmap.setMask(*M_MASK->GetHandle()); return ConvertImage(pixmap.toImage()); } diff --git a/src/qt/clipbrd.cpp b/src/qt/clipbrd.cpp index 582c739d97..d692f6627e 100644 --- a/src/qt/clipbrd.cpp +++ b/src/qt/clipbrd.cpp @@ -43,7 +43,7 @@ public: private: void changed( QClipboard::Mode mode) { - if(mode != m_clipboard->Mode() || !m_clipboard->m_sink) + if ( mode != m_clipboard->Mode() || !m_clipboard->m_sink ) return; wxClipboardEvent *event = new wxClipboardEvent(wxEVT_CLIPBOARD_CHANGED); @@ -148,7 +148,7 @@ bool wxClipboard::GetData( wxDataObject& data ) continue; wxTextDataObject *textdata = dynamic_cast(&data); - if(textdata) + if ( textdata ) textdata->SetText(wxQtConvertString(MimeData->text())); else { diff --git a/src/qt/colordlg.cpp b/src/qt/colordlg.cpp index 146c161c40..5781bd61c2 100644 --- a/src/qt/colordlg.cpp +++ b/src/qt/colordlg.cpp @@ -25,7 +25,7 @@ bool wxColourDialog::Create(wxWindow *parent, wxColourData *data ) { m_qtWindow = new wxQtColorDialog( parent, this ); - if(data) + if ( data ) m_data = *data; if ( m_data.GetChooseFull() ) diff --git a/src/qt/colour.cpp b/src/qt/colour.cpp index 7b6fbd3294..c1e1f6c57e 100644 --- a/src/qt/colour.cpp +++ b/src/qt/colour.cpp @@ -46,7 +46,7 @@ int wxColour::GetPixel() const QColor wxColour::GetQColor() const { - if( valid ) + if ( valid ) return QColor(m_red, m_green, m_blue, m_alpha); return QColor(); } diff --git a/src/qt/dataobj.cpp b/src/qt/dataobj.cpp index 72f7d471d8..040ad0a419 100644 --- a/src/qt/dataobj.cpp +++ b/src/qt/dataobj.cpp @@ -62,7 +62,7 @@ wxDataFormat::wxDataFormat(const wxString &id) wxDataFormat::wxDataFormat(const wxChar *id) { - m_MimeType = (wxString)id; + m_MimeType = id; } wxDataFormat::wxDataFormat(const QString &id) @@ -72,7 +72,7 @@ wxDataFormat::wxDataFormat(const QString &id) void wxDataFormat::SetId( const wxChar *id ) { - m_MimeType = (wxString)id; + m_MimeType = id; } void wxDataFormat::SetId( const wxString& id ) diff --git a/src/qt/dc.cpp b/src/qt/dc.cpp index 977142be7c..8c45c841af 100644 --- a/src/qt/dc.cpp +++ b/src/qt/dc.cpp @@ -652,7 +652,7 @@ void wxQtDCImpl::DoDrawBitmap(const wxBitmap &bmp, wxCoord x, wxCoord y, m_qtPainter->setBackground(savedBrush); m_qtPainter->setPen(savedPen); } else { - if(useMask && bmp.GetMask() && bmp.GetMask()->GetHandle()) + if ( useMask && bmp.GetMask() && bmp.GetMask()->GetHandle() ) pix.setMask(*bmp.GetMask()->GetHandle()); m_qtPainter->drawPixmap(x, y, pix); } diff --git a/src/qt/dcmemory.cpp b/src/qt/dcmemory.cpp index 14c68965e8..3f85c3e3db 100644 --- a/src/qt/dcmemory.cpp +++ b/src/qt/dcmemory.cpp @@ -72,7 +72,7 @@ void wxMemoryDCImpl::DoSelect( const wxBitmap& bitmap ) if ( bitmap.IsOk() && !bitmap.GetHandle()->isNull() ) { QPixmap pixmap(*bitmap.GetHandle()); // apply mask before converting to image - if(bitmap.GetMask() && bitmap.GetMask()->GetHandle()) + if ( bitmap.GetMask() && bitmap.GetMask()->GetHandle() ) pixmap.setMask(*bitmap.GetMask()->GetHandle()); // create the intermediate image for the pixmap: m_qtImage = new QImage( pixmap.toImage() ); @@ -87,7 +87,7 @@ void wxMemoryDCImpl::DoSelect( const wxBitmap& bitmap ) wxBitmap wxMemoryDCImpl::DoGetAsBitmap(const wxRect *subrect) const { - if(!subrect) + if ( !subrect ) return m_selected; return m_selected.GetSubBitmap(*subrect); } diff --git a/src/qt/dcscreen.cpp b/src/qt/dcscreen.cpp index dcdbff8261..dd0e6f11b8 100644 --- a/src/qt/dcscreen.cpp +++ b/src/qt/dcscreen.cpp @@ -46,7 +46,7 @@ bool wxScreenDCImpl::DoGetPixel(wxCoord x, wxCoord y, wxColour *col) const // defered allocation for blit QImage *wxScreenDCImpl::GetQImage() { - if(!m_qtImage) + if ( !m_qtImage ) m_qtImage = new QImage(QApplication::primaryScreen()->grabWindow(QApplication::desktop()->winId()).toImage()); return m_qtImage; } diff --git a/src/qt/dialog.cpp b/src/qt/dialog.cpp index 31713d0570..6c99788e2e 100644 --- a/src/qt/dialog.cpp +++ b/src/qt/dialog.cpp @@ -70,7 +70,7 @@ int wxDialog::ShowModal() wxCHECK_MSG( GetHandle() != NULL, -1, "Invalid dialog" ); bool ret = GetDialogHandle()->exec(); - if(GetReturnCode() == 0) + if ( GetReturnCode() == 0 ) return ret ? wxID_OK : wxID_CANCEL; return GetReturnCode(); } diff --git a/src/qt/evtloop.cpp b/src/qt/evtloop.cpp index 97ca3a98ba..a0f1d549f9 100644 --- a/src/qt/evtloop.cpp +++ b/src/qt/evtloop.cpp @@ -133,7 +133,7 @@ int wxQtEventLoopBase::DispatchTimeout(unsigned long timeout) void wxQtEventLoopBase::WakeUp() { QAbstractEventDispatcher *instance = QAbstractEventDispatcher::instance(); - if(instance) + if ( instance ) instance->wakeUp(); } @@ -243,7 +243,7 @@ wxEventLoopSource *wxQtEventLoopBase::AddSourceForFD(int fd, wxEventLoopSourceHa { wxGUIAppTraits *AppTraits = dynamic_cast(wxApp::GetTraitsIfExists()); - if(AppTraits) + if ( AppTraits ) return AppTraits->GetEventLoopSourcesManager()->AddSourceForFD(fd, handler, flags); return NULL; diff --git a/src/qt/filedlg.cpp b/src/qt/filedlg.cpp index 4fb0920ff8..91ebf1777a 100644 --- a/src/qt/filedlg.cpp +++ b/src/qt/filedlg.cpp @@ -122,7 +122,7 @@ wxString wxFileDialog::GetFilename() const { wxArrayString filenames; GetFilenames(filenames); - if(filenames.size() == 0) + if ( filenames.size() == 0 ) return ""; return filenames[0]; diff --git a/src/qt/glcanvas.cpp b/src/qt/glcanvas.cpp index 0106d8620b..482c8f2588 100644 --- a/src/qt/glcanvas.cpp +++ b/src/qt/glcanvas.cpp @@ -484,7 +484,7 @@ bool wxGLCanvas::ConvertWXAttrsToQtGL(const int *wxattrs, QGLFormat &format) } if ( !isBoolAttr ) { - if(!v) + if ( !v ) return false; // zero parameter arg++; } diff --git a/src/qt/listctrl.cpp b/src/qt/listctrl.cpp index 5756084f7d..7c9c9940cd 100644 --- a/src/qt/listctrl.cpp +++ b/src/qt/listctrl.cpp @@ -301,7 +301,7 @@ bool wxListCtrl::GetItem(wxListItem& info) const bool wxListCtrl::SetItem(wxListItem& info) { const long id = info.GetId(); - if(id < 0) + if ( id < 0 ) return false; QTreeWidgetItem *qitem = QtGetItem(id); if ( qitem != NULL ) diff --git a/src/qt/region.cpp b/src/qt/region.cpp index b37b856f14..ad6551d3ea 100644 --- a/src/qt/region.cpp +++ b/src/qt/region.cpp @@ -96,12 +96,12 @@ wxRegion::wxRegion(const wxBitmap& bmp) wxRegion::wxRegion(const wxBitmap& bmp, const wxColour& transp, int tolerance) { - if(!bmp.GetHandle()) { + if ( !bmp.GetHandle() ) { m_refData = new wxRegionRefData(); return; } - if(tolerance == 0) { + if ( tolerance == 0 ) { m_refData = new wxRegionRefData(bmp.GetHandle()->createMaskFromColor(transp.GetQColor())); return; } @@ -111,15 +111,17 @@ wxRegion::wxRegion(const wxBitmap& bmp, const wxColour& transp, int tolerance) QImage img(bmp.GetHandle()->toImage()); int r = transp.Red(), g = transp.Green(), b = transp.Blue(); - for(int y=0; y tolerance || + if ( abs(c.red() - r ) > tolerance || abs(c.green() - g) > tolerance || abs(c.blue() - b) > tolerance) { int ind = y*img.width()+x; raw[ind>>3] |= 1<<(ind&7); - } + } } } diff --git a/src/qt/spinctrl.cpp b/src/qt/spinctrl.cpp index 9cb720d228..ee0b1e670d 100644 --- a/src/qt/spinctrl.cpp +++ b/src/qt/spinctrl.cpp @@ -34,10 +34,10 @@ bool wxSpinCtrlQt< T, Widget >::Create( wxWindow *parent, wxWindowID id, const wxString& value, const wxPoint& pos, const wxSize& size, long style, T min, T max, T initial, T inc, const wxString& name ) { - if(!(style & wxSP_ARROW_KEYS)) + if ( !(style & wxSP_ARROW_KEYS) ) m_qtSpinBox->setButtonSymbols(QAbstractSpinBox::NoButtons); - if(style & wxSP_WRAP) + if ( style & wxSP_WRAP ) m_qtSpinBox->setWrapping(true); m_qtSpinBox->setAccelerated(true); // to match gtk behavior diff --git a/src/qt/statusbar.cpp b/src/qt/statusbar.cpp index bbbacfb9ae..57881f8a6e 100644 --- a/src/qt/statusbar.cpp +++ b/src/qt/statusbar.cpp @@ -50,7 +50,7 @@ bool wxStatusBar::Create(wxWindow *parent, wxWindowID WXUNUSED(winid), m_qtStatusBar = new wxQtStatusBar( parent, this ); m_qtPanes = new QList < QLabel* >; - if(style & wxSTB_SIZEGRIP) + if ( style & wxSTB_SIZEGRIP ) m_qtStatusBar->setSizeGripEnabled(true); PostCreation(); @@ -65,7 +65,7 @@ bool wxStatusBar::GetFieldRect(int i, wxRect& rect) const wxCHECK_MSG( (i >= 0) && ((size_t)i < m_panes.GetCount()), false, "invalid statusbar field index" ); - if(m_qtPanes->count() != m_panes.GetCount()) + if ( m_qtPanes->count() != m_panes.GetCount() ) const_cast(this)->UpdateFields(); rect = wxQtConvertRect((*m_qtPanes)[i]->geometry()); @@ -89,7 +89,7 @@ int wxStatusBar::GetBorderY() const void wxStatusBar::DoUpdateStatusText(int number) { - if(m_qtPanes->count() != m_panes.GetCount()) + if ( m_qtPanes->count() != m_panes.GetCount() ) UpdateFields(); (*m_qtPanes)[number]->setText( wxQtConvertString( m_panes[number].GetText() ) ); diff --git a/src/qt/textctrl.cpp b/src/qt/textctrl.cpp index 2313df841a..c600216903 100644 --- a/src/qt/textctrl.cpp +++ b/src/qt/textctrl.cpp @@ -293,8 +293,8 @@ void wxTextCtrl::GetSelection(long* from, long* to) const QTextCursor cursor = m_qtTextEdit->textCursor(); *from = cursor.selectionStart(); *to = cursor.selectionEnd(); - if(cursor.hasSelection()) - return; + if ( cursor.hasSelection() ) + return; } else // single line { diff --git a/src/qt/toplevel.cpp b/src/qt/toplevel.cpp index 35a982f5ff..e4a6fa54d8 100644 --- a/src/qt/toplevel.cpp +++ b/src/qt/toplevel.cpp @@ -126,7 +126,7 @@ void wxTopLevelWindowQt::SetWindowStyleFlag( long style ) Centre(); } - if(!GetHandle()) + if ( !GetHandle() ) return; Qt::WindowFlags qtFlags = GetHandle()->windowFlags(); diff --git a/src/qt/window.cpp b/src/qt/window.cpp index a261cb9174..b82a4e3054 100644 --- a/src/qt/window.cpp +++ b/src/qt/window.cpp @@ -243,7 +243,7 @@ bool wxWindowQt::Create( wxWindowQt * parent, wxWindowID id, const wxPoint & pos parent->AddChild( this ); wxPoint p; - if(pos != wxDefaultPosition) + if ( pos != wxDefaultPosition ) p = pos; DoMoveWindow( p.x, p.y, size.GetWidth(), size.GetHeight() ); @@ -573,7 +573,7 @@ void wxWindowQt::SetScrollbar( int orientation, int pos, int thumbvisible, int r void wxWindowQt::SetScrollPos( int orientation, int pos, bool WXUNUSED( refresh )) { wxScrollBar *scrollBar = QtGetScrollBar( orientation ); - if(scrollBar) + if ( scrollBar ) scrollBar->SetThumbPosition( pos ); } @@ -741,7 +741,7 @@ void wxWindowQt::SetWindowStyleFlag( long style ) // qtFrame->setFrameShadow( QFrame::Plain ); // } - if(!GetHandle()) + if ( !GetHandle() ) return; Qt::WindowFlags qtFlags = GetHandle()->windowFlags(); @@ -1163,7 +1163,7 @@ bool wxWindowQt::QtHandleKeyEvent ( QWidget *WXUNUSED( handler ), QKeyEvent *eve // qt sends keyup and keydown events for autorepeat, but this is not // normal for wx which only sends repeated keydown events // discard repeated keyup events - if(event->isAutoRepeat() && event->type() == QEvent::KeyRelease) + if ( event->isAutoRepeat() && event->type() == QEvent::KeyRelease ) return true; #if wxUSE_ACCEL