Merge pull request #177 from quininer/cjk

Fix TextCellContents cjk width
This commit is contained in:
Benjamin Sago 2017-05-17 21:18:17 +01:00 committed by GitHub
commit 5bec218878
2 changed files with 11 additions and 6 deletions

View File

@ -1,5 +1,6 @@
//! The `TextCell` type for the details and lines views.
use std::iter::Sum;
use std::ops::{Add, Deref, DerefMut};
use ansi_term::{Style, ANSIString, ANSIStrings};
@ -163,11 +164,9 @@ impl TextCellContents {
/// Calculates the width that a cell with these contents would take up, by
/// counting the number of characters in each unformatted ANSI string.
pub fn width(&self) -> DisplayWidth {
let sum = self.0.iter()
.map(|anstr| anstr.chars().count())
.sum();
DisplayWidth(sum)
self.0.iter()
.map(|anstr| DisplayWidth::from(anstr.deref()))
.sum()
}
/// Promotes these contents to a full cell containing them alongside
@ -239,6 +238,12 @@ impl Add<usize> for DisplayWidth {
}
}
impl Sum for DisplayWidth {
fn sum<I>(iter: I) -> Self where I: Iterator<Item=Self> {
iter.fold(DisplayWidth(0), Add::add)
}
}
#[cfg(test)]
mod width_unit_test {