$include_dir="/home/hyper-archives/boost-commit/include"; include("$include_dir/msg-header.inc") ?>
From: boost_at_[hidden]
Date: 2008-05-26 13:18:03
Author: matthiasschabel
Date: 2008-05-26 13:18:02 EDT (Mon, 26 May 2008)
New Revision: 45776
URL: http://svn.boost.org/trac/boost/changeset/45776
Log:
fix some problems with output
scaled unit/quantity output still has bugs, demonstrated in examples/composite_output.cpp
Text files modified: 
   sandbox/units/boost/units/io.hpp                      |    29 +++++++++++++----------------           
   sandbox/units/boost/units/scale.hpp                   |    38 ++++++++++++++++++++++++++------------  
   sandbox/units/libs/units/example/composite_output.cpp |    26 ++++++++++++++++++++++----              
   sandbox/units/libs/units/example/kitchen_sink.cpp     |    38 +++++++++++++++++++++++++++++++++++++-  
   4 files changed, 98 insertions(+), 33 deletions(-)
Modified: sandbox/units/boost/units/io.hpp
==============================================================================
--- sandbox/units/boost/units/io.hpp	(original)
+++ sandbox/units/boost/units/io.hpp	2008-05-26 13:18:02 EDT (Mon, 26 May 2008)
@@ -26,6 +26,7 @@
 #include <boost/units/units_fwd.hpp>
 #include <boost/units/heterogeneous_system.hpp>
 #include <boost/units/quantity.hpp>
+#include <boost/units/scale.hpp>
 #include <boost/units/static_rational.hpp>
 #include <boost/units/unit.hpp>
 
@@ -159,32 +160,30 @@
     return(ios);
 }
 
-// by default, return result of static symbol() method for class
-template<class T>
-inline std::string symbol_string(const T&)
+namespace detail {
+
+template<integer_type N, integer_type D>
+std::string exponent_string(const static_rational<N,D>& r)
 {
-	return T::symbol();
+	return '^' + to_string(r);
 }
 
-// by default, return result of static name() method for class
-template<class T>
-inline std::string name_string(const T&)
+template<>
+inline std::string exponent_string(const static_rational<1>& r)
 {
-	return T::name();
+	return "";
 }
 
-namespace detail {
-
 template<class T>
 std::string base_unit_symbol_string(const T&)
 {
-	return base_unit_info<typename T::tag_type>::symbol() + '^' + to_string(typename T::value_type());
+	return base_unit_info<typename T::tag_type>::symbol() + exponent_string(typename T::value_type());
 }
 
 template<class T>	
 std::string base_unit_name_string(const T&)
 {
-	return base_unit_info<typename T::tag_type>::name() + '^' + to_string(typename T::value_type());
+	return base_unit_info<typename T::tag_type>::name() + exponent_string(typename T::value_type());
 }
 
 // stringify with symbols
@@ -238,7 +237,7 @@
         {
         static void value(std::string& str) 
                 {
-            str += mpl::deref<Begin>::type::base::symbol() + ' ';
+            str += mpl::deref<Begin>::type::symbol();
             scale_symbol_string_impl<N - 1>::template apply<typename mpl::next<Begin>::type>::value(str);
         }
     };
@@ -305,7 +304,7 @@
         {
         static void value(std::string& str) 
                 {
-            str += mpl::deref<Begin>::type::base::name() + ' ';
+            str += mpl::deref<Begin>::type::name();
             scale_name_string_impl<N - 1>::template apply<typename mpl::next<Begin>::type>::value(str);
         }
     };
@@ -368,8 +367,6 @@
 template<class Char, class Traits, class Dimension, class System>
 std::basic_ostream<Char, Traits>& operator<<(std::basic_ostream<Char, Traits>& os, const unit<Dimension, System>& u)
 {
-//    os << typename reduce_unit<unit<Dimension, System> >::type();
-//    return(os);
     if(units::get_format(os) == symbol) 
         {
         os << symbol_string(u);
Modified: sandbox/units/boost/units/scale.hpp
==============================================================================
--- sandbox/units/boost/units/scale.hpp	(original)
+++ sandbox/units/boost/units/scale.hpp	2008-05-26 13:18:02 EDT (Mon, 26 May 2008)
@@ -35,6 +35,8 @@
     typedef Exponent exponent;
     typedef double value_type;
     static value_type value() { return(detail::static_rational_power<Exponent>(static_cast<double>(base))); }
+	static std::string name() { return ""; };
+	static std::string symbol() { return ""; };
 };
 
 template<long Base, class Exponent>
@@ -55,22 +57,34 @@
 template<long Base>
 const long scale<Base, static_rational<0> >::base;
 
+template<long Base,class Exponent>
+std::string symbol_string(const scale<Base,Exponent>&)
+{
+	return scale<Base,Exponent>::symbol();
+}
+
+template<long Base,class Exponent>
+std::string name_string(const scale<Base,Exponent>&)
+{
+	return scale<Base,Exponent>::name();
+}
+
 #ifndef BOOST_UNITS_DOXYGEN
 
-#define BOOST_UNITS_SCALE_SPECIALIZATION(base_,exponent_,val,name_,symbol_) \
-template<>                                                                  \
-struct scale<base_, exponent_ >                                             \
-{                                                                           \
-    static const long base = base_;                                         \
-    typedef exponent_ exponent;                                             \
-    typedef double value_type;                                              \
-    static value_type value()   { return(val); }                            \
-    static std::string name()   { return(#name_); }                         \
-    static std::string symbol() { return(#symbol_); }                       \
+#define BOOST_UNITS_SCALE_SPECIALIZATION(base_,exponent_,val_,name_,symbol_) \
+template<>                                                                   \
+struct scale<base_, exponent_ >                                              \
+{                                                                            \
+    static const long base = base_;                                          \
+    typedef exponent_ exponent;                                              \
+    typedef double value_type;                                               \
+    static value_type value()   { return(val_); }                            \
+    static std::string name()   { return(#name_); }                          \
+    static std::string symbol() { return(#symbol_); }                        \
 }
 
-#define BOOST_UNITS_SCALE_DEF(exponent,value,name_,symbol_)                 \
-BOOST_UNITS_SCALE_SPECIALIZATION(10,static_rational<exponent>,value, name_, symbol_)
+#define BOOST_UNITS_SCALE_DEF(exponent_,value_,name_,symbol_)                 \
+BOOST_UNITS_SCALE_SPECIALIZATION(10,static_rational<exponent_>,value_, name_, symbol_)
 
 BOOST_UNITS_SCALE_DEF(-24, 1e-24 ,yocto, y);
 BOOST_UNITS_SCALE_DEF(-21, 1e-21, zepto, z);
Modified: sandbox/units/libs/units/example/composite_output.cpp
==============================================================================
--- sandbox/units/libs/units/example/composite_output.cpp	(original)
+++ sandbox/units/libs/units/example/composite_output.cpp	2008-05-26 13:18:02 EDT (Mon, 26 May 2008)
@@ -15,6 +15,7 @@
 
 #include <boost/units/systems/si/capacitance.hpp>
 #include <boost/units/systems/si/io.hpp>
+#include <boost/units/systems/si/prefixes.hpp>
 
 #include <iostream>
 #include <sstream>
@@ -51,6 +52,11 @@
         using boost::units::cgs::second;
         using boost::units::cgs::dyne;
         
+	scale<10,static_rational<-9> >	s;
+	
+	si::nano*si::farad;
+	1.0*si::nano*si::farad;
+	
     std::cout << 2.0 * dyne << std::endl;
 
     std::cout << symbol_format << 2.0 * dyne << std::endl;
@@ -67,14 +73,26 @@
         
         std::cout << symbol_string(scale<10,static_rational<-9> >()) << std::endl;
         std::cout << name_string(scale<10,static_rational<-9> >()) << std::endl;
-	
-	// should work...
-//	std::cout << symbol_string(scale<10,static_rational<-9> >()*si::farad) << std::endl;
-//	std::cout << name_string(scale<10,static_rational<-9> >()*si::farad) << std::endl;
+
+	// wrong! - should output "n" and "nano", respectively
+	std::cout << symbol_string(si::nano) << std::endl;
+	std::cout << name_string(si::nano) << std::endl;
         
         std::cout << name_format << si::farad << std::endl;
         std::cout << symbol_format << si::farad << std::endl;
         
         std::cout << name_format << 1.0*si::farad << std::endl;
         std::cout << symbol_format << 1.0*si::farad << std::endl;
+	
+	// wrong! - should output "nF" and "nanofarad", respectively
+	std::cout << symbol_string(si::farad*si::nano) << std::endl;
+	std::cout << name_string(si::farad*si::nano) << std::endl;
+	
+	// should work but doesn't
+//	std::cout << symbol_string(si::nano*1.0*si::farad) << std::endl;
+//	std::cout << name_string(1.0*si::nano*si::farad) << std::endl;
+
+	// wrong! - should output "n(cm g s^-1)" and "nano(centimeter gram second^-1)", respectively
+	std::cout << symbol_string(si::nano*gram*centimeter/second) << std::endl;
+	std::cout << name_string(si::nano*gram*centimeter/second) << std::endl;
 }
Modified: sandbox/units/libs/units/example/kitchen_sink.cpp
==============================================================================
--- sandbox/units/libs/units/example/kitchen_sink.cpp	(original)
+++ sandbox/units/libs/units/example/kitchen_sink.cpp	2008-05-26 13:18:02 EDT (Mon, 26 May 2008)
@@ -147,6 +147,7 @@
 #include <boost/units/io.hpp>
 #include <boost/units/systems/si.hpp>
 #include <boost/units/systems/si/codata/physico-chemical_constants.hpp>
+#include <boost/units/systems/si/io.hpp>
 
 #include "measurement.hpp"
 
@@ -421,7 +422,42 @@
               << quantity<power>(1.0*watt) << std::endl
               << quantity<pressure>(1.0*pascals) << std::endl
               << quantity<resistance>(1.0*ohm) << std::endl
-              << quantity<torque>(1.0*newton_meters) << std::endl
+              << std::endl;
+
+    std::cout << symbol_format << quantity<capacitance>(1.0*farad) << std::endl
+              << symbol_format << quantity<catalytic_activity>(1.0*katal) << std::endl
+              << symbol_format << quantity<conductance>(1.0*siemen) << std::endl
+              << symbol_format << quantity<electric_charge>(1.0*coulomb) << std::endl
+              << symbol_format << quantity<electric_potential>(1.0*volt) << std::endl
+              << symbol_format << quantity<energy>(1.0*joule) << std::endl
+              << symbol_format << quantity<force>(1.0*newton) << std::endl
+              << symbol_format << quantity<frequency>(1.0*hertz) << std::endl
+              << symbol_format << quantity<illuminance>(1.0*lux) << std::endl
+              << symbol_format << quantity<inductance>(1.0*henry) << std::endl
+              << symbol_format << quantity<luminous_flux>(1.0*lumen) << std::endl
+              << symbol_format << quantity<magnetic_flux>(1.0*weber) << std::endl
+              << symbol_format << quantity<magnetic_flux_density>(1.0*tesla) << std::endl
+              << symbol_format << quantity<power>(1.0*watt) << std::endl
+              << symbol_format << quantity<pressure>(1.0*pascals) << std::endl
+              << symbol_format << quantity<resistance>(1.0*ohm) << std::endl
+              << std::endl;
+
+    std::cout << name_format << quantity<capacitance>(1.0*farad) << std::endl
+              << name_format << quantity<catalytic_activity>(1.0*katal) << std::endl
+              << name_format << quantity<conductance>(1.0*siemen) << std::endl
+              << name_format << quantity<electric_charge>(1.0*coulomb) << std::endl
+              << name_format << quantity<electric_potential>(1.0*volt) << std::endl
+              << name_format << quantity<energy>(1.0*joule) << std::endl
+              << name_format << quantity<force>(1.0*newton) << std::endl
+              << name_format << quantity<frequency>(1.0*hertz) << std::endl
+              << name_format << quantity<illuminance>(1.0*lux) << std::endl
+              << name_format << quantity<inductance>(1.0*henry) << std::endl
+              << name_format << quantity<luminous_flux>(1.0*lumen) << std::endl
+              << name_format << quantity<magnetic_flux>(1.0*weber) << std::endl
+              << name_format << quantity<magnetic_flux_density>(1.0*tesla) << std::endl
+              << name_format << quantity<power>(1.0*watt) << std::endl
+              << name_format << quantity<pressure>(1.0*pascals) << std::endl
+              << name_format << quantity<resistance>(1.0*ohm) << std::endl
               << std::endl;
 
     sstream2 << "S1 :    2" << std::endl;