X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Flistdata.h;h=9d1a45704fea947c33f1168787c2154b87039da3;hb=HEAD;hp=094b287b703545c294f71a644b33b5be0032af00;hpb=37fc7121c2bde0925e76a9af92c979d7c311315b;p=libs%2Fgltk.git

diff --git a/source/listdata.h b/source/listdata.h
index 094b287..cd4fb9a 100644
--- a/source/listdata.h
+++ b/source/listdata.h
@@ -1,30 +1,37 @@
 #ifndef MSP_GLTK_LISTDATA_H_
 #define MSP_GLTK_LISTDATA_H_
 
+#include <stdexcept>
 #include <string>
 #include <vector>
 #include <sigc++/signal.h>
 #include <msp/strings/lexicalcast.h>
+#include "mspgltk_api.h"
 
 namespace Msp {
 namespace GLtk {
 
-class ListData
+class MSPGLTK_API ListData
 {
 public:
-	sigc::signal<void, unsigned> signal_item_added;
-	sigc::signal<void, unsigned> signal_item_removed;
+	sigc::signal<void, std::size_t> signal_item_added;
+	sigc::signal<void, std::size_t> signal_item_removed;
 	sigc::signal<void> signal_cleared;
-	sigc::signal<void> signal_refresh_strings;
+	sigc::signal<void, std::size_t> signal_refresh_item;
 
 protected:
-	ListData() { }
+	ListData() = default;
 public:
-	virtual ~ListData() { }
+	virtual ~ListData() = default;
 
-	virtual unsigned size() const = 0;
-	virtual std::string get_string(unsigned) const = 0;
-	void refresh_strings() const { signal_refresh_strings.emit(); }
+	virtual std::size_t size() const = 0;
+	virtual std::string get_string(std::size_t) const = 0;
+	void refresh(std::size_t i) const
+	{
+		if(i>=size())
+			throw std::out_of_range("ListData::refresh");
+		signal_refresh_item.emit(i);
+	}
 };
 
 template<typename T>
@@ -33,12 +40,12 @@ class ListDataStore: public ListData
 protected:
 	std::vector<T> items;
 
-	ListDataStore() { }
+	ListDataStore() = default;
 
 public:
 	void append(const T &v) { insert(items.size(), v); }
 
-	void insert(unsigned i, const T & v)
+	void insert(std::size_t i, const T &v)
 	{
 		if(i>items.size())
 			throw std::out_of_range("ListDataStore::insert");
@@ -47,7 +54,7 @@ public:
 		signal_item_added.emit(i);
 	}
 
-	const T &get(unsigned i) const
+	const T &get(std::size_t i) const
 	{
 		if(i>=items.size())
 			throw std::out_of_range("ListDataStore::get");
@@ -55,15 +62,23 @@ public:
 		return items[i];
 	}
 
-	int find(const T &v) const
+	std::size_t find(const T &v) const
 	{
-		for(unsigned i=0; i<items.size(); ++i)
+		for(std::size_t i=0; i<items.size(); ++i)
 			if(items[i]==v)
 				return i;
-		return -1;
+		return std::numeric_limits<std::size_t>::max();
+	}
+
+	using ListData::refresh;
+	void refresh(const T &v) const
+	{
+		std::size_t i = find(v);
+		if(i<items.size())
+			signal_refresh_item.emit(i);
 	}
 
-	void remove(unsigned i)
+	void remove(std::size_t i)
 	{
 		if(i>=items.size())
 			throw std::out_of_range("ListDataStore::remove");
@@ -78,14 +93,14 @@ public:
 		signal_cleared.emit();
 	}
 
-	virtual unsigned size() const { return items.size(); }
+	std::size_t size() const override { return items.size(); }
 };
 
 template<typename T>
 class BasicListData: public ListDataStore<T>
 {
 public:
-	virtual std::string get_string(unsigned i) const
+	virtual std::string get_string(std::size_t i) const
 	{ return lexical_cast<std::string>(this->get(i)); }
 };
 
@@ -96,12 +111,12 @@ public:
 	typedef std::string Func(const T &);
 
 private:
-	Func *func;
+	Func *func = 0;
 
 public:
 	FunctionListData(Func f): func(f) { }
 
-	virtual std::string get_string(unsigned i) const
+	std::string get_string(std::size_t i) const override
 	{ return func(this->get(i)); }
 };