some todos

4 files changed, 22 insertions, 10 deletions

diff --git a/src/parser/conversion/block.rs b/src/parser/conversion/block.rs
index 280aff3..eba50b1 100644
--- a/src/parser/conversion/block.rs
+++ b/src/parser/conversion/block.rs
@@ -17,7 +17,7 @@ use super::inline::convert_inline;
 
 
 pub fn convert_ssubel(pair: Pair<Rule>) -> Result<Option<c::StructuralSubElement>, Error> {
-    // TODO: This is just a proof of concep. Keep closely to DTD in final version!
+    // TODO: This is just a proof of concept. Keep closely to DTD in final version!
     Ok(Some(match pair.as_rule() {
         Rule::title            => convert_title(pair).into(),
         Rule::paragraph        => convert_paragraph(pair)?.into(),
@@ -62,7 +62,7 @@ fn convert_target(pair: Pair<Rule>) -> Result<e::Target, Error> {
     for p in pair.into_inner() {
         match p.as_rule() {
             Rule::target_name_uq | Rule::target_name_qu => {
-                //TODO: abstract
+                //TODO: abstract away
                 attrs.refid = Some(       ID(p.as_str().to_owned().replace(' ', "-")));
                 attrs.refname.push(NameToken(p.as_str().to_owned()));
             },
@@ -103,7 +103,7 @@ fn convert_image<I>(pair: Pair<Rule>) -> Result<I, Error> where I: Element + Ext
                 "alt"    => image.extra_mut().alt    = Some(opt_val.as_str().to_owned()),
                 "height" => image.extra_mut().height = Some(opt_val.parse()?),
                 "width"  => image.extra_mut().width  = Some(opt_val.parse()?),
-                "scale"  => image.extra_mut().scale  = Some(opt_val.parse()?),  // TODO: can end with %
+                "scale"  => image.extra_mut().scale  = Some(parse_scale(&opt_val)?),
                 "align"  => image.extra_mut().align  = Some(opt_val.parse()?),
                 "target" => image.extra_mut().target = Some(opt_val.parse()?),
                 name => bail!("Unknown Image option {}", name),
@@ -113,6 +113,15 @@ fn convert_image<I>(pair: Pair<Rule>) -> Result<I, Error> where I: Element + Ext
     Ok(image)
 }
 
+fn parse_scale(pair: &Pair<Rule>) -> Result<u8, Error> {
+    let input = if pair.as_str().chars().rev().next() == Some('%') { &pair.as_str()[..pair.as_str().len()-1] } else { pair.as_str() };
+    use pest::error::{Error,ErrorVariant};
+    Ok(input.parse().map_err(|e: std::num::ParseIntError| {
+        let var: ErrorVariant<Rule> = ErrorVariant::CustomError { message: e.to_string() };
+        Error::new_from_span(var, pair.as_span())
+    })?)
+}
+
 fn convert_admonition_gen(pair: Pair<Rule>) -> Result<c::BodyElement, Error> {
     let mut iter = pair.into_inner();
     let typ = iter.next().unwrap().as_str();
diff --git a/src/parser/conversion/inline.rs b/src/parser/conversion/inline.rs
index d0aa524..d857bfe 100644
--- a/src/parser/conversion/inline.rs
+++ b/src/parser/conversion/inline.rs
@@ -32,7 +32,7 @@ fn convert_reference(pair: Pair<Rule>) -> Result<e::Reference, Error> {
     match concrete.as_rule() {
         Rule::reference_target => {
             let rt_inner = concrete.into_inner().next().unwrap(); // reference_target_uq or target_name_qu
-            //TODO: abstract
+            //TODO: abstract away
             id   = Some(       ID(rt_inner.as_str().to_owned().replace(' ', "-")));
             name = Some(NameToken(rt_inner.as_str().to_owned()));
         },
diff --git a/src/parser/pair_ext_parse.rs b/src/parser/pair_ext_parse.rs
index 2ce642a..c7d2d45 100644
--- a/src/parser/pair_ext_parse.rs
+++ b/src/parser/pair_ext_parse.rs
@@ -1,5 +1,6 @@
 use std::str::FromStr;
 
+use pest::Span;
 use pest::iterators::Pair;
 use pest::error::{Error,ErrorVariant};
 
@@ -10,9 +11,11 @@ pub trait PairExt<R> where R: pest::RuleType {
 
 impl<'l, R> PairExt<R> for Pair<'l, R> where R: pest::RuleType {
     fn parse<T, E>(&self) -> Result<T, Error<R>> where T: FromStr<Err = E>, E: ToString {
-        self.as_str().parse().map_err(|e: T::Err| {
-            let var: ErrorVariant<R> = ErrorVariant::CustomError { message: e.to_string() };
-            Error::new_from_span(var, self.as_span())
-        })
+        self.as_str().parse().map_err(|e| to_parse_error(self.as_span(), &e))
     }
 }
+
+pub(crate) fn to_parse_error<E, R>(span: Span, e: &E) -> Error<R> where E: ToString, R: pest::RuleType {
+    let var: ErrorVariant<R> = ErrorVariant::CustomError { message: e.to_string() };
+    Error::new_from_span(var, span)
+}
diff --git a/src/parser/tests.rs b/src/parser/tests.rs
index 331f87b..3ebc952 100644
--- a/src/parser/tests.rs
+++ b/src/parser/tests.rs
@@ -108,8 +108,8 @@ fn admonitions() {
     };
 }
 
-// TODO: substitutions
-// TODO: images
+// TODO: test substitutions
+// TODO: test images
 
 #[allow(clippy::cyclomatic_complexity)]
 #[test]