diff --git a/src/app/codegen-data-model-provider/CodegenDataModelProvider_Write.cpp b/src/app/codegen-data-model-provider/CodegenDataModelProvider_Write.cpp index 15cd60ccb6cd0b..9eee57d1491fa5 100644 --- a/src/app/codegen-data-model-provider/CodegenDataModelProvider_Write.cpp +++ b/src/app/codegen-data-model-provider/CodegenDataModelProvider_Write.cpp @@ -331,15 +331,15 @@ DataModel::ActionReturnStatus CodegenDataModelProvider::WriteAttribute(const Dat if (checkAcl && request.previousSuccessPath.has_value()) { // NOTE: explicit cast/check only for attribute path and nothing else. - // + // // In particular `request.path` is a DATA path (contains a list index) - // and we do not want request.previousSuccessPath to be auto-cast to a + // and we do not want request.previousSuccessPath to be auto-cast to a // data path with a empty list and fail the compare. // // This could be `request.previousSuccessPath != request.path` (where order - // is important) however that would seem more brittle (relying that a != b + // is important) however that would seem more brittle (relying that a != b // behaves differently than b != a due to casts). Overall Data paths are not - // the same as attribute paths. + // the same as attribute paths. // // Also note that Concrete path have a mExpanded that is not used in compares. const ConcreteAttributePath & attributePathA = request.path;