[style] remove extra ';' (#4017)

src/core/coap/coap.hpp

@@ -227,7 +227,7 @@ public:
      * @returns A Pointer to the next resource.
      *
      */
-    Resource *GetNext(void) const { return static_cast<Resource *>(mNext); };
+    Resource *GetNext(void) const { return static_cast<Resource *>(mNext); }
 
     /**
      * This method returns a pointer to the Uri-Path.
@@ -235,7 +235,7 @@ public:
      * @returns A Pointer to the Uri-Path.
      *
      */
-    const char *GetUriPath(void) const { return mUriPath; };
+    const char *GetUriPath(void) const { return mUriPath; }
 
 private:
     void HandleRequest(Message &aMessage, const Ip6::MessageInfo &aMessageInfo) const
@@ -526,7 +526,7 @@ public:
     otError SendReset(Message &aRequest, const Ip6::MessageInfo &aMessageInfo)
     {
         return SendEmptyMessage(OT_COAP_TYPE_RESET, aRequest, aMessageInfo);
-    };
+    }
 
     /**
      * This method sends header-only CoAP response message.
@@ -556,7 +556,7 @@ public:
     otError SendAck(const Message &aRequest, const Ip6::MessageInfo &aMessageInfo)
     {
         return SendEmptyMessage(OT_COAP_TYPE_ACKNOWLEDGMENT, aRequest, aMessageInfo);
-    };
+    }
 
     /**
      * This method sends a CoAP ACK message on which a dummy CoAP response is piggybacked.

src/core/coap/coap_message.hpp

@@ -446,7 +446,7 @@ public:
      * @retval FALSE  Message is not an empty message header.
      *
      */
-    bool IsEmpty(void) const { return (GetCode() == 0); };
+    bool IsEmpty(void) const { return (GetCode() == 0); }
 
     /**
      * This method checks if a header is a request header.
@@ -455,7 +455,7 @@ public:
      * @retval FALSE  Message is not a request header.
      *
      */
-    bool IsRequest(void) const { return (GetCode() >= OT_COAP_CODE_GET && GetCode() <= OT_COAP_CODE_DELETE); };
+    bool IsRequest(void) const { return (GetCode() >= OT_COAP_CODE_GET && GetCode() <= OT_COAP_CODE_DELETE); }
 
     /**
      * This method checks if a header is a response header.
@@ -464,7 +464,7 @@ public:
      * @retval FALSE  Message is not a response header.
      *
      */
-    bool IsResponse(void) const { return (GetCode() >= OT_COAP_CODE_RESPONSE_MIN); };
+    bool IsResponse(void) const { return (GetCode() >= OT_COAP_CODE_RESPONSE_MIN); }
 
     /**
      * This method checks if a header is a CON message header.
@@ -473,7 +473,7 @@ public:
      * @retval FALSE  Message is not is a CON message header.
      *
      */
-    bool IsConfirmable(void) const { return (GetType() == OT_COAP_TYPE_CONFIRMABLE); };
+    bool IsConfirmable(void) const { return (GetType() == OT_COAP_TYPE_CONFIRMABLE); }
 
     /**
      * This method checks if a header is a NON message header.
@@ -482,7 +482,7 @@ public:
      * @retval FALSE  Message is not is a NON message header.
      *
      */
-    bool IsNonConfirmable(void) const { return (GetType() == OT_COAP_TYPE_NON_CONFIRMABLE); };
+    bool IsNonConfirmable(void) const { return (GetType() == OT_COAP_TYPE_NON_CONFIRMABLE); }
 
     /**
      * This method checks if a header is a ACK message header.
@@ -491,7 +491,7 @@ public:
      * @retval FALSE  Message is not is a ACK message header.
      *
      */
-    bool IsAck(void) const { return (GetType() == OT_COAP_TYPE_ACKNOWLEDGMENT); };
+    bool IsAck(void) const { return (GetType() == OT_COAP_TYPE_ACKNOWLEDGMENT); }
 
     /**
      * This method checks if a header is a RST message header.
@@ -500,7 +500,7 @@ public:
      * @retval FALSE  Message is not is a RST message header.
      *
      */
-    bool IsReset(void) const { return (GetType() == OT_COAP_TYPE_RESET); };
+    bool IsReset(void) const { return (GetType() == OT_COAP_TYPE_RESET); }
 
     /**
      * This method creates a copy of this CoAP message.
@@ -526,7 +526,7 @@ public:
      * @returns A pointer to the message or NULL if insufficient message buffers are available.
      *
      */
-    Message *Clone(void) const { return Clone(GetLength()); };
+    Message *Clone(void) const { return Clone(GetLength()); }
 
     /**
      * This method returns the minimal reserved bytes required for CoAP message.

src/core/common/logging.cpp

@@ -355,5 +355,5 @@ void otPlatLog(otLogLevel aLogLevel, otLogRegion aLogRegion, const char *aFormat
 #endif
 
 #ifdef __cplusplus
-};
+}
 #endif

src/core/common/logging.hpp

@@ -1918,7 +1918,7 @@ const char *otLogLevelToPrefixString(otLogLevel aLogLevel);
     OPENTHREAD_CONFIG_PLAT_LOG_FUNCTION(aLogLevel, aRegion, aFormat, ##__VA_ARGS__)
 
 #ifdef __cplusplus
-};
+}
 #endif
 
 #endif // LOGGING_HPP_

src/core/common/message.hpp

@@ -473,7 +473,7 @@ public:
      * @returns A pointer to the message or NULL if insufficient message buffers are available.
      *
      */
-    Message *Clone(void) const { return Clone(GetLength()); };
+    Message *Clone(void) const { return Clone(GetLength()); }
 
     /**
      * This method returns the datagram tag used for 6LoWPAN fragmentation.

src/core/common/string.cpp

@@ -59,6 +59,6 @@ otError StringBase::Write(char *aBuffer, uint16_t aSize, uint16_t &aLength, cons
     }
 
     return error;
-};
+}
 
 } // namespace ot

src/core/common/timer.hpp

@@ -400,7 +400,7 @@ public:
      * @param[in]  aTimer  A reference to the timer instance.
      *
      */
-    void Remove(TimerMilli &aTimer) { TimerScheduler::Remove(aTimer, sAlarmMilliApi); };
+    void Remove(TimerMilli &aTimer) { TimerScheduler::Remove(aTimer, sAlarmMilliApi); }
 
     /**
      * This method processes the running timers.
@@ -501,7 +501,7 @@ public:
      * @param[in]  aTimer  A reference to the timer instance.
      *
      */
-    void Remove(TimerMicro &aTimer) { TimerScheduler::Remove(aTimer, sAlarmMicroApi); };
+    void Remove(TimerMicro &aTimer) { TimerScheduler::Remove(aTimer, sAlarmMicroApi); }
 
     /**
      * This method processes the running timers.

src/core/mac/mac_frame.hpp

@@ -579,7 +579,7 @@ public:
      * @returns The IEEE 802.15.4 Frame Version.
      *
      */
-    uint16_t GetVersion(void) const { return GetFrameControlField() & kFcfFrameVersionMask; };
+    uint16_t GetVersion(void) const { return GetFrameControlField() & kFcfFrameVersionMask; }
 
     /**
      * This method indicates whether or not security is enabled.
@@ -631,7 +631,7 @@ public:
      * @retval FALSE  If no IE present.
      *
      */
-    bool IsIePresent(void) const { return (GetFrameControlField() & kFcfIePresent) != 0; };
+    bool IsIePresent(void) const { return (GetFrameControlField() & kFcfIePresent) != 0; }
 
     /**
      * This method returns the Sequence Number value.

src/core/net/dns_headers.hpp

@@ -565,7 +565,9 @@ public:
      *
      */
     QuestionAaaa(void)
-        : Question(kType, kClass){};
+        : Question(kType, kClass)
+    {
+    }
 
     /**
      * This method appends request data to the message.
@@ -576,7 +578,7 @@ public:
      * @retval OT_ERROR_NO_BUFS  Insufficient available buffers to grow the message.
      *
      */
-    otError AppendTo(Message &aMessage) const { return aMessage.Append(this, sizeof(*this)); };
+    otError AppendTo(Message &aMessage) const { return aMessage.Append(this, sizeof(*this)); }
 };
 
 /**

src/core/net/sntp_client.hpp

@@ -431,7 +431,7 @@ public:
      * Default constructor for the object.
      *
      */
-    QueryMetadata(void) { memset(this, 0, sizeof(*this)); };
+    QueryMetadata(void) { memset(this, 0, sizeof(*this)); }
 
     /**
      * This constructor initializes the object with specific values.
@@ -445,7 +445,7 @@ public:
         memset(this, 0, sizeof(*this));
         mResponseHandler = aHandler;
         mResponseContext = aContext;
-    };
+    }
 
     /**
      * This method appends request data to the message.
@@ -456,7 +456,7 @@ public:
      * @retval OT_ERROR_NO_BUFS  Insufficient available buffers to grow the message.
      *
      */
-    otError AppendTo(Message &aMessage) const { return aMessage.Append(this, sizeof(*this)); };
+    otError AppendTo(Message &aMessage) const { return aMessage.Append(this, sizeof(*this)); }
 
     /**
      * This method reads request data from the message.
@@ -492,7 +492,7 @@ public:
      * @retval TRUE   If the message shall be sent before the given time.
      * @retval FALSE  Otherwise.
      */
-    bool IsEarlier(uint32_t aTime) const { return (static_cast<int32_t>(aTime - mTransmissionTime) > 0); };
+    bool IsEarlier(uint32_t aTime) const { return (static_cast<int32_t>(aTime - mTransmissionTime) > 0); }
 
     /**
      * This method checks if the message shall be sent after the given time.
@@ -502,7 +502,7 @@ public:
      * @retval TRUE   If the message shall be sent after the given time.
      * @retval FALSE  Otherwise.
      */
-    bool IsLater(uint32_t aTime) const { return (static_cast<int32_t>(aTime - mTransmissionTime) < 0); };
+    bool IsLater(uint32_t aTime) const { return (static_cast<int32_t>(aTime - mTransmissionTime) < 0); }
 
 private:
     uint32_t              mTransmitTimestamp;   ///< Time at the client when the request departed for the server.

src/core/thread/mle.hpp

@@ -367,7 +367,7 @@ public:
      * Default constructor for the object.
      *
      */
-    DelayedResponseHeader(void) { memset(this, 0, sizeof(*this)); };
+    DelayedResponseHeader(void) { memset(this, 0, sizeof(*this)); }
 
     /**
      * This constructor initializes the object with specific values.
@@ -380,7 +380,7 @@ public:
     {
         mSendTime    = aSendTime;
         mDestination = aDestination;
-    };
+    }
 
     /**
      * This method appends delayed response header to the message.
@@ -391,7 +391,7 @@ public:
      * @retval OT_ERROR_NO_BUFS  Insufficient available buffers to grow the message.
      *
      */
-    otError AppendTo(Message &aMessage) { return aMessage.Append(this, sizeof(*this)); };
+    otError AppendTo(Message &aMessage) { return aMessage.Append(this, sizeof(*this)); }
 
     /**
      * This method reads delayed response header from the message.
@@ -425,7 +425,7 @@ public:
      * @returns  A time when the message shall be sent.
      *
      */
-    uint32_t GetSendTime(void) const { return mSendTime; };
+    uint32_t GetSendTime(void) const { return mSendTime; }
 
     /**
      * This method returns a destination of the delayed message.
@@ -433,7 +433,7 @@ public:
      * @returns  A destination of the delayed message.
      *
      */
-    const Ip6::Address &GetDestination(void) const { return mDestination; };
+    const Ip6::Address &GetDestination(void) const { return mDestination; }
 
     /**
      * This method checks if the message shall be sent before the given time.
@@ -443,7 +443,7 @@ public:
      * @retval TRUE   If the message shall be sent before the given time.
      * @retval FALSE  Otherwise.
      */
-    bool IsEarlier(uint32_t aTime) { return (static_cast<int32_t>(aTime - mSendTime) > 0); };
+    bool IsEarlier(uint32_t aTime) { return (static_cast<int32_t>(aTime - mSendTime) > 0); }
 
     /**
      * This method checks if the message shall be sent after the given time.
@@ -453,7 +453,7 @@ public:
      * @retval TRUE   If the message shall be sent after the given time.
      * @retval FALSE  Otherwise.
      */
-    bool IsLater(uint32_t aTime) { return (static_cast<int32_t>(aTime - mSendTime) < 0); };
+    bool IsLater(uint32_t aTime) { return (static_cast<int32_t>(aTime - mSendTime) < 0); }
 
 private:
     Ip6::Address mDestination; ///< IPv6 address of the message destination.

src/core/thread/time_sync_service.hpp

@@ -96,7 +96,7 @@ public:
      * @returns The time synchronization sequence.
      *
      */
-    uint8_t GetTimeSyncSeq(void) const { return mTimeSyncSeq; };
+    uint8_t GetTimeSyncSeq(void) const { return mTimeSyncSeq; }
 
     /**
      * This method gets the time offset to the Thread network time.
@@ -104,7 +104,7 @@ public:
      * @returns The time offset to the Thread network time, in microseconds.
      *
      */
-    int64_t GetNetworkTimeOffset(void) const { return mNetworkTimeOffset; };
+    int64_t GetNetworkTimeOffset(void) const { return mNetworkTimeOffset; }
 
     /**
      * Set the time synchronization period.