[PATCH] Replace boost::bind by lambdas in canvas/source/directx tree...

Mario Rugiero (via Code Review) gerrit at gerrit.libreoffice.org
Fri Oct 30 23:58:41 PDT 2015


Hi,

I would like you to review the following patch:

    https://gerrit.libreoffice.org/19702

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/02/19702/1

Replace boost::bind by lambdas in canvas/source/directx tree.

Change-Id: I6ecdd97a5054880d3c502f482db0839f8601e295
---
M canvas/source/directx/dx_canvashelper_texturefill.cxx
M canvas/source/directx/dx_textlayout_drawhelper.cxx
2 files changed, 2 insertions(+), 9 deletions(-)



diff --git a/canvas/source/directx/dx_canvashelper_texturefill.cxx b/canvas/source/directx/dx_canvashelper_texturefill.cxx
index c2e0c96..3d5f893 100644
--- a/canvas/source/directx/dx_canvashelper_texturefill.cxx
+++ b/canvas/source/directx/dx_canvashelper_texturefill.cxx
@@ -19,7 +19,6 @@
 
 #include <sal/config.h>
 
-#include <boost/bind.hpp>
 #include <boost/tuple/tuple.hpp>
 
 #include <basegfx/matrix/b2dhommatrix.hxx>
@@ -576,10 +575,7 @@
                     std::transform(&rValues.maColors[0],
                                    &rValues.maColors[0]+rValues.maColors.getLength(),
                                    aColors.begin(),
-                                   boost::bind(
-                                       (Gdiplus::ARGB (*)( const uno::Sequence< double >& ))(
-                                           &tools::sequenceToArgb),
-                                       _1));
+                                   [](const uno::Sequence< double >& aDoubleSequence) { return tools::sequenceToArgb(aDoubleSequence); } );
                     std::vector< Gdiplus::REAL > aStops;
                     comphelper::sequenceToContainer(aStops,rValues.maStops);
 
diff --git a/canvas/source/directx/dx_textlayout_drawhelper.cxx b/canvas/source/directx/dx_textlayout_drawhelper.cxx
index 55c267d..3db3996 100644
--- a/canvas/source/directx/dx_textlayout_drawhelper.cxx
+++ b/canvas/source/directx/dx_textlayout_drawhelper.cxx
@@ -20,7 +20,6 @@
 #include <sal/config.h>
 
 #include <memory>
-#include <boost/bind.hpp>
 
 #include <basegfx/polygon/b2dpolypolygon.hxx>
 #include <basegfx/tools/canvastools.hxx>
@@ -80,9 +79,7 @@
 
         // issue an ReleaseHDC() when leaving the scope
         const ::comphelper::ScopeGuard aGuard(
-            boost::bind( &Gdiplus::Graphics::ReleaseHDC,
-                         rGraphics.get(),
-                         hdc ));
+            [&rGraphics, &hdc]() mutable { rGraphics->ReleaseHDC(hdc); } );
 
         SystemGraphicsData aSystemGraphicsData;
         aSystemGraphicsData.nSize = sizeof(SystemGraphicsData);

-- 
To view, visit https://gerrit.libreoffice.org/19702
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I6ecdd97a5054880d3c502f482db0839f8601e295
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Mario Rugiero <mrugiero at gmail.com>
Gerrit-Reviewer: Anonymous Coward #1000450
Gerrit-Reviewer: Anonymous Coward #1000808
Gerrit-Reviewer: Gerrit Buildbot for LibreOffice
Gerrit-Reviewer: Jenkins <ci at libreoffice.org>
Gerrit-Reviewer: Joel Madero <jmadero.dev at gmail.com>
Gerrit-Reviewer: LibreOffice gerrit bot <gerrit at libreoffice.org>
Gerrit-Reviewer: Mario Rugiero <mrugiero at gmail.com>
Gerrit-Reviewer: Tinderbox 21 - Mac <tb21 at shmget.com>
Gerrit-Reviewer: libreoffice buildbot1 <lobuilbot1 at gmail.com>
Gerrit-Reviewer: libreoffice buildbot2 <libreoffice at lists.freedesktop.org>
Gerrit-Reviewer: tb-37 MacOSX tdf <tb37 at documentfoundation.org>
Gerrit-Reviewer: tb16 <tb16 at libreoffice.org>
Gerrit-Reviewer: tb20 at libreoffice.org
Gerrit-Reviewer: tb2 at shmget.com
Gerrit-Reviewer: tb33 <tb33 at libreoffice.org>
Gerrit-Reviewer: tb38 at libreoffice.org
Gerrit-Reviewer: tb39 <tb39 at libreoffice.org>
Gerrit-Reviewer: tb42 <tb42 at libreoffice.org>
Gerrit-Reviewer: tb45 at libreoffice.org
Gerrit-Reviewer: test_jenkins_mac at libreoffice.org


More information about the LibreOffice mailing list