Delete equality operator of QgsMapLayerRef and make its bool operator…

… explicit

This will avoid implicit conversion to bool when comparing QList<QgsMapLayerRef>s in the future.

src/3d/qgs3dmapsettings.cpp

@@ -31,6 +31,7 @@
 #include "qgsrasterlayer.h"
 #include "qgspointlightsettings.h"
 #include "qgsdirectionallightsettings.h"
+#include "qgsmaplayerlistutils_p.h"
 
 #include <QDomDocument>
 #include <QDomElement>
@@ -546,17 +547,12 @@ double Qgs3DMapSettings::terrainVerticalScale() const
 
 void Qgs3DMapSettings::setLayers( const QList<QgsMapLayer *> &layers )
 {
-  QList<QgsMapLayerRef> lst;
-  lst.reserve( layers.count() );
-  for ( QgsMapLayer *layer : layers )
-  {
-    lst.append( layer );
-  }
+  const QList<QgsMapLayer *> raw = _qgis_listRefToRaw( mLayers );
 
-  if ( mLayers == lst )
+  if ( layers == raw )
     return;
 
-  mLayers = lst;
+  mLayers = _qgis_listRawToRef( layers );
   emit layersChanged();
 }
 

src/core/qgsmaplayerref.h

@@ -85,11 +85,16 @@ struct _LayerRef
     provider = l && l->dataProvider() ? l->dataProvider()->name() : QString();
   }
 
+  /**
+   * Equality operator is deleted to avoid confusion as there are multiple ways two _LayerRef objects can be considered equal.
+   */
+  bool operator==( const _LayerRef &other ) = delete;
+
   /**
    * Returns TRUE if the layer reference is resolved and contains a reference to an existing
    * map layer.
    */
-  operator bool() const
+  explicit operator bool() const
   {
     return static_cast< bool >( layer.data() );
   }