[PATCH 0/1] MR3521: sane.ds: Do the reverse convertion of frame edges before setting.

[Ilia Docin (＠IDocin)]

Sane backends must attach a unit of either pixels or millimeters to scan area options.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/3521

[PATCH 1/1] sane.ds: Do the reverse convertion of frame edges before setting.

[Ilia Docin]

From: Ilia Docin <ilya.docin(a)contentai.ru>

Sane backends must attach a unit of either pixels or millimeters to scan area options.
---
 dlls/sane.ds/ds_image.c | 8 ++++----
 dlls/sane.ds/options.c  | 5 +++++
 dlls/sane.ds/sane_i.h   | 1 +
 3 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/dlls/sane.ds/ds_image.c b/dlls/sane.ds/ds_image.c
index b2fad94c848..93193abc69f 100644
--- a/dlls/sane.ds/ds_image.c
+++ b/dlls/sane.ds/ds_image.c
@@ -164,10 +164,10 @@ TW_UINT16 SANE_ImageLayoutSet (pTW_IDENTITY pOrigin,
             img->Frame.Right.Whole, img->Frame.Right.Frac,
             img->Frame.Bottom.Whole, img->Frame.Bottom.Frac);
 
-    tlx = img->Frame.Left.Whole   * 65536 + img->Frame.Left.Frac;
-    tly = img->Frame.Top.Whole    * 65536 + img->Frame.Top.Frac;
-    brx = img->Frame.Right.Whole  * 65536 + img->Frame.Right.Frac;
-    bry = img->Frame.Bottom.Whole * 65536 + img->Frame.Bottom.Frac;
+    tlx = convert_twain_res_to_sane( img->Frame.Left );
+    tly = convert_twain_res_to_sane( img->Frame.Top );
+    brx = convert_twain_res_to_sane( img->Frame.Right );
+    bry = convert_twain_res_to_sane( img->Frame.Bottom );
 
     twrc = sane_option_set_scan_area( tlx, tly, brx, bry, &changed );
     if (twrc != TWRC_SUCCESS)
diff --git a/dlls/sane.ds/options.c b/dlls/sane.ds/options.c
index 30ca441fd28..fabbb554744 100644
--- a/dlls/sane.ds/options.c
+++ b/dlls/sane.ds/options.c
@@ -200,6 +200,11 @@ TW_FIX32 convert_sane_res_to_twain(int res)
     return value;
 }
 
+int convert_twain_res_to_sane( TW_FIX32 res )
+{
+    return MulDiv( res.Whole * 65536 + res.Frac, 254, 10 );  /* inch -> mm */
+}
+
 TW_UINT16 get_sane_params( struct frame_parameters *params )
 {
     return SANE_CALL( get_params, params );
diff --git a/dlls/sane.ds/sane_i.h b/dlls/sane.ds/sane_i.h
index 571917944e1..685ccbe0814 100644
--- a/dlls/sane.ds/sane_i.h
+++ b/dlls/sane.ds/sane_i.h
@@ -208,6 +208,7 @@ TW_UINT16 sane_option_get_scan_area( int *tlx, int *tly, int *brx, int *bry ) DE
 TW_UINT16 sane_option_get_max_scan_area( int *tlx, int *tly, int *brx, int *bry ) DECLSPEC_HIDDEN;
 TW_UINT16 sane_option_set_scan_area( int tlx, int tly, int brx, int bry, BOOL *reload ) DECLSPEC_HIDDEN;
 TW_FIX32 convert_sane_res_to_twain( int res ) DECLSPEC_HIDDEN;
+int convert_twain_res_to_sane( TW_FIX32 res ) DECLSPEC_HIDDEN;
 TW_UINT16 get_sane_params( struct frame_parameters *params ) DECLSPEC_HIDDEN;
 
 #endif
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/3521

[PATCH 0/1] MR3521: sane.ds: Do the reverse convertion of frame edges before setting. - approved

[Esme Povirk (＠madewokherd)]

This merge request was approved by Esme Povirk.
-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/3521