From 757af312d2828928d9b38d64f27f047f156b8c99 Mon Sep 17 00:00:00 2001
From: Hombre <natureh.510@gmail.com>
Date: Tue, 19 Mar 2019 02:02:12 +0100
Subject: [PATCH 1/4] Suppress 'rtwindow.h' from main-cli.cc

---
 rtgui/main-cli.cc | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/rtgui/main-cli.cc b/rtgui/main-cli.cc
index ddd42eb6a..cbea1d477 100644
--- a/rtgui/main-cli.cc
+++ b/rtgui/main-cli.cc
@@ -28,15 +28,17 @@
 #include <giomm.h>
 #include <iostream>
 #include <tiffio.h>
-#include "rtwindow.h"
 #include <cstring>
 #include <cstdlib>
 #include <locale.h>
+#include "../rtengine/procparams.h"
+#include "../rtengine/profilestore.h"
 #include "options.h"
 #include "soundman.h"
 #include "rtimage.h"
 #include "version.h"
 #include "extprog.h"
+#include "pathutils.h"
 
 #ifndef WIN32
 #include <glibmm/fileutils.h>
@@ -44,6 +46,8 @@
 #include <glib/gstdio.h>
 #include <glibmm/threads.h>
 #else
+#include <windows.h>
+#include <shlobj.h>
 #include <glibmm/thread.h>
 #include "conio.h"
 #endif

From d4d535720db237bd26069719efee51faac3ed39d Mon Sep 17 00:00:00 2001
From: heckflosse <heckflosse67@gmx.de>
Date: Tue, 19 Mar 2019 15:09:18 +0100
Subject: [PATCH 2/4] Don't apply histogram matching when thumb has less than
 19200 pixels

---
 rtengine/histmatching.cc | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/rtengine/histmatching.cc b/rtengine/histmatching.cc
index 3d1eb1657..48969368f 100644
--- a/rtengine/histmatching.cc
+++ b/rtengine/histmatching.cc
@@ -289,7 +289,10 @@ void RawImageSource::getAutoMatchedToneCurve(const ColorManagementParams &cp, st
             histMatchingCache = outCurve;
             *histMatchingParams = cp;
             return;
-        } else if (w * 32 < fw) {
+        } else if (w * 32 < fw || w * h < 19200) {
+             // Some cameras have extremely small thumbs, for example Canon PowerShot A3100 IS has 128x96 thumbs.
+             // For them we skip histogram matching.
+             // With 160x120 thumbs from RICOH GR DIGITAL 2 it works fine, so we use 19200 as limit.
             if (settings->verbose) {
                 std::cout << "histogram matching: the embedded thumbnail is too small: " << w << "x" << h << std::endl;
             }

From 28b785682a308447f512536a4fc6cf7da93f4ef2 Mon Sep 17 00:00:00 2001
From: Ingo Weyrich <heckflosse@users.noreply.github.com>
Date: Tue, 19 Mar 2019 17:43:42 +0100
Subject: [PATCH 3/4] Allow histogram matching for YI M1 Camera (160x120
 thumbs)

---
 rtengine/histmatching.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/rtengine/histmatching.cc b/rtengine/histmatching.cc
index 48969368f..f91086285 100644
--- a/rtengine/histmatching.cc
+++ b/rtengine/histmatching.cc
@@ -289,7 +289,7 @@ void RawImageSource::getAutoMatchedToneCurve(const ColorManagementParams &cp, st
             histMatchingCache = outCurve;
             *histMatchingParams = cp;
             return;
-        } else if (w * 32 < fw || w * h < 19200) {
+        } else if (w * 33 < fw || w * h < 19200) {
              // Some cameras have extremely small thumbs, for example Canon PowerShot A3100 IS has 128x96 thumbs.
              // For them we skip histogram matching.
              // With 160x120 thumbs from RICOH GR DIGITAL 2 it works fine, so we use 19200 as limit.

From d059caa7de9b923feb5a2e00c62970d519420442 Mon Sep 17 00:00:00 2001
From: Hombre <natureh.510@gmail.com>
Date: Tue, 19 Mar 2019 20:45:12 +0100
Subject: [PATCH 4/4] Enhancing RTSurface: 'surface' now private

---
 rtgui/edit.cc      | 2 +-
 rtgui/lwbutton.cc  | 2 +-
 rtgui/rtsurface.cc | 5 +++++
 rtgui/rtsurface.h  | 7 ++++++-
 4 files changed, 13 insertions(+), 3 deletions(-)

diff --git a/rtgui/edit.cc b/rtgui/edit.cc
index eb91d3fda..c59c2dc04 100644
--- a/rtgui/edit.cc
+++ b/rtgui/edit.cc
@@ -896,7 +896,7 @@ void OPIcon::drawImage(Cairo::RefPtr<RTSurface> &img,
     rtengine::Coord tl, br; // Coordinate of the rectangle in the CropBuffer coordinate system
     drivenPointToRectangle(pos, tl, br, imgW, imgH);
 
-    cr->set_source(img->surface, tl.x, tl.y);
+    cr->set_source(img->get(), tl.x, tl.y);
     cr->set_line_width(0.);
     cr->rectangle(tl.x, tl.y, imgW, imgH);
     cr->fill();
diff --git a/rtgui/lwbutton.cc b/rtgui/lwbutton.cc
index b1ed940c2..48843a02e 100644
--- a/rtgui/lwbutton.cc
+++ b/rtgui/lwbutton.cc
@@ -205,7 +205,7 @@ void LWButton::redraw (Cairo::RefPtr<Cairo::Context> context)
     }
 
     if (icon) {
-        context->set_source (icon->surface, xpos + dilat, ypos + dilat);
+        context->set_source (icon->get(), xpos + dilat, ypos + dilat);
         context->paint ();
     }
 }
diff --git a/rtgui/rtsurface.cc b/rtgui/rtsurface.cc
index faf1a750d..9f692878d 100644
--- a/rtgui/rtsurface.cc
+++ b/rtgui/rtsurface.cc
@@ -132,3 +132,8 @@ bool RTSurface::hasSurface() const
 {
     return surface ? true : false;
 }
+
+const Cairo::RefPtr<Cairo::ImageSurface>& RTSurface::get() const
+{
+    return surface;
+}
diff --git a/rtgui/rtsurface.h b/rtgui/rtsurface.h
index 58093d323..d55506269 100644
--- a/rtgui/rtsurface.h
+++ b/rtgui/rtsurface.h
@@ -26,12 +26,15 @@
  */
 class RTSurface : public RTScalable
 {
+
+private:
+
     static double dpiBack; // used to keep track of master dpi change
     static int scaleBack;  // used to keep track of master scale change
+    Cairo::RefPtr<Cairo::ImageSurface> surface;
     void changeImage (Glib::ustring imageName);
 
 public:
-    Cairo::RefPtr<Cairo::ImageSurface> surface;
 
     RTSurface ();
     RTSurface (const RTSurface&  other);
@@ -42,6 +45,8 @@ public:
     int getHeight() const;
     bool hasSurface() const;
 
+    const Cairo::RefPtr<Cairo::ImageSurface>& get() const;
+
     static void init();
     static void updateImages ();
     static void setDPInScale (const double newDPI, const int newScale);