Skip to content

Commit 133da08

Browse files
author
Johann Hofmann
committed
Merge pull request #4 from durka/expect
convert some unwrap() to expect(format!(...))
2 parents 53e08be + 4e4ce4c commit 133da08

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

src/lexer.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -64,10 +64,10 @@ pub fn tokenize(text: &str) -> Vec<Element> {
6464
split_blocks(text).iter().map(|block| {
6565
if TAG.is_match(*block) {
6666
let caps = TAG.captures(*block).unwrap();
67-
Tag(granularize(caps.at(1).unwrap()), block.to_string())
67+
Tag(granularize(caps.at(1).expect("Empty tag")), block.to_string())
6868
}else if EXPRESSION.is_match(*block) {
6969
let caps = EXPRESSION.captures(*block).unwrap();
70-
Expression(granularize(caps.at(1).unwrap()), block.to_string())
70+
Expression(granularize(caps.at(1).expect("Empty expression")), block.to_string())
7171
}else{
7272
Raw(block.to_string())
7373
}
@@ -110,7 +110,7 @@ fn granularize(block: &str) -> Vec<Token>{
110110
x if DOTDOT.is_match(x) => DotDot,
111111
x if SINGLE_STRING_LITERAL.is_match(x) => StringLiteral(x[1..x.len() -1].to_string()),
112112
x if DOUBLE_STRING_LITERAL.is_match(x) => StringLiteral(x[1..x.len() -1].to_string()),
113-
x if NUMBER_LITERAL.is_match(x) => NumberLiteral(x.parse::<f32>().unwrap()),
113+
x if NUMBER_LITERAL.is_match(x) => NumberLiteral(x.parse::<f32>().expect(&format!("Could not parse {:?} as float", x))),
114114
x if IDENTIFIER.is_match(x) => Identifier(x.to_string()),
115115
x => panic!("{} is not a valid identifier", x)
116116
})

src/output.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,8 +29,8 @@ pub struct Output{
2929
impl Renderable for Output {
3030
fn render (&self, context: &mut Context) -> Option<String>{
3131
let mut entry = match self.entry {
32-
VarOrVal::Val(ref x) => x.render(context).unwrap(),
33-
VarOrVal::Var(ref x) => x.render(context).unwrap()
32+
VarOrVal::Val(ref x) => x.render(context).expect(&format!("Could not render {:?}", x)),
33+
VarOrVal::Var(ref x) => x.render(context).expect(&format!("Could not render {:?}", x))
3434
};
3535
for filter in self.filters.iter(){
3636
let f = match context.filters.get(&filter.name) {

0 commit comments

Comments
 (0)