From 3f299d37015ce3fb2e09688dee75c4ce66e78aa0 Mon Sep 17 00:00:00 2001
From: Jonathan Hui <jonhui@google.com>
Date: Fri, 8 May 2020 14:41:14 -0700
Subject: [PATCH] [mle] change InformPreviousParent() to return void (#4941)

---
 src/core/thread/mle.cpp | 6 ++----
 src/core/thread/mle.hpp | 2 +-
 2 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/src/core/thread/mle.cpp b/src/core/thread/mle.cpp
index 41da7ec14..04332ac9b 100644
--- a/src/core/thread/mle.cpp
+++ b/src/core/thread/mle.cpp
@@ -807,7 +807,7 @@ void Mle::SetStateChild(uint16_t aRloc16)
 #endif
 
 #if OPENTHREAD_CONFIG_MLE_INFORM_PREVIOUS_PARENT_ON_REATTACH
-    IgnoreError(InformPreviousParent());
+    InformPreviousParent();
     mPreviousParentRloc = mParent.GetRloc16();
 #endif
 }
@@ -4052,7 +4052,7 @@ otError Mle::CheckReachability(uint16_t aMeshDest, Ip6::Header &aIp6Header)
 }
 
 #if OPENTHREAD_CONFIG_MLE_INFORM_PREVIOUS_PARENT_ON_REATTACH
-otError Mle::InformPreviousParent(void)
+void Mle::InformPreviousParent(void)
 {
     otError          error   = OT_ERROR_NONE;
     Message *        message = NULL;
@@ -4085,8 +4085,6 @@ exit:
             message->Free();
         }
     }
-
-    return error;
 }
 #endif // OPENTHREAD_CONFIG_MLE_INFORM_PREVIOUS_PARENT_ON_REATTACH
 
diff --git a/src/core/thread/mle.hpp b/src/core/thread/mle.hpp
index 7b4c5b74f..0387a1e85 100644
--- a/src/core/thread/mle.hpp
+++ b/src/core/thread/mle.hpp
@@ -1758,7 +1758,7 @@ private:
 #endif
 
 #if OPENTHREAD_CONFIG_MLE_INFORM_PREVIOUS_PARENT_ON_REATTACH
-    otError InformPreviousParent(void);
+    void InformPreviousParent(void);
 #endif
 
 #if OPENTHREAD_CONFIG_PARENT_SEARCH_ENABLE