able/qrcode-rust
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Merge pull request #22 from aignas/TODO-fix-1

Browse source 
Fix TODO: use sum insted of left fold
This commit is contained in:
kennytm 2018-11-05 10:12:29 +08:00 committed by GitHub
parent 9607069265 594724759c
commit 820ebfba0e
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/optimize.rs
View file

@ -332,8 +332,9 @@ impl<I: Iterator<Item = Segment>> Iterator for Optimizer<I> {
/// Computes the total encoded length of all segments. /// Computes the total encoded length of all segments.
pub fn total_encoded_len(segments: &[Segment], version: Version) -> usize { pub fn total_encoded_len(segments: &[Segment], version: Version) -> usize {
// TODO revert to `.map().sum()` after `sum()` is stable. segments.iter()
segments.iter().fold(0, |acc, seg| acc + seg.encoded_len(version)) .map(|seg| seg.encoded_len(version))
.sum()
} }
#[cfg(test)] #[cfg(test)]