summaryrefslogtreecommitdiff
path: root/src/wx
diff options
context:
space:
mode:
authorCarl Hetherington <cth@carlh.net>2015-01-28 22:52:13 +0000
committerCarl Hetherington <cth@carlh.net>2015-01-28 22:52:13 +0000
commita112eeb4d052b1212f94e95efd83a215213da691 (patch)
treecc023a901e0cbbd14e79a3ce084691fd9445c03a /src/wx
parenta76f484d1e5c6c44c8d2e4938340e8559b2ef65a (diff)
Small fixes and tidy-ups spotted by cppcheck.
Diffstat (limited to 'src/wx')
-rw-r--r--src/wx/audio_plot.cc2
-rw-r--r--src/wx/content_menu.h2
-rw-r--r--src/wx/content_panel.h2
-rw-r--r--src/wx/content_widget.h2
-rw-r--r--src/wx/dcp_panel.h3
-rw-r--r--src/wx/job_manager_view.cc2
6 files changed, 7 insertions, 6 deletions
diff --git a/src/wx/audio_plot.cc b/src/wx/audio_plot.cc
index 641ddd503..16930ffd8 100644
--- a/src/wx/audio_plot.cc
+++ b/src/wx/audio_plot.cc
@@ -280,7 +280,7 @@ AudioPlot::plot_rms (wxGraphicsPath& path, int channel, Metrics const & metrics)
p += pow (*j, 2);
}
- if (smoothing.size() > 0) {
+ if (!smoothing.empty ()) {
p = sqrt (p / smoothing.size ());
}
diff --git a/src/wx/content_menu.h b/src/wx/content_menu.h
index 77cf29a30..cd51e86f1 100644
--- a/src/wx/content_menu.h
+++ b/src/wx/content_menu.h
@@ -27,7 +27,7 @@
class Film;
-class ContentMenu
+class ContentMenu : public boost::noncopyable
{
public:
ContentMenu (wxWindow* p);
diff --git a/src/wx/content_panel.h b/src/wx/content_panel.h
index 88aac7049..6b94e7d5b 100644
--- a/src/wx/content_panel.h
+++ b/src/wx/content_panel.h
@@ -33,7 +33,7 @@ class FilmEditor;
class ContentSubPanel;
class Film;
-class ContentPanel
+class ContentPanel : public boost::noncopyable
{
public:
ContentPanel (wxNotebook *, boost::shared_ptr<Film>);
diff --git a/src/wx/content_widget.h b/src/wx/content_widget.h
index b4d06286e..6f406dadb 100644
--- a/src/wx/content_widget.h
+++ b/src/wx/content_widget.h
@@ -41,7 +41,7 @@
* @param V Data type of state as used by the view.
*/
template <class S, class T, typename U, typename V>
-class ContentWidget
+class ContentWidget : public boost::noncopyable
{
public:
/** @param parent Parent window.
diff --git a/src/wx/dcp_panel.h b/src/wx/dcp_panel.h
index ced6e6efd..060ec31ff 100644
--- a/src/wx/dcp_panel.h
+++ b/src/wx/dcp_panel.h
@@ -18,6 +18,7 @@
*/
#include <boost/shared_ptr.hpp>
+#include <boost/noncopyable.hpp>
class wxNotebook;
class wxPanel;
@@ -32,7 +33,7 @@ class wxSizer;
class Film;
-class DCPPanel
+class DCPPanel : public boost::noncopyable
{
public:
DCPPanel (wxNotebook *, boost::shared_ptr<Film>);
diff --git a/src/wx/job_manager_view.cc b/src/wx/job_manager_view.cc
index fd3ea6bf7..783535a12 100644
--- a/src/wx/job_manager_view.cc
+++ b/src/wx/job_manager_view.cc
@@ -36,7 +36,7 @@ using std::cout;
using boost::shared_ptr;
using boost::weak_ptr;
-class JobRecord
+class JobRecord : public boost::noncopyable
{
public:
JobRecord (shared_ptr<Job> job, wxScrolledWindow* window, wxPanel* panel, wxFlexGridSizer* table)