Real fix from Bernhard Roider

[lyx.git] / src / Spacing.C

diff --git a/src/Spacing.C b/src/Spacing.C
index 832fe8f1e3d1897884a731aadfdb63e5e21ba08d..35eecde79e0d2b268d4e26fdaa1ea6cb8f9ecb56 100644 (file)
--- a/src/Spacing.C
+++ b/src/Spacing.C
@@ -1,84 +1,95 @@
-/* This file is part of
- * ====================================================== 
- * 
- *           LyX, The Document Processor
- *        
- *           Copyright 1995 Matthias Ettrich
- *           Copyright 1995-2000 The LyX Team.
+/**
+ * \file Spacing.C
+ * This file is part of LyX, the document processor.
+ * Licence details can be found in the file COPYING.
  *
- * ====================================================== */
+ * \author Lars Gullik Bjønnes
+ * \author Jean-Marc Lasgouttes
+ *
+ * Full author contact details are available in file CREDITS.
+ */
 
 #include <config.h>
 
-#ifdef __GNUG__
-#pragma implementation
-#endif
-
-#include "Lsstream.h"
 #include "Spacing.h"
-#include "LString.h"
+#include "support/lstrings.h"
+#include "support/convert.h"
+
+#include <sstream>
+#include <string>
+
+
+namespace lyx {
 
-using std::ios;
-using std::ostream;
 
-/// how can I put this inside of Spacing (class)
-namespace {
+using std::ostream;
+using std::ostringstream;
+using std::string;
 
-char const * const spacing_string[] = {"single", "onehalf", "double", "other"};
 
-} // namespace anon
+string const Spacing::spacing_string[]
+       = {"single", "onehalf", "double", "other"};
 
 
-float Spacing::getValue() const 
+string const Spacing::getValueAsString() const
 {
        switch (space) {
        case Default: // nothing special should happen with this...
-       case Single: return 1.0;
-       case Onehalf: return 1.25;
-       case Double: return 1.667;
+       case Single: return "1.0";
+       case Onehalf: return "1.25";
+       case Double: return "1.667";
        case Other: return value;
        }
-       return 1.0;
+       return "1.0";
 }
 
 
-void Spacing::set(Spacing::Space sp, float val)
+double Spacing::getValue() const
 {
-       space = sp;
-       if (sp == Other) {
-               switch (int(val * 1000 + 0.5)) {
-               case 1000: space = Single; break;
-               case 1250: space = Onehalf; break;
-               case 1667: space = Double; break;
-               default: value = val; break;
-               }
-       }
+       return convert<double>(getValueAsString());
+}
+
+
+void Spacing::set(Spacing::Space sp, double val)
+{
+       set(sp, convert<string>(val));
 }
 
 
 void Spacing::set(Spacing::Space sp, string const & val)
 {
-       float fval;
-       istringstream istr(val.c_str());
-       istr >> fval;
-       set(sp, fval);
+       space = sp;
+       if (sp == Other) {
+               switch (int(convert<double>(val) * 1000 + 0.5)) {
+               case 1000:
+                       space = Single;
+                       break;
+               case 1250:
+                       space = Onehalf;
+                       break;
+               case 1667:
+                       space = Double;
+                       break;
+               default:
+                       value = val;
+                       break;
+               }
+       }
 }
 
 
 void Spacing::writeFile(ostream & os, bool para) const
 {
        if (space == Default) return;
-       
+
        string cmd = para ? "\\paragraph_spacing " : "\\spacing ";
-       
+
        if (getSpace() == Spacing::Other) {
-               os.setf(ios::showpoint|ios::fixed);
-               os.precision(2);
                os << cmd << spacing_string[getSpace()]
-                  << " " << getValue() << " \n";
+                  << ' ' << getValueAsString() << "\n";
        } else {
-               os << cmd << spacing_string[getSpace()] << " \n";
-       }       
+               os << cmd << spacing_string[getSpace()] << "\n";
+       }
 }
 
 
@@ -96,8 +107,8 @@ string const Spacing::writeEnvirBegin() const
        {
                ostringstream ost;
                ost << "\\begin{spacing}{"
-                   << getValue() << "}";
-               return ost.str().c_str();
+                   << getValueAsString() << '}';
+               return ost.str();
        }
        }
        return string();
@@ -119,3 +130,6 @@ string const Spacing::writeEnvirEnd() const
        }
        return string();
 }
+
+
+} // namespace lyx