Two more replacements for state compare logic

Author: andy31415

src/app/util/ember-compatibility-functions.cpp

@@ -430,8 +430,13 @@ CHIP_ERROR ReadViaAccessInterface(SubjectDescriptor subjectDescriptor, bool aIsF
                                   AttributeEncodeState * aEncoderState, AttributeAccessInterface * aAccessInterface,
                                   bool * aTriedEncode)
 {
-    AttributeEncodeState state = (aEncoderState == nullptr ? AttributeEncodeState() : *aEncoderState);
-    DataVersion version        = 0;
+    AttributeEncodeState state;
+    if (aEncoderState != nullptr)
+    {
+        state = *aEncoderState;
+    }
+
+    DataVersion version = 0;
     ReturnErrorOnFailure(ReadClusterDataVersion(aPath, version));
     AttributeValueEncoder valueEncoder(aAttributeReports, subjectDescriptor, aPath, version, aIsFabricFiltered, state);
     CHIP_ERROR err = aAccessInterface->Read(aPath, valueEncoder);

src/app/util/mock/attribute-storage.cpp

@@ -350,7 +350,11 @@ CHIP_ERROR ReadSingleMockClusterData(FabricIndex aAccessingFabricIndex, const Co
     // Attribute 4 acts as a large attribute to trigger chunking.
     if (aPath.mAttributeId == MockAttributeId(4))
     {
-        AttributeEncodeState state = (apEncoderState == nullptr ? AttributeEncodeState() : *apEncoderState);
+        AttributeEncodeState state;
+        if (apEncoderState != nullptr)
+        {
+            state = *apEncoderState;
+        }
         Access::SubjectDescriptor subject;
         subject.fabricIndex = aAccessingFabricIndex;