Skip to content

Commit

Permalink
Remove unused (and not useful) QgsRenderContext::constPainter getter
Browse files Browse the repository at this point in the history
Since const QPainters cannot be painted to, there's no value in
leaving this in the API
  • Loading branch information
nyalldawson committed Jan 16, 2017
1 parent cc56c80 commit 48797fa
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 2 deletions.
2 changes: 2 additions & 0 deletions doc/api_break.dox
Original file line number Diff line number Diff line change
Expand Up @@ -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}
---------------------------
Expand Down
1 change: 0 additions & 1 deletion python/core/qgsrendercontext.sip
Original file line number Diff line number Diff line change
Expand Up @@ -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.
Expand Down
1 change: 0 additions & 1 deletion src/core/qgsrendercontext.h
Original file line number Diff line number Diff line change
Expand Up @@ -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.
Expand Down

0 comments on commit 48797fa

Please sign in to comment.