From 1c06fa5e2552b291534f0b96542070f00fd0778b Mon Sep 17 00:00:00 2001
From: Matthew Petroff <matthew@mpetroff.net>
Date: Mon, 24 Jun 2013 20:47:57 -0400
Subject: Added length class.

(bzr r12380.1.1)
---
 src/util/units.cpp | 39 ++++++++++++++++++++++++++++++++++++++-
 1 file changed, 38 insertions(+), 1 deletion(-)

(limited to 'src/util/units.cpp')

diff --git a/src/util/units.cpp b/src/util/units.cpp
index f822d01de..3bcbabf89 100644
--- a/src/util/units.cpp
+++ b/src/util/units.cpp
@@ -334,10 +334,47 @@ void UnitsSAXHandler::_endElement(xmlChar const *xname)
     }
 }
 
+/** Initialize a length. */
+Length::Length(Unit *u, double l) {
+    unit = u;
+    length = l;
+}
+
+/** Checks if a length is compatible with the specified unit. */
+bool Length::compatibleWith(Unit *u) {
+    // Percentages
+    if (unit->type == UNIT_TYPE_DIMENSIONLESS || u->type == UNIT_TYPE_DIMENSIONLESS) {
+        return true;
+    }
+    
+    // Other units with same type
+    if (unit->type == u->type) {
+        return true;
+    }
+    
+    // Different, incompatible types
+    return false;
+}
+
+/** Return the length's value in the specified unit. */
+double Length::value(Unit *u) {
+    return convert(length, unit, u);
+}
+
+/** Convert distances. */
+double Length::convert(double from_dist, Unit *from, Unit *to) const {
+    // Incompatible units
+    if (from->type != to->type) {
+        return -1;
+    }
+    
+    // Compatible units
+    return from_dist / from->factor * to->factor;
+}
+
 } // namespace Util
 } // namespace Inkscape
 
-
 /*
   Local Variables:
   mode:c++
-- 
cgit v1.2.3


From e921e43e62cc68aaf29079b26392b17208689568 Mon Sep 17 00:00:00 2001
From: Matthew Petroff <matthew@mpetroff.net>
Date: Mon, 1 Jul 2013 23:39:15 -0400
Subject: Renamed Length class to Quantity class, fixed bugs, and added
 functions.

(bzr r12380.1.3)
---
 src/util/units.cpp | 84 ++++++++++++++++++++++++++++++++++++++++--------------
 1 file changed, 62 insertions(+), 22 deletions(-)

(limited to 'src/util/units.cpp')

diff --git a/src/util/units.cpp b/src/util/units.cpp
index 3bcbabf89..d0e4e7941 100644
--- a/src/util/units.cpp
+++ b/src/util/units.cpp
@@ -55,6 +55,57 @@ int Unit::defaultDigits() const {
     }
 }
 
+/** Checks if a unit is compatible with the specified unit. */
+bool Unit::compatibleWith(const Unit *u) const {
+    // Percentages
+    if (type == UNIT_TYPE_DIMENSIONLESS || u->type == UNIT_TYPE_DIMENSIONLESS) {
+        return true;
+    }
+    
+    // Other units with same type
+    if (type == u->type) {
+        return true;
+    }
+    
+    // Different, incompatible types
+    return false;
+}
+
+/** Check if units are equal. */
+bool operator== (const Unit &u1, const Unit &u2) {
+    return (u1.type == u2.type && u1.name.compare(u2.name) == 0);
+}
+
+/** Check if units are not equal. */ 
+bool operator!= (const Unit &u1, const Unit &u2) {
+    return !(u1 == u2);
+}
+
+/** Temporary - get SVG unit. */
+int Unit::svgUnit() const {
+    if (!abbr.compare("px"))
+        return 1;
+    if (!abbr.compare("pt"))
+        return 2;
+    if (!abbr.compare("pc"))
+        return 3;
+    if (!abbr.compare("mm"))
+        return 4;
+    if (!abbr.compare("cm"))
+        return 5;
+    if (!abbr.compare("in"))
+        return 6;
+    if (!abbr.compare("ft"))
+        return 7;
+    if (!abbr.compare("em"))
+        return 8;
+    if (!abbr.compare("ex"))
+        return 9;
+    if (!abbr.compare("%"))
+        return 10;
+    return 0;
+}
+
 /**
  * Initializes the unit tables and identifies the primary unit types.
  *
@@ -334,42 +385,31 @@ void UnitsSAXHandler::_endElement(xmlChar const *xname)
     }
 }
 
-/** Initialize a length. */
-Length::Length(Unit *u, double l) {
+/** Initialize a quantity. */
+Quantity::Quantity(Unit *u, double q) {
     unit = u;
-    length = l;
+    quantity = q;
 }
 
-/** Checks if a length is compatible with the specified unit. */
-bool Length::compatibleWith(Unit *u) {
-    // Percentages
-    if (unit->type == UNIT_TYPE_DIMENSIONLESS || u->type == UNIT_TYPE_DIMENSIONLESS) {
-        return true;
-    }
-    
-    // Other units with same type
-    if (unit->type == u->type) {
-        return true;
-    }
-    
-    // Different, incompatible types
-    return false;
+/** Checks if a quantity is compatible with the specified unit. */
+bool Quantity::compatibleWith(const Unit *u) const {
+    return unit->compatibleWith(u);
 }
 
-/** Return the length's value in the specified unit. */
-double Length::value(Unit *u) {
-    return convert(length, unit, u);
+/** Return the quantity's value in the specified unit. */
+double Quantity::value(Unit *u) const {
+    return convert(quantity, unit, u);
 }
 
 /** Convert distances. */
-double Length::convert(double from_dist, Unit *from, Unit *to) const {
+double Quantity::convert(const double from_dist, const Unit *from, const Unit *to) {
     // Incompatible units
     if (from->type != to->type) {
         return -1;
     }
     
     // Compatible units
-    return from_dist / from->factor * to->factor;
+    return from_dist * from->factor / to->factor;
 }
 
 } // namespace Util
-- 
cgit v1.2.3


From d04405f745da587b2a3ccca8d2ca7bf49edf2d4d Mon Sep 17 00:00:00 2001
From: Matthew Petroff <matthew@mpetroff.net>
Date: Sat, 6 Jul 2013 11:01:59 -0400
Subject: Switch setWidth and setHeight to use Quantity and switch to forward
 declaration of Inkscape::Util::Quantity in document.h.

(bzr r12380.1.7)
---
 src/util/units.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'src/util/units.cpp')

diff --git a/src/util/units.cpp b/src/util/units.cpp
index d0e4e7941..a51eb2570 100644
--- a/src/util/units.cpp
+++ b/src/util/units.cpp
@@ -386,7 +386,7 @@ void UnitsSAXHandler::_endElement(xmlChar const *xname)
 }
 
 /** Initialize a quantity. */
-Quantity::Quantity(Unit *u, double q) {
+Quantity::Quantity(double q, const Unit *u) {
     unit = u;
     quantity = q;
 }
-- 
cgit v1.2.3


From 9dc7b786c9ef31060012ea4ae13a8188548b4f62 Mon Sep 17 00:00:00 2001
From: Matthew Petroff <matthew@mpetroff.net>
Date: Tue, 9 Jul 2013 16:42:04 -0400
Subject: Ported sp-namedview.cpp (todo: fix a bunch of things).

(bzr r12380.1.8)
---
 src/util/units.cpp | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

(limited to 'src/util/units.cpp')

diff --git a/src/util/units.cpp b/src/util/units.cpp
index a51eb2570..6c225f717 100644
--- a/src/util/units.cpp
+++ b/src/util/units.cpp
@@ -106,6 +106,27 @@ int Unit::svgUnit() const {
     return 0;
 }
 
+/** Temporary - get metric. */
+int Unit::metric() const {
+    if (!abbr.compare("mm"))
+        return 1;
+    if (!abbr.compare("cm"))
+        return 2;
+    if (!abbr.compare("in"))
+        return 3;
+    if (!abbr.compare("ft"))
+        return 4;
+    if (!abbr.compare("pt"))
+        return 5;
+    if (!abbr.compare("pc"))
+        return 6;
+    if (!abbr.compare("px"))
+        return 7;
+    if (!abbr.compare("m"))
+        return 8;
+    return 0;
+}
+
 /**
  * Initializes the unit tables and identifies the primary unit types.
  *
-- 
cgit v1.2.3


From ed0b9d91e08cf55287745636608bddd5ebd5ea34 Mon Sep 17 00:00:00 2001
From: Matthew Petroff <matthew@mpetroff.net>
Date: Wed, 17 Jul 2013 00:29:41 -0400
Subject: Added copyright header and name to AUTHORS.

(bzr r12380.1.13)
---
 src/util/units.cpp | 11 +++++++++++
 1 file changed, 11 insertions(+)

(limited to 'src/util/units.cpp')

diff --git a/src/util/units.cpp b/src/util/units.cpp
index 6c225f717..e120207c9 100644
--- a/src/util/units.cpp
+++ b/src/util/units.cpp
@@ -1,3 +1,14 @@
+/*
+ * Inkscape Units
+ *
+ * Authors:
+ *   Matthew Petroff <matthew@mpetroff.net>
+ *
+ * Copyright (C) 2013 Matthew Petroff
+ *
+ * Released under GNU GPL, read the file 'COPYING' for more information
+ */
+
 #ifdef HAVE_CONFIG_H
 # include <config.h>
 #endif
-- 
cgit v1.2.3


From 63071dedd65d63cfac2041399dab0191e3753b42 Mon Sep 17 00:00:00 2001
From: Matthew Petroff <matthew@mpetroff.net>
Date: Wed, 17 Jul 2013 01:31:18 -0400
Subject: Fixed botched merge.

(bzr r12380.1.18)
---
 src/util/units.cpp | 5 -----
 1 file changed, 5 deletions(-)

(limited to 'src/util/units.cpp')

diff --git a/src/util/units.cpp b/src/util/units.cpp
index f6350d569..d485f6aef 100644
--- a/src/util/units.cpp
+++ b/src/util/units.cpp
@@ -196,11 +196,6 @@ int Unit::metric() const {
     return 0;
 }
 
-/**
- * Initializes the unit tables and identifies the primary unit types.
- *
- * The primary unit's conversion factor is required to be 1.00
- */
 UnitTable::UnitTable()
 {
     // if we swich to the xml file, don't forget to force locale to 'C'
-- 
cgit v1.2.3


From 65e3eb1a68581bede3788501a3052e97df16c91a Mon Sep 17 00:00:00 2001
From: Matthew Petroff <matthew@mpetroff.net>
Date: Thu, 18 Jul 2013 15:00:03 -0400
Subject: Added quantity string parsing.

(bzr r12380.1.24)
---
 src/util/units.cpp | 22 ++++++++++++++++++++++
 1 file changed, 22 insertions(+)

(limited to 'src/util/units.cpp')

diff --git a/src/util/units.cpp b/src/util/units.cpp
index d485f6aef..705fc850c 100644
--- a/src/util/units.cpp
+++ b/src/util/units.cpp
@@ -16,6 +16,7 @@
 #include <cmath>
 #include <cerrno>
 #include <glib.h>
+#include <glibmm/regex.h>
 
 #include "io/simple-sax.h"
 #include "util/units.h"
@@ -228,6 +229,27 @@ Unit UnitTable::getUnit(Glib::ustring const &unit_abbr) const {
     }
 }
 
+Quantity UnitTable::getQuantity(Glib::ustring const& q) const {
+    Glib::MatchInfo match_info;
+    
+    // Extract value
+    double value = 0;
+    Glib::RefPtr<Glib::Regex> value_regex = Glib::Regex::create("\\d+\\.?\\d");
+    if (value_regex->match(q, match_info)) {
+        value = atof(match_info.fetch(0).c_str());
+    }
+    
+    // Extract unit abbreviation
+    Glib::ustring abbr;
+    Glib::RefPtr<Glib::Regex> unit_regex = Glib::Regex::create("[A-z]+");
+    if (unit_regex->match(q, match_info)) {
+        abbr = match_info.fetch(0);
+    }
+    Unit *u = new Inkscape::Util::Unit(getUnit(abbr));
+    
+    return Quantity(value, u);
+}
+
 bool UnitTable::deleteUnit(Unit const &u) {
     bool deleted = false;
     // Cannot delete the primary unit type since it's
-- 
cgit v1.2.3


From 286ea5f976f27f7fdfec5859b08bcb31e53e6728 Mon Sep 17 00:00:00 2001
From: Matthew Petroff <matthew@mpetroff.net>
Date: Thu, 18 Jul 2013 16:13:16 -0400
Subject: Added more convienient unit conversion functions.

(bzr r12380.1.26)
---
 src/util/units.cpp | 26 +++++++++++++++++++++++++-
 1 file changed, 25 insertions(+), 1 deletion(-)

(limited to 'src/util/units.cpp')

diff --git a/src/util/units.cpp b/src/util/units.cpp
index 705fc850c..ed8fbfa34 100644
--- a/src/util/units.cpp
+++ b/src/util/units.cpp
@@ -462,9 +462,14 @@ bool Quantity::compatibleWith(const Unit *u) const {
 }
 
 /** Return the quantity's value in the specified unit. */
-double Quantity::value(Unit *u) const {
+double Quantity::value(const Unit *u) const {
     return convert(quantity, unit, u);
 }
+double Quantity::value(const Glib::ustring u) const {
+    static UnitTable unit_table;
+    Unit to_unit = unit_table.getUnit(u);
+    return value(&to_unit);
+}
 
 /** Convert distances. */
 double Quantity::convert(const double from_dist, const Unit *from, const Unit *to) {
@@ -476,6 +481,25 @@ double Quantity::convert(const double from_dist, const Unit *from, const Unit *t
     // Compatible units
     return from_dist * from->factor / to->factor;
 }
+double Quantity::convert(const double from_dist, const Glib::ustring from, const Unit &to)
+{
+    static UnitTable unit_table;
+    Unit from_unit = unit_table.getUnit(from);
+    return convert(from_dist, &from_unit, &to);
+}
+double Quantity::convert(const double from_dist, const Unit &from, const Glib::ustring to)
+{
+    static UnitTable unit_table;
+    Unit to_unit = unit_table.getUnit(to);
+    return convert(from_dist, &from, &to_unit);
+}
+double Quantity::convert(const double from_dist, const Glib::ustring from, const Glib::ustring to)
+{
+    static UnitTable unit_table;
+    Unit from_unit = unit_table.getUnit(from);
+    Unit to_unit = unit_table.getUnit(to);
+    return convert(from_dist, &from_unit, &to_unit);
+}
 
 } // namespace Util
 } // namespace Inkscape
-- 
cgit v1.2.3


From 51ffb1d56821be424fc50c91e486d6143976ba30 Mon Sep 17 00:00:00 2001
From: Matthew Petroff <matthew@mpetroff.net>
Date: Thu, 18 Jul 2013 16:40:36 -0400
Subject: Added more more convientent unit functions.

(bzr r12380.1.27)
---
 src/util/units.cpp | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

(limited to 'src/util/units.cpp')

diff --git a/src/util/units.cpp b/src/util/units.cpp
index ed8fbfa34..ffbd74fdd 100644
--- a/src/util/units.cpp
+++ b/src/util/units.cpp
@@ -140,6 +140,12 @@ bool Unit::compatibleWith(const Unit *u) const {
     // Different, incompatible types
     return false;
 }
+bool Unit::compatibleWith(const Glib::ustring u) const
+{
+    static UnitTable unit_table;
+    Unit compatible_unit = unit_table.getUnit(u);
+    return compatibleWith(&compatible_unit);
+}
 
 /** Check if units are equal. */
 bool operator== (const Unit &u1, const Unit &u2) {
@@ -455,11 +461,22 @@ Quantity::Quantity(double q, const Unit *u) {
     unit = u;
     quantity = q;
 }
+Quantity::Quantity(double q, const Glib::ustring u) {
+    UnitTable unit_table;
+    unit = new Unit(unit_table.getUnit(u));
+    quantity = q;
+}
 
 /** Checks if a quantity is compatible with the specified unit. */
 bool Quantity::compatibleWith(const Unit *u) const {
     return unit->compatibleWith(u);
 }
+bool Quantity::compatibleWith(const Glib::ustring u) const
+{
+    static UnitTable unit_table;
+    Unit other_unit = unit_table.getUnit(u);
+    return compatibleWith(&other_unit);
+}
 
 /** Return the quantity's value in the specified unit. */
 double Quantity::value(const Unit *u) const {
-- 
cgit v1.2.3


From 3772fc428950b2b946a1bd7c7c97e06219c3165f Mon Sep 17 00:00:00 2001
From: Matthew Petroff <matthew@mpetroff.net>
Date: Thu, 18 Jul 2013 17:21:24 -0400
Subject: Switch unit functions from using pointer arguements to reference
 arguements.

(bzr r12380.1.28)
---
 src/util/units.cpp | 92 ++++++++++++++++++++++++++++++------------------------
 1 file changed, 51 insertions(+), 41 deletions(-)

(limited to 'src/util/units.cpp')

diff --git a/src/util/units.cpp b/src/util/units.cpp
index ffbd74fdd..78531bfaf 100644
--- a/src/util/units.cpp
+++ b/src/util/units.cpp
@@ -115,7 +115,8 @@ void Unit::clear()
     *this = Unit();
 }
 
-int Unit::defaultDigits() const {
+int Unit::defaultDigits() const
+{
     int factor_digits = int(log10(factor));
     if (factor_digits < 0) {
         g_warning("factor = %f, factor_digits = %d", factor, factor_digits);
@@ -126,14 +127,15 @@ int Unit::defaultDigits() const {
 }
 
 /** Checks if a unit is compatible with the specified unit. */
-bool Unit::compatibleWith(const Unit *u) const {
+bool Unit::compatibleWith(const Unit &u) const
+{
     // Percentages
-    if (type == UNIT_TYPE_DIMENSIONLESS || u->type == UNIT_TYPE_DIMENSIONLESS) {
+    if (type == UNIT_TYPE_DIMENSIONLESS || u.type == UNIT_TYPE_DIMENSIONLESS) {
         return true;
     }
     
     // Other units with same type
-    if (type == u->type) {
+    if (type == u.type) {
         return true;
     }
     
@@ -143,22 +145,24 @@ bool Unit::compatibleWith(const Unit *u) const {
 bool Unit::compatibleWith(const Glib::ustring u) const
 {
     static UnitTable unit_table;
-    Unit compatible_unit = unit_table.getUnit(u);
-    return compatibleWith(&compatible_unit);
+    return compatibleWith(unit_table.getUnit(u));
 }
 
 /** Check if units are equal. */
-bool operator== (const Unit &u1, const Unit &u2) {
+bool operator== (const Unit &u1, const Unit &u2)
+{
     return (u1.type == u2.type && u1.name.compare(u2.name) == 0);
 }
 
 /** Check if units are not equal. */ 
-bool operator!= (const Unit &u1, const Unit &u2) {
+bool operator!= (const Unit &u1, const Unit &u2)
+{
     return !(u1 == u2);
 }
 
 /** Temporary - get SVG unit. */
-int Unit::svgUnit() const {
+int Unit::svgUnit() const
+{
     if (!abbr.compare("px"))
         return 1;
     if (!abbr.compare("pt"))
@@ -183,7 +187,8 @@ int Unit::svgUnit() const {
 }
 
 /** Temporary - get metric. */
-int Unit::metric() const {
+int Unit::metric() const
+{
     if (!abbr.compare("mm"))
         return 1;
     if (!abbr.compare("cm"))
@@ -212,21 +217,24 @@ UnitTable::UnitTable()
     g_free(filename);
 }
 
-UnitTable::~UnitTable() {
+UnitTable::~UnitTable()
+{
     for (UnitMap::iterator iter = _unit_map.begin(); iter != _unit_map.end(); ++iter)
     {
         delete (*iter).second;
     }
 }
 
-void UnitTable::addUnit(Unit const &u, bool primary) {
+void UnitTable::addUnit(Unit const &u, bool primary)
+{
     _unit_map[u.abbr] = new Unit(u);
     if (primary) {
         _primary_unit[u.type] = u.abbr;
     }
 }
 
-Unit UnitTable::getUnit(Glib::ustring const &unit_abbr) const {
+Unit UnitTable::getUnit(Glib::ustring const &unit_abbr) const
+{
     UnitMap::const_iterator iter = _unit_map.find(unit_abbr);
     if (iter != _unit_map.end()) {
         return *((*iter).second);
@@ -235,7 +243,8 @@ Unit UnitTable::getUnit(Glib::ustring const &unit_abbr) const {
     }
 }
 
-Quantity UnitTable::getQuantity(Glib::ustring const& q) const {
+Quantity UnitTable::getQuantity(Glib::ustring const& q) const
+{
     Glib::MatchInfo match_info;
     
     // Extract value
@@ -251,12 +260,12 @@ Quantity UnitTable::getQuantity(Glib::ustring const& q) const {
     if (unit_regex->match(q, match_info)) {
         abbr = match_info.fetch(0);
     }
-    Unit *u = new Inkscape::Util::Unit(getUnit(abbr));
     
-    return Quantity(value, u);
+    return Quantity(value, abbr);
 }
 
-bool UnitTable::deleteUnit(Unit const &u) {
+bool UnitTable::deleteUnit(Unit const &u)
+{
     bool deleted = false;
     // Cannot delete the primary unit type since it's
     // used for conversions
@@ -365,7 +374,8 @@ bool UnitTable::loadText(Glib::ustring const &filename)
     return true;
 }
 
-bool UnitTable::load(Glib::ustring const &filename) {
+bool UnitTable::load(Glib::ustring const &filename)
+{
     UnitsSAXHandler handler(this);
 
     int result = handler.parseFile( filename.c_str() );
@@ -378,8 +388,8 @@ bool UnitTable::load(Glib::ustring const &filename) {
     return true;
 }
 
-bool UnitTable::save(Glib::ustring const &filename) {
-
+bool UnitTable::save(Glib::ustring const &filename)
+{
     // open file for writing
     FILE *f = fopen(filename.c_str(), "w");
     if (f == NULL) {
@@ -457,65 +467,65 @@ void UnitsSAXHandler::_endElement(xmlChar const *xname)
 }
 
 /** Initialize a quantity. */
-Quantity::Quantity(double q, const Unit *u) {
-    unit = u;
+Quantity::Quantity(double q, const Unit &u)
+{
+    unit = new Unit(u);
     quantity = q;
 }
-Quantity::Quantity(double q, const Glib::ustring u) {
+Quantity::Quantity(double q, const Glib::ustring u)
+{
     UnitTable unit_table;
     unit = new Unit(unit_table.getUnit(u));
     quantity = q;
 }
 
 /** Checks if a quantity is compatible with the specified unit. */
-bool Quantity::compatibleWith(const Unit *u) const {
+bool Quantity::compatibleWith(const Unit &u) const
+{
     return unit->compatibleWith(u);
 }
 bool Quantity::compatibleWith(const Glib::ustring u) const
 {
     static UnitTable unit_table;
-    Unit other_unit = unit_table.getUnit(u);
-    return compatibleWith(&other_unit);
+    return compatibleWith(unit_table.getUnit(u));
 }
 
 /** Return the quantity's value in the specified unit. */
-double Quantity::value(const Unit *u) const {
-    return convert(quantity, unit, u);
+double Quantity::value(const Unit &u) const
+{
+    return convert(quantity, *unit, u);
 }
-double Quantity::value(const Glib::ustring u) const {
+double Quantity::value(const Glib::ustring u) const
+{
     static UnitTable unit_table;
-    Unit to_unit = unit_table.getUnit(u);
-    return value(&to_unit);
+    return value(unit_table.getUnit(u));
 }
 
 /** Convert distances. */
-double Quantity::convert(const double from_dist, const Unit *from, const Unit *to) {
+double Quantity::convert(const double from_dist, const Unit &from, const Unit &to)
+{
     // Incompatible units
-    if (from->type != to->type) {
+    if (from.type != to.type) {
         return -1;
     }
     
     // Compatible units
-    return from_dist * from->factor / to->factor;
+    return from_dist * from.factor / to.factor;
 }
 double Quantity::convert(const double from_dist, const Glib::ustring from, const Unit &to)
 {
     static UnitTable unit_table;
-    Unit from_unit = unit_table.getUnit(from);
-    return convert(from_dist, &from_unit, &to);
+    return convert(from_dist, unit_table.getUnit(from), to);
 }
 double Quantity::convert(const double from_dist, const Unit &from, const Glib::ustring to)
 {
     static UnitTable unit_table;
-    Unit to_unit = unit_table.getUnit(to);
-    return convert(from_dist, &from, &to_unit);
+    return convert(from_dist, from, unit_table.getUnit(to));
 }
 double Quantity::convert(const double from_dist, const Glib::ustring from, const Glib::ustring to)
 {
     static UnitTable unit_table;
-    Unit from_unit = unit_table.getUnit(from);
-    Unit to_unit = unit_table.getUnit(to);
-    return convert(from_dist, &from_unit, &to_unit);
+    return convert(from_dist, unit_table.getUnit(from), unit_table.getUnit(to));
 }
 
 } // namespace Util
-- 
cgit v1.2.3


From b4b22e6dba56bc9b0b8c35a9d484f1d86d61f45b Mon Sep 17 00:00:00 2001
From: Matthew Petroff <matthew@mpetroff.net>
Date: Sat, 20 Jul 2013 12:29:12 -0400
Subject: Added percentage support to "Inkscape::Util::Quantity::convert".

(bzr r12380.1.43)
---
 src/util/units.cpp | 5 +++++
 1 file changed, 5 insertions(+)

(limited to 'src/util/units.cpp')

diff --git a/src/util/units.cpp b/src/util/units.cpp
index 78531bfaf..dcb3ae4b1 100644
--- a/src/util/units.cpp
+++ b/src/util/units.cpp
@@ -504,6 +504,11 @@ double Quantity::value(const Glib::ustring u) const
 /** Convert distances. */
 double Quantity::convert(const double from_dist, const Unit &from, const Unit &to)
 {
+    // Percentage
+    if (to.type == UNIT_TYPE_DIMENSIONLESS) {
+        return from_dist * to.factor;
+    }
+    
     // Incompatible units
     if (from.type != to.type) {
         return -1;
-- 
cgit v1.2.3


From 86abdc99654356a2047571e907b933505dbfab76 Mon Sep 17 00:00:00 2001
From: Matthew Petroff <matthew@mpetroff.net>
Date: Sat, 20 Jul 2013 14:45:18 -0400
Subject: Add string output functions for units.

(bzr r12380.1.46)
---
 src/util/units.cpp | 13 +++++++++++++
 1 file changed, 13 insertions(+)

(limited to 'src/util/units.cpp')

diff --git a/src/util/units.cpp b/src/util/units.cpp
index dcb3ae4b1..01424520b 100644
--- a/src/util/units.cpp
+++ b/src/util/units.cpp
@@ -15,6 +15,7 @@
 
 #include <cmath>
 #include <cerrno>
+#include <iomanip>
 #include <glib.h>
 #include <glibmm/regex.h>
 
@@ -501,6 +502,18 @@ double Quantity::value(const Glib::ustring u) const
     return value(unit_table.getUnit(u));
 }
 
+/** Return a printable string of the value in the specified unit. */
+Glib::ustring Quantity::string(const Unit &u) const {
+    return Glib::ustring::format(std::fixed, std::setprecision(2), value(u)) + " " + unit->abbr;
+}
+Glib::ustring Quantity::string(const Glib::ustring u) const {
+    static UnitTable unit_table;
+    return string(unit_table.getUnit(u));
+}
+Glib::ustring Quantity::string() const {
+    return string(*unit);
+}
+
 /** Convert distances. */
 double Quantity::convert(const double from_dist, const Unit &from, const Unit &to)
 {
-- 
cgit v1.2.3


From 0b2190b4a186f4b1a2265c85b4baa801d49e5534 Mon Sep 17 00:00:00 2001
From: Matthew Petroff <matthew@mpetroff.net>
Date: Sun, 21 Jul 2013 23:50:53 -0400
Subject: Removed SPMetric.

(bzr r12380.1.49)
---
 src/util/units.cpp | 22 ----------------------
 1 file changed, 22 deletions(-)

(limited to 'src/util/units.cpp')

diff --git a/src/util/units.cpp b/src/util/units.cpp
index 01424520b..582c52090 100644
--- a/src/util/units.cpp
+++ b/src/util/units.cpp
@@ -187,28 +187,6 @@ int Unit::svgUnit() const
     return 0;
 }
 
-/** Temporary - get metric. */
-int Unit::metric() const
-{
-    if (!abbr.compare("mm"))
-        return 1;
-    if (!abbr.compare("cm"))
-        return 2;
-    if (!abbr.compare("in"))
-        return 3;
-    if (!abbr.compare("ft"))
-        return 4;
-    if (!abbr.compare("pt"))
-        return 5;
-    if (!abbr.compare("pc"))
-        return 6;
-    if (!abbr.compare("px"))
-        return 7;
-    if (!abbr.compare("m"))
-        return 8;
-    return 0;
-}
-
 UnitTable::UnitTable()
 {
     // if we swich to the xml file, don't forget to force locale to 'C'
-- 
cgit v1.2.3


From 90edb3a743ab4e9fc18ff18844cc3624433cd3fc Mon Sep 17 00:00:00 2001
From: Matthew Petroff <matthew@mpetroff.net>
Date: Mon, 29 Jul 2013 21:47:12 -0400
Subject: Update unit extraction regular expressions.

(bzr r12380.1.50)
---
 src/util/units.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

(limited to 'src/util/units.cpp')

diff --git a/src/util/units.cpp b/src/util/units.cpp
index 582c52090..342c9ff32 100644
--- a/src/util/units.cpp
+++ b/src/util/units.cpp
@@ -228,14 +228,14 @@ Quantity UnitTable::getQuantity(Glib::ustring const& q) const
     
     // Extract value
     double value = 0;
-    Glib::RefPtr<Glib::Regex> value_regex = Glib::Regex::create("\\d+\\.?\\d");
+    Glib::RefPtr<Glib::Regex> value_regex = Glib::Regex::create("[-+]*[\\d+]*\\.*[\\d+]*[eE]*[-+]*\\d+");
     if (value_regex->match(q, match_info)) {
         value = atof(match_info.fetch(0).c_str());
     }
     
     // Extract unit abbreviation
     Glib::ustring abbr;
-    Glib::RefPtr<Glib::Regex> unit_regex = Glib::Regex::create("[A-z]+");
+    Glib::RefPtr<Glib::Regex> unit_regex = Glib::Regex::create("[A-z%]+");
     if (unit_regex->match(q, match_info)) {
         abbr = match_info.fetch(0);
     }
-- 
cgit v1.2.3


From 7aab446af9e2eb34ce50c8ef0ec58710fac49396 Mon Sep 17 00:00:00 2001
From: Matthew Petroff <matthew@mpetroff.net>
Date: Mon, 29 Jul 2013 22:51:28 -0400
Subject: Cleanup.

(bzr r12380.1.52)
---
 src/util/units.cpp | 9 ---------
 1 file changed, 9 deletions(-)

(limited to 'src/util/units.cpp')

diff --git a/src/util/units.cpp b/src/util/units.cpp
index f40e33c67..7f60eb391 100644
--- a/src/util/units.cpp
+++ b/src/util/units.cpp
@@ -132,7 +132,6 @@ int Unit::defaultDigits() const
     return factor_digits;
 }
 
-/** Checks if a unit is compatible with the specified unit. */
 bool Unit::compatibleWith(const Unit &u) const
 {
     // Percentages
@@ -154,19 +153,16 @@ bool Unit::compatibleWith(const Glib::ustring u) const
     return compatibleWith(unit_table.getUnit(u));
 }
 
-/** Check if units are equal. */
 bool operator== (const Unit &u1, const Unit &u2)
 {
     return (u1.type == u2.type && u1.name.compare(u2.name) == 0);
 }
 
-/** Check if units are not equal. */ 
 bool operator!= (const Unit &u1, const Unit &u2)
 {
     return !(u1 == u2);
 }
 
-/** Temporary - get SVG unit. */
 int Unit::svgUnit() const
 {
     if (!abbr.compare("px"))
@@ -355,7 +351,6 @@ void UnitParser::on_end_element(Ctx &ctx, Glib::ustring const &name)
     }
 }
 
-/** Initialize a quantity. */
 Quantity::Quantity(double q, const Unit &u)
 {
     unit = new Unit(u);
@@ -368,7 +363,6 @@ Quantity::Quantity(double q, const Glib::ustring u)
     quantity = q;
 }
 
-/** Checks if a quantity is compatible with the specified unit. */
 bool Quantity::compatibleWith(const Unit &u) const
 {
     return unit->compatibleWith(u);
@@ -379,7 +373,6 @@ bool Quantity::compatibleWith(const Glib::ustring u) const
     return compatibleWith(unit_table.getUnit(u));
 }
 
-/** Return the quantity's value in the specified unit. */
 double Quantity::value(const Unit &u) const
 {
     return convert(quantity, *unit, u);
@@ -390,7 +383,6 @@ double Quantity::value(const Glib::ustring u) const
     return value(unit_table.getUnit(u));
 }
 
-/** Return a printable string of the value in the specified unit. */
 Glib::ustring Quantity::string(const Unit &u) const {
     return Glib::ustring::format(std::fixed, std::setprecision(2), value(u)) + " " + unit->abbr;
 }
@@ -402,7 +394,6 @@ Glib::ustring Quantity::string() const {
     return string(*unit);
 }
 
-/** Convert distances. */
 double Quantity::convert(const double from_dist, const Unit &from, const Unit &to)
 {
     // Percentage
-- 
cgit v1.2.3


From 6ae6c0bea96eef09907091279e0678aa5f83102d Mon Sep 17 00:00:00 2001
From: Matthew Petroff <matthew@mpetroff.net>
Date: Sun, 4 Aug 2013 18:01:18 -0400
Subject: Switched to global UnitTable.

(bzr r12380.1.62)
---
 src/util/units.cpp | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

(limited to 'src/util/units.cpp')

diff --git a/src/util/units.cpp b/src/util/units.cpp
index 7f60eb391..7bc910fcc 100644
--- a/src/util/units.cpp
+++ b/src/util/units.cpp
@@ -303,6 +303,8 @@ bool UnitTable::save(std::string const &filename) {
     return true;
 }
 
+Inkscape::Util::UnitTable unit_table;
+
 void UnitParser::on_start_element(Ctx &ctx, Glib::ustring const &name, AttrMap const &attrs)
 {
     if (name == "unit") {
@@ -358,7 +360,6 @@ Quantity::Quantity(double q, const Unit &u)
 }
 Quantity::Quantity(double q, const Glib::ustring u)
 {
-    UnitTable unit_table;
     unit = new Unit(unit_table.getUnit(u));
     quantity = q;
 }
@@ -369,7 +370,6 @@ bool Quantity::compatibleWith(const Unit &u) const
 }
 bool Quantity::compatibleWith(const Glib::ustring u) const
 {
-    static UnitTable unit_table;
     return compatibleWith(unit_table.getUnit(u));
 }
 
@@ -379,7 +379,6 @@ double Quantity::value(const Unit &u) const
 }
 double Quantity::value(const Glib::ustring u) const
 {
-    static UnitTable unit_table;
     return value(unit_table.getUnit(u));
 }
 
@@ -387,7 +386,6 @@ Glib::ustring Quantity::string(const Unit &u) const {
     return Glib::ustring::format(std::fixed, std::setprecision(2), value(u)) + " " + unit->abbr;
 }
 Glib::ustring Quantity::string(const Glib::ustring u) const {
-    static UnitTable unit_table;
     return string(unit_table.getUnit(u));
 }
 Glib::ustring Quantity::string() const {
@@ -411,17 +409,14 @@ double Quantity::convert(const double from_dist, const Unit &from, const Unit &t
 }
 double Quantity::convert(const double from_dist, const Glib::ustring from, const Unit &to)
 {
-    static UnitTable unit_table;
     return convert(from_dist, unit_table.getUnit(from), to);
 }
 double Quantity::convert(const double from_dist, const Unit &from, const Glib::ustring to)
 {
-    static UnitTable unit_table;
     return convert(from_dist, from, unit_table.getUnit(to));
 }
 double Quantity::convert(const double from_dist, const Glib::ustring from, const Glib::ustring to)
 {
-    static UnitTable unit_table;
     return convert(from_dist, unit_table.getUnit(from), unit_table.getUnit(to));
 }
 
-- 
cgit v1.2.3