[librsvg: 10/17] ResizeStrategy::Fit - use a Size instead of separate width/height
- From: Marge Bot <marge-bot src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [librsvg: 10/17] ResizeStrategy::Fit - use a Size instead of separate width/height
- Date: Thu, 24 Feb 2022 03:22:56 +0000 (UTC)
commit b376aac1bab5e753fe12281b7e6c5db8e974ff4b
Author: Federico Mena Quintero <federico gnome org>
Date: Mon Feb 21 20:18:25 2022 -0600
ResizeStrategy::Fit - use a Size instead of separate width/height
Part-of: <https://gitlab.gnome.org/GNOME/librsvg/-/merge_requests/669>
src/bin/rsvg-convert.rs | 20 +++++++-------------
1 file changed, 7 insertions(+), 13 deletions(-)
---
diff --git a/src/bin/rsvg-convert.rs b/src/bin/rsvg-convert.rs
index 4057f8496..7cb05db11 100644
--- a/src/bin/rsvg-convert.rs
+++ b/src/bin/rsvg-convert.rs
@@ -103,8 +103,7 @@ impl Size {
enum ResizeStrategy {
Scale(Scale),
Fit {
- width: f64,
- height: f64,
+ size: Size,
keep_aspect_ratio: bool,
},
FitWidth(f64),
@@ -133,10 +132,9 @@ impl ResizeStrategy {
),
ResizeStrategy::Fit {
- width,
- height,
+ size,
keep_aspect_ratio,
- } => (Size::new(width, height), keep_aspect_ratio),
+ } => (size, keep_aspect_ratio),
ResizeStrategy::FitWidth(w) => (
Size::new(w, input.h * w / input.w),
@@ -633,8 +631,7 @@ impl Converter {
// when w and h are specified, but zoom is not, scale to the requested size
(Some(width), Some(height)) if self.zoom.is_identity() => ResizeStrategy::Fit {
- width,
- height,
+ size: Size::new(width, height),
keep_aspect_ratio: self.keep_aspect_ratio,
},
@@ -1220,8 +1217,7 @@ mod sizing_tests {
#[test]
fn fit_non_proportional() {
let strategy = ResizeStrategy::Fit {
- width: 40.0,
- height: 10.0,
+ size: Size::new(40.0, 10.0),
keep_aspect_ratio: false,
};
@@ -1234,8 +1230,7 @@ mod sizing_tests {
#[test]
fn fit_proportional_wider_than_tall() {
let strategy = ResizeStrategy::Fit {
- width: 40.0,
- height: 10.0,
+ size: Size::new(40.0, 10.0),
keep_aspect_ratio: true,
};
@@ -1248,8 +1243,7 @@ mod sizing_tests {
#[test]
fn fit_proportional_taller_than_wide() {
let strategy = ResizeStrategy::Fit {
- width: 100.0,
- height: 50.0,
+ size: Size::new(100.0, 50.0),
keep_aspect_ratio: true,
};
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]