Remove unused (and not useful) QgsRenderContext::constPainter getter

Since const QPainters cannot be painted to, there's no value in leaving this in the API
qgis · Jan 16, 2017 · 48797fa · 48797fa
1 parent cc56c80
commit 48797fa
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 2 deletions.
diff --git a/doc/api_break.dox b/doc/api_break.dox
@@ -1533,6 +1533,8 @@ QgsRenderContext        {#qgis_api_break_3_0_QgsRenderContext}
 be returned instead of a null pointer if no transformation is required.
 - setCoordinateTransform() now takes a QgsCoordinateTransform reference, not a pointer. An invalid QgsCoordinateTransform should be used instead of a null pointer if no transformation is required.
 - rasterScaleFactor() and setRasterScaleFactor() were removed. In QGIS 3.0 QPainter destinations should always be constructed so that 1 painter unit = 1 pixel.
+- The constPainter() getter was removed. Const QPainters cannot be painted to or modified, so this
+method was of little use.
 
 QgsRendererRangeLabelFormat        {#qgis_api_break_3_0_QgsRendererRangeLabelFormat}
 ---------------------------

diff --git a/python/core/qgsrendercontext.sip b/python/core/qgsrendercontext.sip
@@ -62,7 +62,6 @@ class QgsRenderContext
     //getters
 
     QPainter* painter();
-    const QPainter* constPainter() const;
 
     /** Returns the current coordinate transform for the context, or an invalid
      * transform is no coordinate transformation is required.

diff --git a/src/core/qgsrendercontext.h b/src/core/qgsrendercontext.h
@@ -105,7 +105,6 @@ class CORE_EXPORT QgsRenderContext
      * @see setPainter()
     */
     QPainter* painter() {return mPainter;}
-    const QPainter* constPainter() const { return mPainter; }
 
     /** Returns the current coordinate transform for the context, or an invalid
      * transform is no coordinate transformation is required.