matthiasbeyer/imag
1
0
Fork 0
Code Releases Activity

use CounterUnit::new instead of a pub member

Browse source
This commit is contained in:
Kai Sickeler 2016-07-17 16:26:11 +02:00
parent 7e4166acbf
commit aeb27072a7
2 changed files with 8 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
imag-counter/src/create.rs
View file

@ -20,7 +20,7 @@ pub fn create(rt: &Runtime) {
.value_of("unit") .value_of("unit")
.unwrap_or("unit"); .unwrap_or("unit");
match Counter::new(rt.store(), String::from(name), init, CounterUnit(String::from(unit))) { match Counter::new(rt.store(), String::from(name), init, CounterUnit::new(unit)) {
Err(e) => { Err(e) => {
warn!("Could not create Counter '{}' with initial value '{} {}'", name, init, unit); warn!("Could not create Counter '{}' with initial value '{} {}'", name, init, unit);
trace_error_exit(&e, 1); trace_error_exit(&e, 1);

8
libimagcounter/src/counter.rs
View file

@ -21,7 +21,7 @@ use error::CounterErrorKind as CEK;
pub type CounterName = String; pub type CounterName = String;
#[derive(Clone, Debug, Eq, PartialEq, Ord, PartialOrd)] #[derive(Clone, Debug, Eq, PartialEq, Ord, PartialOrd)]
pub struct CounterUnit(pub String); pub struct CounterUnit(String);
impl Display for CounterUnit { impl Display for CounterUnit {
fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result { fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
@ -29,6 +29,12 @@ impl Display for CounterUnit {
} }
} }
impl CounterUnit {
pub fn new<S: Into<String>>(unit: S) -> CounterUnit {
CounterUnit(unit.into())
}
}
pub struct Counter<'a> { pub struct Counter<'a> {
fle: FileLockEntry<'a>, fle: FileLockEntry<'a>,
} }