3 files changed, 5 insertions(+), 16 deletions(-)

M module-apps/application-settings-new/models/QuotesModel.cpp
M module-apps/application-settings-new/windows/QuotesAddWindow.cpp
M module-utils/utf8/utf8/UTF8.cpp

@@ 107,11 107,7 @@ namespace Quotes
 
     void QuotesModel::add(const Quotes::QuoteRecord &record)
     {
-        LOG_DEBUG("Adding quote: lang_id = %u, author = %s, quote = %s",
-                  static_cast<unsigned>(record.lang_id),
-                  record.quote.c_str(),
-                  record.author.c_str());
-
+        LOG_DEBUG("Adding quote: lang_id = %u", static_cast<unsigned>(record.lang_id));
         auto query = std::make_unique<Messages::AddQuoteRequest>(record.lang_id, record.quote, record.author, true);
         auto task  = app::AsyncQuery::createFromQuery(std::move(query), db::Interface::Name::Quotes);
         task->execute(app, this);


@@ 119,10 115,7 @@ namespace Quotes
 
     void QuotesModel::edit(const Quotes::QuoteRecord &record)
     {
-        LOG_DEBUG("Saving quote: lang_id = %u, author = %s, quote = %s",
-                  static_cast<unsigned>(record.lang_id),
-                  record.quote.c_str(),
-                  record.author.c_str());
+        LOG_DEBUG("Saving quote: lang_id = %u", static_cast<unsigned>(record.lang_id));
 
         auto query = std::make_unique<Messages::WriteQuoteRequest>(
             record.quote_id, record.lang_id, record.quote, record.author, record.enabled);


@@ 132,10 125,7 @@ namespace Quotes
 
     void QuotesModel::remove(const Quotes::QuoteRecord &record)
     {
-        LOG_DEBUG("Removing quote: lang_id = %u, author = %s, quote = %s",
-                  static_cast<unsigned>(record.lang_id),
-                  record.quote.c_str(),
-                  record.author.c_str());
+        LOG_DEBUG("Removing quote: lang_id = %u", static_cast<unsigned>(record.lang_id));
 
         auto query = std::make_unique<Messages::DeleteQuoteRequest>(record.quote_id);
         auto task  = app::AsyncQuery::createFromQuery(std::move(query), db::Interface::Name::Quotes);

@@ 150,7 150,7 @@ namespace gui
     bool QuoteAddEditWindow::onInput(const gui::InputEvent &inputEvent)
     {
         if (inputEvent.isShortRelease(gui::KeyCode::KEY_ENTER)) {
-            LOG_DEBUG("Save Quote: %s", quoteText->getText().c_str());
+            LOG_DEBUG("Save Quote requested");
             quoteData.quote  = quoteText->getText();
             quoteData.author = authorText->getText();
 

@@ 155,7 155,6 @@ UTF8::UTF8(const UTF8 &utf)
     if (strLength != 0) {
         data = std::make_unique<char[]>(sizeAllocated);
         if (data == nullptr) {
-            // LOG_FATAL("No memory for copy constructor.");
             sizeAllocated = 0;
             sizeUsed      = 0;
             return;


@@ 677,7 676,7 @@ bool UTF8::insertCode(const uint32_t &charCode, const uint32_t &index)
 {
     auto u = U8char(charCode);
     if (u.size == 0) {
-        LOG_ERROR("Failed to encode value: %x", static_cast<int>(charCode));
+        LOG_ERROR("Failed to encode character code");
         return false;
     }
     debug_utf("from 0x%x to size: %d -- 0x%x 0x%x\n", charCode, u.size, u.utf8[0], u.utf8[1]);