Apply suggestions from code review

Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>
Co-authored-by: Kiel Oleson <kielo@apple.com>

Author: 3 people

src/app/clusters/thread-network-directory-server/thread-network-directory-server.cpp

@@ -119,7 +119,7 @@ CHIP_ERROR ThreadNetworkDirectoryServer::WritePreferredExtendedPanId(const Concr
 
     // "A zero-length value SHALL be allowed for nullable values ... and SHALL have the same semantics as the null value."
     ByteSpan value = nullableValue.ValueOr(ByteSpan());
-    // Ensure the provided value is value (correct size) and refers to PAN from the list.
+    // Ensure the provided value is valid (correct size) and refers to PAN from the list.
     VerifyOrReturnError(value.empty() || (value.size() == ExtendedPanId::size() && mStorage.ContainsNetwork(ExtendedPanId(value))),
                         StatusIB(IMStatus::ConstraintError).ToChipError());
 

src/app/tests/TestDefaultThreadNetworkDirectoryStorage.cpp

@@ -35,8 +35,8 @@ class TestDefaultThreadNetworkDirectoryStorage : public ::testing::Test
     static void TearDownTestSuite() { chip::Platform::MemoryShutdown(); }
 
 protected:
-    TestPersistentStorageDelegate persistendStorageDelegate;
-    DefaultThreadNetworkDirectoryStorage storageImpl{ persistendStorageDelegate };
+    TestPersistentStorageDelegate persistentStorageDelegate;
+    DefaultThreadNetworkDirectoryStorage storageImpl{ persistentStorageDelegate };
     ThreadNetworkDirectoryStorage & storage = storageImpl;
 };
 

src/lib/support/ThreadOperationalDataset.cpp

@@ -39,7 +39,7 @@ namespace Thread {
 class ThreadTLV final
 {
     static constexpr uint8_t kLengthEscape = 0xff; ///< This length value indicates the actual length is of two-bytes length, which
-                                                   ///< not allowed in Thread Operational Dataset TLVs.
+                                                   ///< is not allowed in Thread Operational Dataset TLVs.
 
 public:
     static constexpr uint8_t kMaxLength = kLengthEscape - 1;