From b636c521799ddcc958d8aaa677474dfcde80655b Mon Sep 17 00:00:00 2001
From: Hugo Lefeuvre <hugo.lefeuvre@savoirfairelinux.com>
Date: Thu, 21 Jun 2018 14:39:53 -0400
Subject: [PATCH] media_recorder: fix start time in startRecording
Whenever startRecording() is called and path_ is not empty (new API)
the start time variable is left uninitialized.
This commit addresses this issue.
Change-Id: I9020e91695c6803d881967999d868d4ba4fd1c72
Reviewed-by: Philippe Gorley <philippe.gorley@savoirfairelinux.com>
---
src/media/media_recorder.cpp | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/media/media_recorder.cpp b/src/media/media_recorder.cpp
index a45b9f2320..58a0c0f808 100644
--- a/src/media/media_recorder.cpp
+++ b/src/media/media_recorder.cpp
@@ -144,10 +144,11 @@ MediaRecorder::toggleRecording()
int
MediaRecorder::startRecording()
{
+ std::time_t t = std::time(nullptr);
+ startTime_ = *std::localtime(&t);
+
if (path_.empty()) {
// FIXME deprecated code, will be removed once all clients transitioned to startRecording(path).
- std::time_t t = std::time(nullptr);
- startTime_ = *std::localtime(&t);
std::stringstream ss;
ss << std::put_time(&startTime_, "%Y%m%d-%H%M%S");
filename_ = ss.str();
--
GitLab