Avoid unnecessary calculations in SVG fill symbol cloning

nyalldawson · nyalldawson · commit f0218316f650 · 2019-05-14T20:45:46.000+10:00
Minor speed boost
diff --git a/src/core/symbology/qgsfillsymbollayer.cpp b/src/core/symbology/qgsfillsymbollayer.cpp
@@ -1724,11 +1724,10 @@ QgsSVGFillSymbolLayer::QgsSVGFillSymbolLayer( const QString &svgFilePath, double
   : QgsImageFillSymbolLayer()
   , mPatternWidth( width )
 {
-  setSvgFilePath( svgFilePath );
   mStrokeWidth = 0.3;
   mAngle = angle;
   mColor = QColor( 255, 255, 255 );
-  setDefaultSvgParams();
+  setSvgFilePath( svgFilePath );
 }
 
 QgsSVGFillSymbolLayer::QgsSVGFillSymbolLayer( const QByteArray &svgData, double width, double angle )

Original file line number	Diff line number	Diff line change
`@@ -1724,11 +1724,10 @@ QgsSVGFillSymbolLayer::QgsSVGFillSymbolLayer( const QString &svgFilePath, double`
`1724`	`1724`	`: QgsImageFillSymbolLayer()`
`1725`	`1725`	`, mPatternWidth( width )`
`1726`	`1726`	`{`
`1727`		`- setSvgFilePath( svgFilePath );`
`1728`	`1727`	`mStrokeWidth = 0.3;`
`1729`	`1728`	`mAngle = angle;`
`1730`	`1729`	`mColor = QColor( 255, 255, 255 );`
`1731`		`- setDefaultSvgParams();`
	`1730`	`+ setSvgFilePath( svgFilePath );`
`1732`	`1731`	`}`
`1733`	`1732`
`1734`	`1733`	`QgsSVGFillSymbolLayer::QgsSVGFillSymbolLayer( const QByteArray &svgData, double width, double angle )`