From e0f52c1f7f87d790f566941bb4e17b4510e99c99 Mon Sep 17 00:00:00 2001
From: Guillaume Roguez <guillaume.roguez@savoirfairelinux.com>
Date: Thu, 19 Oct 2017 13:30:06 -0400
Subject: [PATCH] remove build warnings in API header
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

fix to_string() and to_status in lrc::api::call to remove
build warnings.

Change-Id: I29d964fca41c26795f28af5887397082199154c7
Reviewed-by: Sébastien Blin <sebastien.blin@savoirfairelinux.com>
---
 src/api/call.h | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/api/call.h b/src/api/call.h
index 4674dce9..9cafe66d 100644
--- a/src/api/call.h
+++ b/src/api/call.h
@@ -52,7 +52,7 @@ enum class Status {
     AUTO_ANSWERING
 };
 
-static inline const std::string
+static inline std::string
 to_string(const call::Status& status)
 {
     switch(status)
@@ -85,6 +85,8 @@ to_string(const call::Status& status)
         return QObject::tr("Communication established").toStdString();
     case call::Status::AUTO_ANSWERING:
         return QObject::tr("Auto answering").toStdString();
+    default:
+        return ""; // to remove a build warning, should not happen
     }
 }
 
@@ -94,7 +96,7 @@ to_string(const call::Status& status)
  * @param  status
  * @return
  */
-static inline const Status
+static inline Status
 to_status(const std::string& status)
 {
     if (status == "INCOMING")
-- 
GitLab