From b6961e8f039f7d0dc0297b37f12437455095caa9 Mon Sep 17 00:00:00 2001
From: Ichthyostega <prg@ichthyostega.de>
Date: Sat, 13 Jan 2018 00:58:08 +0100
Subject: [PATCH] DockAccess: better pass functor as const& into partial
 application

seems to be the most orthogonal way to strip adornments from the SIG type
Moreover, we want to move the functor into the closure, where it will be stored anyay.
From there on, we can pass as const& into the binder (for creating the partially closed functor)
---
 src/gui/interact/view-spec-dsl.hpp |  6 +++---
 src/lib/meta/function-closure.hpp  |  4 ++--
 wiki/thinkPad.ichthyo.mm           | 18 +++++++++++-------
 3 files changed, 16 insertions(+), 12 deletions(-)

diff --git a/src/gui/interact/view-spec-dsl.hpp b/src/gui/interact/view-spec-dsl.hpp
index 1e486cec1..63e18197c 100644
--- a/src/gui/interact/view-spec-dsl.hpp
+++ b/src/gui/interact/view-spec-dsl.hpp
@@ -177,7 +177,7 @@ namespace interact {
     {
       template<class FUN>
       static auto
-      buildPartialApplicator (FUN fun)
+      buildPartialApplicator (FUN&& fun)
         {
           using lib::meta::_Fun;
           using lib::meta::Split;
@@ -202,13 +202,13 @@ namespace interact {
           return [=](auto&&... args) -> Allocator
                     {
                       ArgTuple params {forward<decltype(args)> (args)...};
-                      return PApply<FUN const&,FurtherArgs>::bindBack (fun, params);
+                      return PApply<FUN,FurtherArgs>::bindBack (fun, params);
                     };
         }
       
     public:
       template<class FUN>
-      AllocSpec(FUN fun)
+      AllocSpec(FUN&& fun)
         : std::function<Allocator(ARGS&&...)> (buildPartialApplicator (forward<FUN> (fun)))
         { }
     };
diff --git a/src/lib/meta/function-closure.hpp b/src/lib/meta/function-closure.hpp
index 24d5ec433..564c77de2 100644
--- a/src/lib/meta/function-closure.hpp
+++ b/src/lib/meta/function-closure.hpp
@@ -620,7 +620,7 @@ namespace func{
        *          closed arguments; on invocation, only the remaining arguments need to be supplied.
        */
       static LeftReducedFunc
-      bindFront (SIG& f, Tuple<ValTypes> const& arg)
+      bindFront (SIG const& f, Tuple<ValTypes> const& arg)
         {
           LeftReplacedArgs params {BuildL(arg)};
           return func::Apply<ARG_CNT>::template bind<LeftReducedFunc> (f, params);
@@ -635,7 +635,7 @@ namespace func{
        *          closed arguments; on invocation, only the remaining arguments need to be supplied.
        */
       static RightReducedFunc
-      bindBack (SIG& f, Tuple<ValTypes> const& arg)
+      bindBack (SIG const& f, Tuple<ValTypes> const& arg)
         {
           RightReplacedArgs params {BuildR(arg)};
           return func::Apply<ARG_CNT>::template bind<RightReducedFunc> (f, params);
diff --git a/wiki/thinkPad.ichthyo.mm b/wiki/thinkPad.ichthyo.mm
index b7b029335..8ed87bf44 100644
--- a/wiki/thinkPad.ichthyo.mm
+++ b/wiki/thinkPad.ichthyo.mm
@@ -9098,8 +9098,7 @@
       n&#228;mlich in lib::meta::func::PApply::bindBack
     </p>
   </body>
-</html>
-</richcontent>
+</html></richcontent>
 </node>
 <node CREATED="1515725834771" ID="ID_403863327" MODIFIED="1515725871768" TEXT="variadisch Template-Argumente">
 <icon BUILTIN="messagebox_warning"/>
@@ -9147,8 +9146,7 @@
       };
     </p>
   </body>
-</html>
-</richcontent>
+</html></richcontent>
 <icon BUILTIN="idea"/>
 </node>
 <node COLOR="#ef5d2b" CREATED="1515726754041" ID="ID_315893428" LINK="http://en.cppreference.com/w/cpp/language/lambda" MODIFIED="1515726793618" TEXT="cppreference..."/>
@@ -9157,7 +9155,8 @@
 <node CREATED="1515726268721" ID="ID_243053845" MODIFIED="1515726329213" TEXT="welches intern PApply aufruft"/>
 </node>
 <node CREATED="1515732350136" ID="ID_1294398139" MODIFIED="1515732355403" TEXT="technisch....">
-<node BACKGROUND_COLOR="#fdfdcf" COLOR="#ff0000" CREATED="1515732356319" ID="ID_1071937152" MODIFIED="1515798985447" TEXT="Problem: generisches Lambda in std::function binden">
+<node COLOR="#a41e57" CREATED="1515732356319" HGAP="25" ID="ID_1071937152" MODIFIED="1515800950319" TEXT="Problem: generisches Lambda in std::function binden">
+<font ITALIC="true" NAME="SansSerif" SIZE="12"/>
 <icon BUILTIN="broken-line"/>
 </node>
 <node CREATED="1515732383331" ID="ID_1688687565" MODIFIED="1515797049652">
@@ -9170,8 +9169,7 @@
       Vermutung: mu&#223; Lambda <b>instantiieren</b>...
     </p>
   </body>
-</html>
-</richcontent>
+</html></richcontent>
 <icon BUILTIN="button_cancel"/>
 </node>
 <node CREATED="1515797030493" ID="ID_973074141" MODIFIED="1515799063868" TEXT="Nein: es ist wohl ehr der R&#xfc;ckgabetyp">
@@ -9182,6 +9180,12 @@
 <icon BUILTIN="idea"/>
 </node>
 </node>
+<node COLOR="#338800" CREATED="1515801302955" ID="ID_208115762" MODIFIED="1515801394951" TEXT="Argument-&#xdc;bergabe">
+<icon BUILTIN="button_ok"/>
+<node CREATED="1515801313185" ID="ID_1411407565" MODIFIED="1515801322004" TEXT="Funktor bis in die Closure per Move"/>
+<node CREATED="1515801324016" ID="ID_814347755" MODIFIED="1515801353352" TEXT="Closure an PApply per const&amp;  --&gt;  Binder erzeugen"/>
+<node CREATED="1515801355124" ID="ID_1735797839" MODIFIED="1515801393059" TEXT="Lambda: Argumente in das Argument-Tupel per Move"/>
+</node>
 </node>
 </node>
 </node>