diff --git a/src/providers/ogr/qgsgeopackagedataitems.cpp b/src/providers/ogr/qgsgeopackagedataitems.cpp
index b990200b2ab..07664162666 100644
--- a/src/providers/ogr/qgsgeopackagedataitems.cpp
+++ b/src/providers/ogr/qgsgeopackagedataitems.cpp
@@ -77,7 +77,6 @@ QVector<QgsDataItem *> QgsGeoPackageRootItem::createChildren()
 #ifdef HAVE_GUI
 QList<QAction *> QgsGeoPackageAbstractLayerItem::actions( QWidget * )
 {
-  //dave: decide if more than one layer is selected. if more then "Delete selected Layers"
   QList<QAction *> lst;
 
   const QString deleteText = selectedItems().count() == 1 ? tr( "Delete Layer '%1'…" ).arg( mName )
@@ -501,7 +500,6 @@ void QgsGeoPackageCollectionItem::vacuumGeoPackageDbAction()
 
 void QgsGeoPackageAbstractLayerItem::deleteLayer()
 {
-  //dave: here we are - we need to find out what other layers in the gpkg are selected and delete em
   // Check if the layer(s) are in the registry
   QList<QgsMapLayer *> layersList;
   const auto mapLayers( QgsProject::instance()->mapLayers() );
diff --git a/src/providers/ogr/qgsgeopackagedataitems.h b/src/providers/ogr/qgsgeopackagedataitems.h
index 6b34f98ac63..76c25d43f4c 100644
--- a/src/providers/ogr/qgsgeopackagedataitems.h
+++ b/src/providers/ogr/qgsgeopackagedataitems.h
@@ -28,9 +28,6 @@ class QgsGeoPackageAbstractLayerItem : public QgsLayerItem
 {
     Q_OBJECT
 
-    //! Returns layer uri or empty string if layer cannot be created
-    QString uri() const { return mUri; }
-
   protected:
     QgsGeoPackageAbstractLayerItem( QgsDataItem *parent, const QString &name, const QString &path, const QString &uri, LayerType layerType, const QString &providerKey );