Updated the Service Area code documentation following a review comment. Move the IsProgressElement to the delegate and added a default implementation.

Author: hicklin

src/app/clusters/service-area-server/service-area-cluster-objects.h

@@ -35,9 +35,6 @@ inline constexpr size_t kMapNameMaxSize      = 64u;
  */
 struct LocationStructureWrapper : public chip::app::Clusters::ServiceArea::Structs::LocationStruct::Type
 {
-    /**
-     * @brief This is a default constructor that initializes the location object with default values.
-     */
     LocationStructureWrapper()
     {
         Set(0, 0, CharSpan(), DataModel::Nullable<int16_t>(), DataModel::Nullable<AreaTypeTag>(),
@@ -274,9 +271,6 @@ struct LocationStructureWrapper : public chip::app::Clusters::ServiceArea::Struc
  */
 struct MapStructureWrapper : public chip::app::Clusters::ServiceArea::Structs::MapStruct::Type
 {
-    /**
-     * @brief This is a default constructor that initializes the map object with default values.
-     */
     MapStructureWrapper() { Set(0, CharSpan()); }
 
     /**

src/app/clusters/service-area-server/service-area-delegate.cpp

@@ -24,13 +24,6 @@ bool Delegate::GetSupportedLocationById(uint32_t aLocationId, uint32_t & listInd
     return ret_value;
 };
 
-/**
- * When we cannot guarantee that the restrictions imposed on the SupportedLocations, CurrentLocation and Progress attributes
- * can be upheld, these attributes are set to null.
- *
- * The user is free the redefine this method as their device may have more information on what has changed and may be able to
- * maintain the restrictions on these attributes by selectively editing them.
- */
 void Delegate::HandleSupportedLocationsUpdated()
 {
     mInstance->ClearSelectedLocations();
@@ -99,3 +92,11 @@ bool Delegate::GetProgressElementById(uint32_t aLocationId, uint32_t & listIndex
 
     return ret_value;
 };
+
+bool Delegate::IsProgressElement(uint32_t aLocationId)
+{
+    uint32_t index;
+    Structs::ProgressStruct::Type progressElement;
+
+    return GetProgressElementById(aLocationId, index, progressElement);
+}