<div dir="ltr">Dear list,<div><br></div><div>A small warning that I am about to merge PR #9437</div><div><a href="https://github.com/qgis/QGIS/pull/9437">https://github.com/qgis/QGIS/pull/9437</a><br></div><div><br></div><div>This pull request adds a new scope based enum for map layer types.</div><div>In other words, QgsMapLayer::LayerType is now QgsMapLayerType.</div><div><br></div><div>This change has been motivated by:</div><div>- scope based enums are nicely displayed in the PyQGIS docs (see <a href="https://qgis.org/pyqgis/master/gui/QgsAdvancedDigitizingDockWidget.html#qgis.gui.QgsAdvancedDigitizingDockWidget.AdditionalConstraint">https://qgis.org/pyqgis/master/gui/QgsAdvancedDigitizingDockWidget.html#qgis.gui.QgsAdvancedDigitizingDockWidget.AdditionalConstraint</a> )</div><div>- taking the enum outside of the QgsMapLayer class:</div><div>  * allows to use forward declarations of the enum (and not including the whole qgsmaplayer.h file)</div><div>  * avoid the rather long call: QgsMapLayer::LayerType::VectorLayer (mandatory with scope based enums)</div><div><br></div><div>This means that current "green" pull request **might** actually need a rebase and retest.</div><div><br></div><div>Regarding Python API compatibility, it is kept by using monkey patching. So no worries on this side.</div><div><br></div><div>This follows a former merged PR that introduced a scope based enum, but this one was kept in class</div><div><a href="https://github.com/qgis/QGIS/pull/9425">https://github.com/qgis/QGIS/pull/9425</a><br></div><div><br></div><div>So, basically to be nicely displayed in the PyQGIS API docs, all enums shall be converted to scope based enums. Then, deciding to take them out of class or not depends a bit if the enum is meant to be mainly used within the class or not.</div><div><br></div><div>Thanks for reading, best wishes</div><div>And have a good rebase :)</div><div><br></div><div>Denis</div><div><br></div><div><br></div></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">





<p class="inbox-inbox-inbox-inbox-p1"><span style="color:rgb(0,0,0);font-family:Verdana,sans-serif;font-size:10pt">Denis Rouzaud</span><br style="color:rgb(0,0,0);font-family:Times;font-size:medium"><a href="mailto:denis@opengis.ch" target="_blank" style="font-family:Times;font-size:medium"><span style="color:rgb(0,0,0);font-family:Verdana,sans-serif;font-size:8pt">denis@opengis.ch</span> </a><br style="color:rgb(0,0,0);font-family:Times;font-size:medium"><span style="color:rgb(0,0,0);font-family:Verdana,sans-serif;font-size:8pt"><a>+41 76 370 21 22</a></span></p><p class="inbox-inbox-inbox-inbox-p1"><span style="color:rgb(0,0,0);font-family:Verdana,sans-serif;font-size:8pt"><a><br></a></span></p></div></div>