Commit bac71c94 authored by Sébastien Blin's avatar Sébastien Blin Committed by Philippe Gorley

misc: clean some compiler warnings

Change-Id: I0bafe5c95373308e6ec16d8452fcb7f394b7c3df
Reviewed-by: Philippe Gorley's avatarPhilippe Gorley <philippe.gorley@savoirfairelinux.com>
parent 592fa473
......@@ -156,7 +156,7 @@ class Account : public Serializable, public std::enable_shared_from_this<Account
virtual uint64_t sendTextMessage(const std::string& to UNUSED,
const std::map<std::string, std::string>& payloads UNUSED) { return 0; }
virtual std::vector<DRing::Message> getLastMessages(const uint64_t& base_timestamp) {
virtual std::vector<DRing::Message> getLastMessages(const uint64_t& /*base_timestamp*/) {
return {};
}
......
......@@ -230,7 +230,7 @@ bool VideoInput::captureFrame()
case MediaDecoder::Status::FrameFinished:
publishFrame();
return true;
// continue decoding
case MediaDecoder::Status::Success:
default:
......
......@@ -278,10 +278,10 @@ RingAccount::RingAccount(const std::string& accountID, bool /* presenceEnabled *
, idPath_(fileutils::get_data_dir()+DIR_SEPARATOR_STR+getAccountID())
, cachePath_(fileutils::get_cache_dir()+DIR_SEPARATOR_STR+getAccountID())
, dataPath_(cachePath_ + DIR_SEPARATOR_STR "values")
, dhtPeerConnector_ {new DhtPeerConnector {*this}}
, proxyEnabled_(false)
, proxyServer_("")
, deviceKey_("")
, dhtPeerConnector_ {new DhtPeerConnector {*this}}
{
// Force the SFL turn server if none provided yet
turnServer_ = DEFAULT_TURN_SERVER;
......@@ -490,7 +490,7 @@ RingAccount::startOutgoingCall(const std::shared_ptr<SIPCall>& call, const std::
});
return false;
});
}, [wCall](const std::shared_ptr<RingAccount>& sthis, bool ok){
}, [wCall](const std::shared_ptr<RingAccount>&, bool ok){
if (not ok) {
if (auto call = wCall.lock()) {
RING_WARN("[call:%s] no devices found", call->getCallId().c_str());
......
......@@ -82,7 +82,7 @@ static constexpr int ICE_VIDEO_RTCP_COMPID {3};
const char* const SIPCall::LINK_TYPE = SIPAccount::ACCOUNT_TYPE;
static void
dtmfSend(SIPCall &call, char code, const std::string &dtmf)
dtmfSend(SIPCall &call, char code, const std::string& /*dtmf*/)
{
int duration = Manager::instance().voipPreferences.getPulseLength();
char dtmf_body[1000];
......
......@@ -274,15 +274,11 @@ transaction_request_cb(pjsip_rx_data *rdata)
Manager::instance().hookPreference.runHook(rdata->msg_info.msg);
bool hasVideo = false;
bool hasAudio = false;
if (r_sdp) {
auto pj_str_video = pj_str((char*) "video");
auto pj_str_audio = pj_str((char*) "audio");
for (decltype(r_sdp->media_count) i=0 ; i < r_sdp->media_count; i++) {
if (pj_strcmp(&r_sdp->media[i]->desc.media, &pj_str_video) == 0)
hasVideo = true;
else if (pj_strcmp(&r_sdp->media[i]->desc.media, &pj_str_audio) == 0)
hasAudio = true;
}
}
......@@ -852,7 +848,8 @@ invite_session_state_changed_cb(pjsip_inv_session *inv, pjsip_event *ev)
if (inv->role != PJSIP_ROLE_UAC)
break;
// close call
call->onClosed();
break;
// The call terminates normally - BYE / CANCEL
case PJSIP_SC_OK:
case PJSIP_SC_REQUEST_TERMINATED:
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment