X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2FLColor.h;h=b9595e393f061cf3efe547bf27498bed369689df;hb=6688a55908e69456ea774cea04f4b318e9b2b64a;hp=d07e899dc875fd88e951d6407cfb05652301ede0;hpb=2e7ac7ca77bef2290428603b0a19a9517f675f26;p=lyx.git

diff --git a/src/LColor.h b/src/LColor.h
index d07e899dc8..b9595e393f 100644
--- a/src/LColor.h
+++ b/src/LColor.h
@@ -19,17 +19,15 @@
 #ifndef LCOLOR_H
 #define LCOLOR_H
 
-#include "support/std_string.h"
 #include <boost/scoped_ptr.hpp>
+#include <string>
 
 /**
-  This is a stateless class.
-
-  It has one basic purposes:
-  To serve as a color-namespace container (the Color enum).
-
-
-  */
+ * This is a stateless class.
+ *
+ * It has one basic purposes:
+ * To serve as a color-namespace container (the Color enum).
+ */
 /**
  * \class LColor
  *
@@ -191,49 +189,42 @@ public:
 	///
 	~LColor();
 	///
-	void operator=(LColor const &);
+	LColor & operator=(LColor);
 
-	///
-	void LColor::fill(LColor::color c,
-			string const & lyxname,
-			string const & x11name = string(),
-			string const & latexname = string(),
-			string const & guiname = string());
-
-	/// set the given LyX color to the color defined by the X11 name given
-	void setColor(LColor::color col, string const & x11name);
-	/// set the given LyX color to the color defined by the X11 name given
-	bool setColor(string const & lyxname, string const & x11name);
-
-	/// Get GUI name of color
-	string const getGUIName(LColor::color c) const;
-	///
-	string const getGUIName(string const & s) const;
+	/** set the given LyX color to the color defined by the X11 name given
+	 *  \returns true if successful.
+	 */
+	bool setColor(LColor::color col, std::string const & x11name);
 
-	/// Get X11 name of color
-	string const getX11Name(LColor::color c) const;
-	///
-	string const getX11Name(string const & s) const;
+	/** set the given LyX color to the color defined by the X11
+	 *  name given \returns true if successful. A new color entry
+	 *  is created if the color is unknown
+	 */
+	bool setColor(std::string const & lyxname, std::string const & x11name);
 
-	/// Get LaTeX name of color
-	string const getLaTeXName(LColor::color c) const;
-	///
-	string const getLaTeXName(string const & s) const;
+	/// Get the GUI name of \c color.
+	std::string const getGUIName(LColor::color c) const;
 
-	/// Get LyX name of color
-	string const getLyXName(LColor::color c) const;
-	/// (string-to-string version not needed as it is identity)
+	/// Get the X11 name of \c color.
+	std::string const getX11Name(LColor::color c) const;
 
-	///
-	size_t size() const;
+	/// Get the LaTeX name of \c color.
+	std::string const getLaTeXName(LColor::color c) const;
+
+	/// Get the LyX name of \c color.
+	std::string const getLyXName(LColor::color c) const;
 
-	/// get the color from the GUI name
-	LColor::color getFromGUIName(string const & guiname) const;
-	/// get the color from the LyX name
-	LColor::color getFromLyXName(string const & lyxname) const;
+	/// \returns the LColor::color associated with the GUI name.
+	LColor::color getFromGUIName(std::string const & guiname) const;
+	/// \returns the LColor::color associated with the LyX name.
+	LColor::color getFromLyXName(std::string const & lyxname) const;
+	/// \returns the LColor::color associated with the LaTeX name.
+	LColor::color getFromLaTeXName(std::string const & latexname) const;
 private:
 	///
-	struct Pimpl;
+	void addColor(LColor::color c, std::string const & lyxname) const;
+	///
+	class Pimpl;
 	///
 	boost::scoped_ptr<Pimpl> pimpl_;
 };
@@ -243,15 +234,15 @@ private:
  *  passed as a function argument without having to expose LColor.h.
  */
 class LColor_color {
-        LColor::color val_;
+	LColor::color val_;
 public:
-        /** The default constructor is nasty,
+	/** The default constructor is nasty,
 	 *  but allows us to use LColor_color in STL containers.
 	 */
 	LColor_color() : val_(static_cast<LColor::color>(-1)) {}
 
 	LColor_color(LColor::color val) : val_(val) {}
-        operator LColor::color() const{ return val_; }
+	operator LColor::color() const{ return val_; }
 };