Transmission/transmission
1
0
Fork 0
mirror of https://github.com/transmission/transmission.git synced 2026-05-08 09:39:08 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix: sonarcloud (#2558)

Browse Source 
* fix: add const modifier for functions

* fix: many sonarcloud use-init-statement warnings
This commit is contained in:
Charles Kerr
2022-01-31 22:46:27 -06:00
committed by GitHub
parent cf2c32e1ce
commit 8b9483f7fb
58 changed files with 158 additions and 288 deletions
Download Patch File Download Diff File Expand all files Collapse all files
gtk/Actions.cc
+1 -1
View File
@@ -26,7 +26,7 @@ namespace
Session* myCore = nullptr;
void action_cb(Gio::SimpleAction& action, gpointer user_data)
void action_cb(Gio::SimpleAction const& action, gpointer user_data)
{
gtr_actions_handler(action.get_name(), user_data);
}
gtk/FileList.cc
+4 -11
View File
@@ -301,9 +301,7 @@ bool getSelectedFilesForeach(
Glib::RefPtr<Gtk::TreeSelection> const& sel,
std::vector<tr_file_index_t>& indexBuf)
{
bool const is_file = iter->children().empty();
if (is_file)
if (bool const is_file = iter->children().empty(); is_file)
{
/* active means: if it's selected or any ancestor is selected */
bool is_active = sel->is_selected(iter);
@@ -345,9 +343,7 @@ bool getSubtreeForeach(
Gtk::TreeModel::Path const& subtree_path,
std::vector<tr_file_index_t>& indexBuf)
{
bool const is_file = iter->children().empty();
if (is_file)
if (bool const is_file = iter->children().empty(); is_file)
{
if (path == subtree_path || path.is_descendant(subtree_path))
{
@@ -479,8 +475,7 @@ void FileList::Impl::set_torrent(int torrentId)
torrent_id_ = torrentId;
/* populate the model */
auto* const tor = torrent_id_ > 0 ? core_->find_torrent(torrent_id_) : nullptr;
if (tor != nullptr)
if (auto* const tor = torrent_id_ > 0 ? core_->find_torrent(torrent_id_) : nullptr; tor != nullptr)
{
// build a GNode tree of the files
FileRowNode root;
@@ -690,9 +685,7 @@ bool FileList::Impl::getAndSelectEventPath(GdkEventButton const* event, Gtk::Tre
if (view_->get_path_at_pos(event->x, event->y, path, col, cell_x, cell_y))
{
auto const sel = view_->get_selection();
if (!sel->is_selected(path))
if (auto const sel = view_->get_selection(); !sel->is_selected(path))
{
sel->unselect_all();
sel->select(path);
gtk/IconCache.cc
+1 -1
View File
@@ -80,7 +80,7 @@ std::string _icon_cache_get_icon_key(Glib::RefPtr<Gio::Icon> const& icon)
return key;
}
Glib::RefPtr<Gdk::Pixbuf> get_themed_icon_pixbuf(Gio::ThemedIcon& icon, int size, Gtk::IconTheme& icon_theme)
Glib::RefPtr<Gdk::Pixbuf> get_themed_icon_pixbuf(Gio::ThemedIcon const& icon, int size, Gtk::IconTheme& icon_theme)
{
auto const icon_names = icon.get_names();
gtk/MessageLogWindow.cc
+1 -3
View File
@@ -376,9 +376,7 @@ bool MessageLogWindow::Impl::onRefresh()
if (!isPaused_)
{
auto* msgs = tr_logGetQueue();
if (msgs != nullptr)
if (auto* msgs = tr_logGetQueue(); msgs != nullptr)
{
/* add the new messages and append them to the end of
* our persistent list */
gtk/Prefs.cc
+1 -2
View File
@@ -174,8 +174,7 @@ std::vector<std::string> gtr_pref_strv_get(tr_quark const key)
{
std::vector<std::string> ret;
tr_variant* list = nullptr;
if (tr_variantDictFindList(getPrefs(), key, &list))
if (tr_variant* list = nullptr; tr_variantDictFindList(getPrefs(), key, &list))
{
size_t const n = tr_variantListSize(list);
ret.reserve(n);
gtk/PrefsDialog.cc
+3 -2
View File
@@ -3,6 +3,7 @@
// or any future license endorsed by Mnemosyne LLC.
// License text can be found in the licenses/ folder.
#include <array>
#include <climits> /* USHRT_MAX, INT_MAX */
#include <sstream>
#include <string>
@@ -1097,7 +1098,7 @@ PrefsDialog::Impl::Impl(PrefsDialog& dialog, Glib::RefPtr<Session> const& core)
: dialog_(dialog)
, core_(core)
{
static tr_quark const prefs_quarks[] = { TR_KEY_peer_port, TR_KEY_download_dir };
auto constexpr PrefsQuarks = std::array<tr_quark, 2>{ TR_KEY_peer_port, TR_KEY_download_dir };
core_prefs_tag_ = core_->signal_prefs_changed().connect(sigc::mem_fun(*this, &Impl::on_core_prefs_changed));
@@ -1118,7 +1119,7 @@ PrefsDialog::Impl::Impl(PrefsDialog& dialog, Glib::RefPtr<Session> const& core)
n->append_page(*remotePage(), _("Remote"));
/* init from prefs keys */
for (auto const key : prefs_quarks)
for (auto const key : PrefsQuarks)
{
on_core_prefs_changed(key);
}
gtk/Session.cc
+4 -5
View File
@@ -145,7 +145,7 @@ private:
void on_pref_changed(tr_quark key);
void on_torrent_completeness_changed(tr_torrent* tor, tr_completeness completeness, bool was_running);
void on_torrent_metadata_changed(tr_torrent* tor);
void on_torrent_metadata_changed(tr_torrent const* tor);
private:
Session& core_;
@@ -896,7 +896,7 @@ Gtk::TreeModel::iterator find_row_from_torrent_id(Glib::RefPtr<Gtk::TreeModel> c
/* this is called in the libtransmission thread, *NOT* the GTK+ thread,
so delegate to the GTK+ thread before changing our list store... */
void Session::Impl::on_torrent_metadata_changed(tr_torrent* tor)
void Session::Impl::on_torrent_metadata_changed(tr_torrent const* tor)
{
Glib::signal_idle().connect(
[this, core = get_core_ptr(), torrent_id = tr_torrentId(tor)]()
@@ -925,7 +925,7 @@ void Session::Impl::on_torrent_metadata_changed(tr_torrent* tor)
namespace
{
unsigned int build_torrent_trackers_hash(tr_torrent* tor)
unsigned int build_torrent_trackers_hash(tr_torrent const* tor)
{
auto hash = uint64_t{};
@@ -1135,9 +1135,8 @@ void Session::Impl::add_file_async_callback(
bool Session::Impl::add_file(Glib::RefPtr<Gio::File> const& file, bool do_start, bool do_prompt, bool do_notify)
{
bool handled = false;
auto const* const session = get_session();
if (session != nullptr)
if (auto const* const session = get_session(); session != nullptr)
{
tr_ctor* ctor;
bool tried = false;
gtk/SystemTrayIcon.cc
+1 -3
View File
@@ -134,11 +134,9 @@ Glib::ustring getIconName()
{
Glib::ustring icon_name;
auto theme = Gtk::IconTheme::get_default();
// if the tray's icon is a 48x48 file, use it.
// otherwise, use the fallback builtin icon.
if (!theme->has_icon(TrayIconName))
if (auto theme = Gtk::IconTheme::get_default(); !theme->has_icon(TrayIconName))
{
icon_name = AppIconName;
}
gtk/Utils.cc
+1 -3
View File
@@ -272,9 +272,7 @@ bool on_tree_view_button_pressed(
* clear all the selections. */
bool on_tree_view_button_released(Gtk::TreeView* view, GdkEventButton* event)
{
Gtk::TreeModel::Path path;
if (!view->get_path_at_pos((int)event->x, (int)event->y, path))
if (Gtk::TreeModel::Path path; !view->get_path_at_pos((int)event->x, (int)event->y, path))
{
view->get_selection()->unselect_all();
}
libtransmission/announce-list.cc
+3 -6
View File
@@ -56,8 +56,7 @@ bool tr_announce_list::remove(tr_tracker_id_t id)
bool tr_announce_list::replace(tr_tracker_id_t id, std::string_view announce_url_sv)
{
auto const announce = tr_urlParseTracker(announce_url_sv);
if (!announce || !canAdd(*announce))
if (auto const announce = tr_urlParseTracker(announce_url_sv); !announce || !canAdd(*announce))
{
return false;
}
@@ -88,8 +87,7 @@ bool tr_announce_list::add(std::string_view announce_url_sv, tr_tracker_tier_t t
tracker.id = nextUniqueId();
tracker.host = tr_strvJoin(tracker.announce.host, ":"sv, tracker.announce.portstr);
auto const scrape_str = announceToScrape(announce_url_sv);
if (scrape_str)
if (auto const scrape_str = announceToScrape(announce_url_sv); scrape_str)
{
tracker.scrape_str = *scrape_str;
tracker.scrape = *tr_urlParseTracker(tracker.scrape_str.sv());
@@ -127,8 +125,7 @@ std::optional<std::string> tr_announce_list::announceToScrape(std::string_view a
tr_quark tr_announce_list::announceToScrape(tr_quark announce)
{
auto const scrape_str = announceToScrape(tr_quark_get_string_view(announce));
if (scrape_str)
if (auto const scrape_str = announceToScrape(tr_quark_get_string_view(announce)); scrape_str)
{
return tr_quark_new(*scrape_str);
}
libtransmission/announcer-http.cc
+3 -8
View File
@@ -110,8 +110,7 @@ static std::string announce_url_new(tr_session const* session, tr_announce_reque
announce twice. At any rate, we're already computing our IPv6
address (for the LTEP handshake), so this comes for free. */
unsigned char const* const ipv6 = tr_globalIPv6(session);
if (ipv6 != nullptr)
if (auto const* const ipv6 = tr_globalIPv6(session); ipv6 != nullptr)
{
auto ipv6_readable = std::array<char, INET6_ADDRSTRLEN>{};
evutil_inet_ntop(AF_INET6, ipv6, std::data(ipv6_readable), std::size(ipv6_readable));
@@ -418,9 +417,7 @@ void tr_announcerParseHttpScrapeResponse(tr_scrape_response& response, std::stri
bool Int64(int64_t value, Context const& context) override
{
auto const key = currentKey();
if (row_ && key == "complete"sv)
if (auto const key = currentKey(); row_ && key == "complete"sv)
{
response_.rows[*row_].seeders = value;
}
@@ -443,9 +440,7 @@ void tr_announcerParseHttpScrapeResponse(tr_scrape_response& response, std::stri
bool String(std::string_view value, Context const& context) override
{
auto const key = currentKey();
if (depth() == 1 && key == "failure reason"sv)
if (auto const key = currentKey(); depth() == 1 && key == "failure reason"sv)
{
response_.errmsg = value;
}
libtransmission/announcer.cc
+1 -2
View File
@@ -1322,8 +1322,7 @@ static void on_scrape_done(tr_scrape_response const* response, void* vsession)
tier->scheduleNextScrape();
tr_logAddTorDbg(tier->tor, "Scrape successful. Rescraping in %d seconds.", tier->scrapeIntervalSec);
tr_tracker* const tracker = tier->currentTracker();
if (tracker != nullptr)
if (auto* const tracker = tier->currentTracker(); tracker != nullptr)
{
if (row.seeders >= 0)
{
libtransmission/blocklist.cc
+1 -1
View File
@@ -165,7 +165,7 @@ int tr_blocklistFileGetRuleCount(tr_blocklistFile const* b)
return b->ruleCount;
}
bool tr_blocklistFileIsEnabled(tr_blocklistFile* b)
bool tr_blocklistFileIsEnabled(tr_blocklistFile const* b)
{
return b->isEnabled;
}
libtransmission/blocklist.h
+1 -1
View File
@@ -22,7 +22,7 @@ int tr_blocklistFileGetRuleCount(tr_blocklistFile const* b);
void tr_blocklistFileFree(tr_blocklistFile* b);
bool tr_blocklistFileIsEnabled(tr_blocklistFile* b);
bool tr_blocklistFileIsEnabled(tr_blocklistFile const* b);
void tr_blocklistFileSetEnabled(tr_blocklistFile* b, bool isEnabled);
libtransmission/cache.cc
+3 -7
View File
@@ -356,9 +356,8 @@ int tr_cacheReadBlock(
uint8_t* setme)
{
int err = 0;
struct cache_block* cb = findBlock(cache, torrent, piece, offset);
if (cb != nullptr)
if (auto* cb = findBlock(cache, torrent, piece, offset); cb != nullptr)
{
evbuffer_copyout(cb->evbuf, setme, len);
}
@@ -373,9 +372,8 @@ int tr_cacheReadBlock(
int tr_cachePrefetchBlock(tr_cache* cache, tr_torrent* torrent, tr_piece_index_t piece, uint32_t offset, uint32_t len)
{
int err = 0;
struct cache_block const* const cb = findBlock(cache, torrent, piece, offset);
if (cb == nullptr)
if (auto const* const cb = findBlock(cache, torrent, piece, offset); cb == nullptr)
{
err = tr_ioPrefetch(torrent, piece, offset, len);
}
@@ -454,9 +452,7 @@ int tr_cacheFlushTorrent(tr_cache* cache, tr_torrent* torrent)
/* flush out all the blocks in that torrent */
while (err == 0 && pos < tr_ptrArraySize(&cache->blocks))
{
auto const* b = static_cast<struct cache_block const*>(tr_ptrArrayNth(&cache->blocks, pos));
if (b->tor != torrent)
if (auto const* b = static_cast<struct cache_block const*>(tr_ptrArrayNth(&cache->blocks, pos)); b->tor != torrent)
{
break;
}
libtransmission/file-posix.cc
+3 -7
View File
@@ -917,9 +917,7 @@ bool tr_sys_file_advise(
TR_ASSERT(native_advice != POSIX_FADV_NORMAL);
int const code = posix_fadvise(handle, offset, size, native_advice);
if (code != 0)
if (int const code = posix_fadvise(handle, offset, size, native_advice); code != 0)
{
set_system_error(error, code);
ret = false;
@@ -1052,8 +1050,7 @@ bool tr_sys_file_preallocate(tr_sys_file_t handle, uint64_t size, int flags, tr_
{
errno = 0;
auto const success = approach(handle, size);
if (success)
if (auto const success = approach(handle, size); success)
{
return success;
}
@@ -1315,9 +1312,8 @@ char const* tr_sys_dir_read_name(tr_sys_dir_t handle, tr_error** error)
char const* ret = nullptr;
errno = 0;
struct dirent const* const entry = readdir((DIR*)handle);
if (entry != nullptr)
if (auto const* const entry = readdir((DIR*)handle); entry != nullptr)
{
ret = entry->d_name;
}
libtransmission/handshake.cc
+1 -2
View File
@@ -769,8 +769,7 @@ static ReadState readPadA(tr_handshake* handshake, struct evbuffer* inbuf)
return READ_NOW;
}
size_t const len = evbuffer_get_length(inbuf);
if (len > SHA_DIGEST_LENGTH)
if (size_t const len = evbuffer_get_length(inbuf); len > SHA_DIGEST_LENGTH)
{
evbuffer_drain(inbuf, len - SHA_DIGEST_LENGTH);
}
libtransmission/net.cc
+1 -3
View File
@@ -598,9 +598,7 @@ static int tr_globalAddress(int af, void* addr, int* addr_len)
return -1;
}
int const rc = get_source_address(sa, salen, (struct sockaddr*)&ss, &sslen);
if (rc < 0)
if (int const rc = get_source_address(sa, salen, (struct sockaddr*)&ss, &sslen); rc < 0)
{
return -1;
}
libtransmission/peer-mgr.cc
+9 -6
View File
@@ -668,7 +668,7 @@ static void refillUpkeep(evutil_socket_t /*fd*/, short /*what*/, void* vmgr)
tr_timerAddMsec(mgr->refillUpkeepTimer, RefillUpkeepPeriodMsec);
}
static void addStrike(tr_swarm* s, tr_peer* peer)
static void addStrike(tr_swarm const* s, tr_peer* peer)
{
tordbg(s, "increasing peer %s strike count to %d", tr_atomAddrStr(peer->atom), peer->strikes + 1);
@@ -681,7 +681,11 @@ static void addStrike(tr_swarm* s, tr_peer* peer)
}
}
static void peerSuggestedPiece(tr_swarm* /*s*/, tr_peer* /*peer*/, tr_piece_index_t /*pieceIndex*/, int /*isFastAllowed*/)
static void peerSuggestedPiece(
tr_swarm const* /*s*/,
tr_peer const* /*peer*/,
tr_piece_index_t /*pieceIndex*/,
int /*isFastAllowed*/)
{
#if 0
@@ -1826,7 +1830,7 @@ void tr_peerMgrClearInterest(tr_torrent* tor)
}
/* does this peer have any pieces that we want? */
static bool isPeerInteresting(tr_torrent* const tor, bool const* const piece_is_interesting, tr_peer const* const peer)
static bool isPeerInteresting(tr_torrent const* const tor, bool const* const piece_is_interesting, tr_peer const* const peer)
{
/* these cases should have already been handled by the calling code... */
TR_ASSERT(!tor->isDone());
@@ -2401,12 +2405,11 @@ static void closePeer(tr_peer* peer)
{
TR_ASSERT(peer != nullptr);
auto* const s = peer->swarm;
peer_atom* const atom = peer->atom;
/* if we transferred piece data, then they might be good peers,
so reset their `numFails' weight to zero. otherwise we connected
to them fruitlessly, so mark it as another fail */
if (atom->piece_data_time != 0)
if (auto* const atom = peer->atom; atom->piece_data_time != 0)
{
tordbg(s, "resetting atom %s numFails to 0", tr_atomAddrStr(atom));
atom->numFails = 0;
@@ -2940,7 +2943,7 @@ static bool swarmIsAllSeeds(tr_swarm* swarm)
}
/** @return an array of all the atoms we might want to connect to */
static std::vector<peer_candidate> getPeerCandidates(tr_session* session, size_t max)
static std::vector<peer_candidate> getPeerCandidates(tr_session const* session, size_t max)
{
time_t const now = tr_time();
uint64_t const now_msec = tr_time_msec();
libtransmission/peer-msgs.cc
+3 -5
View File
@@ -707,7 +707,7 @@ static void pokeBatchPeriod(tr_peerMsgsImpl* msgs, int interval)
}
}
static void dbgOutMessageLen(tr_peerMsgsImpl* msgs)
static void dbgOutMessageLen(tr_peerMsgsImpl const* msgs)
{
dbgmsg(msgs, "outMessage size is now %zu", evbuffer_get_length(msgs->outMessages));
}
@@ -1156,8 +1156,7 @@ static void parseLtepHandshake(tr_peerMsgsImpl* msgs, uint32_t len, struct evbuf
msgs->peerSupportsPex = false;
msgs->peerSupportsMetadataXfer = false;
tr_variant* sub = nullptr;
if (tr_variantDictFindDict(&val, TR_KEY_m, &sub))
if (tr_variant* sub = nullptr; tr_variantDictFindDict(&val, TR_KEY_m, &sub))
{
if (tr_variantDictFindInt(sub, TR_KEY_ut_pex, &i))
{
@@ -1914,8 +1913,7 @@ static int clientGotBlock(tr_peerMsgsImpl* msgs, struct evbuffer* data, struct p
*** Save the block
**/
int const err = tr_cacheWriteBlock(msgs->session->cache, tor, req->index, req->offset, req->length, data);
if (err != 0)
if (int const err = tr_cacheWriteBlock(msgs->session->cache, tor, req->index, req->offset, req->length, data); err != 0)
{
return err;
}
libtransmission/peer-msgs.h
+1 -1
View File
@@ -28,7 +28,7 @@ struct tr_torrent;
class tr_peerMsgs : public tr_peer
{
public:
tr_peerMsgs(tr_torrent* torrent, peer_atom* atom_in)
tr_peerMsgs(tr_torrent const* torrent, peer_atom* atom_in)
: tr_peer{ torrent, atom_in }
{
}
libtransmission/quark.cc
+1 -2
View File
@@ -436,8 +436,7 @@ std::optional<tr_quark> tr_quark_lookup(std::string_view key)
/* was it added during runtime? */
auto const rbegin = std::begin(my_runtime);
auto const rend = std::end(my_runtime);
auto const rit = std::find(rbegin, rend, key);
if (rit != rend)
if (auto const rit = std::find(rbegin, rend, key); rit != rend)
{
return TR_N_KEYS + std::distance(rbegin, rit);
}
libtransmission/resume.cc
+12 -20
View File
@@ -228,7 +228,7 @@ static uint64_t loadFilePriorities(tr_variant* dict, tr_torrent* tor)
****
***/
static void saveSingleSpeedLimit(tr_variant* d, tr_torrent* tor, tr_direction dir)
static void saveSingleSpeedLimit(tr_variant* d, tr_torrent const* tor, tr_direction dir)
{
tr_variantDictReserve(d, 3);
tr_variantDictAddInt(d, TR_KEY_speed_Bps, tor->speedLimitBps(dir));
@@ -236,20 +236,20 @@ static void saveSingleSpeedLimit(tr_variant* d, tr_torrent* tor, tr_direction di
tr_variantDictAddBool(d, TR_KEY_use_speed_limit, tr_torrentUsesSpeedLimit(tor, dir));
}
static void saveSpeedLimits(tr_variant* dict, tr_torrent* tor)
static void saveSpeedLimits(tr_variant* dict, tr_torrent const* tor)
{
saveSingleSpeedLimit(tr_variantDictAddDict(dict, TR_KEY_speed_limit_down, 0), tor, TR_DOWN);
saveSingleSpeedLimit(tr_variantDictAddDict(dict, TR_KEY_speed_limit_up, 0), tor, TR_UP);
}
static void saveRatioLimits(tr_variant* dict, tr_torrent* tor)
static void saveRatioLimits(tr_variant* dict, tr_torrent const* tor)
{
tr_variant* d = tr_variantDictAddDict(dict, TR_KEY_ratio_limit, 2);
tr_variantDictAddReal(d, TR_KEY_ratio_limit, tr_torrentGetRatioLimit(tor));
tr_variantDictAddInt(d, TR_KEY_ratio_mode, tr_torrentGetRatioMode(tor));
}
static void saveIdleLimits(tr_variant* dict, tr_torrent* tor)
static void saveIdleLimits(tr_variant* dict, tr_torrent const* tor)
{
tr_variant* d = tr_variantDictAddDict(dict, TR_KEY_idle_limit, 2);
tr_variantDictAddInt(d, TR_KEY_idle_limit, tr_torrentGetIdleLimit(tor));
@@ -305,17 +305,14 @@ static uint64_t loadRatioLimits(tr_variant* dict, tr_torrent* tor)
{
auto ret = uint64_t{};
tr_variant* d = nullptr;
if (tr_variantDictFindDict(dict, TR_KEY_ratio_limit, &d))
if (tr_variant* d = nullptr; tr_variantDictFindDict(dict, TR_KEY_ratio_limit, &d))
{
auto dratio = double{};
if (tr_variantDictFindReal(d, TR_KEY_ratio_limit, &dratio))
if (auto dratio = double{}; tr_variantDictFindReal(d, TR_KEY_ratio_limit, &dratio))
{
tr_torrentSetRatioLimit(tor, dratio);
}
auto i = int64_t{};
if (tr_variantDictFindInt(d, TR_KEY_ratio_mode, &i))
if (auto i = int64_t{}; tr_variantDictFindInt(d, TR_KEY_ratio_mode, &i))
{
tr_torrentSetRatioMode(tor, tr_ratiolimit(i));
}
@@ -330,17 +327,14 @@ static uint64_t loadIdleLimits(tr_variant* dict, tr_torrent* tor)
{
auto ret = uint64_t{};
tr_variant* d = nullptr;
if (tr_variantDictFindDict(dict, TR_KEY_idle_limit, &d))
if (tr_variant* d = nullptr; tr_variantDictFindDict(dict, TR_KEY_idle_limit, &d))
{
auto imin = int64_t{};
if (tr_variantDictFindInt(d, TR_KEY_idle_limit, &imin))
if (auto imin = int64_t{}; tr_variantDictFindInt(d, TR_KEY_idle_limit, &imin))
{
tr_torrentSetIdleLimit(tor, imin);
}
auto i = int64_t{};
if (tr_variantDictFindInt(d, TR_KEY_idle_mode, &i))
if (auto i = int64_t{}; tr_variantDictFindInt(d, TR_KEY_idle_mode, &i))
{
tr_torrentSetIdleMode(tor, tr_idlelimit(i));
}
@@ -580,8 +574,7 @@ static uint64_t loadProgress(tr_variant* dict, tr_torrent* tor)
auto blocks = tr_bitfield{ tor->blockCount() };
char const* err = nullptr;
auto sv = std::string_view{};
tr_variant const* const b = tr_variantDictFind(prog, TR_KEY_blocks);
if (b != nullptr)
if (auto const* const b = tr_variantDictFind(prog, TR_KEY_blocks); b != nullptr)
{
uint8_t const* buf = nullptr;
auto buflen = size_t{};
@@ -678,8 +671,7 @@ void tr_torrentSaveResume(tr_torrent* tor)
saveName(&top, tor);
saveLabels(&top, tor);
auto const err = tr_variantToFile(&top, TR_VARIANT_FMT_BENC, tor->resumeFile());
if (err != 0)
if (auto const err = tr_variantToFile(&top, TR_VARIANT_FMT_BENC, tor->resumeFile()); err != 0)
{
tor->setLocalError(tr_strvJoin("Unable to save resume file: ", tr_strerror(err)));
}
libtransmission/rpc-server.cc
+1 -1
View File
@@ -342,7 +342,7 @@ static bool isIPAddressWithOptionalPort(char const* host)
return evutil_parse_sockaddr_port(host, (struct sockaddr*)&address, &address_len) != -1;
}
static bool isHostnameAllowed(tr_rpc_server const* server, struct evhttp_request* req)
static bool isHostnameAllowed(tr_rpc_server const* server, evhttp_request const* req)
{
/* If password auth is enabled, any hostname is permitted. */
if (server->isPasswordEnabled)
libtransmission/rpcimpl.cc
+4 -8
View File
@@ -1305,8 +1305,7 @@ static char const* torrentRenamePath(
auto newname = std::string_view{};
(void)tr_variantDictFindStrView(args_in, TR_KEY_name, &newname);
auto const torrents = getTorrents(session, args_in);
if (std::size(torrents) == 1)
if (auto const torrents = getTorrents(session, args_in); std::size(torrents) == 1)
{
torrents[0]->renamePath(oldpath, newname, torrentRenamePathDone, idle_data);
}
@@ -2405,8 +2404,7 @@ void tr_rpc_request_exec_json(
tr_variantDictAddDict(&response, TR_KEY_arguments, 0);
tr_variantDictAddStr(&response, TR_KEY_result, result);
auto tag = int64_t{};
if (tr_variantDictFindInt(mutable_request, TR_KEY_tag, &tag))
if (auto tag = int64_t{}; tr_variantDictFindInt(mutable_request, TR_KEY_tag, &tag))
{
tr_variantDictAddInt(&response, TR_KEY_tag, tag);
}
@@ -2429,8 +2427,7 @@ void tr_rpc_request_exec_json(
tr_variantDictAddStr(&response, TR_KEY_result, result);
auto tag = int64_t{};
if (tr_variantDictFindInt(mutable_request, TR_KEY_tag, &tag))
if (auto tag = int64_t{}; tr_variantDictFindInt(mutable_request, TR_KEY_tag, &tag))
{
tr_variantDictAddInt(&response, TR_KEY_tag, tag);
}
@@ -2446,8 +2443,7 @@ void tr_rpc_request_exec_json(
data->response = tr_new0(tr_variant, 1);
tr_variantInitDict(data->response, 3);
auto tag = int64_t{};
if (tr_variantDictFindInt(mutable_request, TR_KEY_tag, &tag))
if (auto tag = int64_t{}; tr_variantDictFindInt(mutable_request, TR_KEY_tag, &tag))
{
tr_variantDictAddInt(data->response, TR_KEY_tag, tag);
}
libtransmission/session.cc
+3 -5
View File
@@ -118,7 +118,7 @@ tr_peer_id_t tr_peerIdInit()
****
***/
tr_encryption_mode tr_sessionGetEncryption(tr_session* session)
tr_encryption_mode tr_sessionGetEncryption(tr_session const* session)
{
TR_ASSERT(session != nullptr);
@@ -600,8 +600,7 @@ tr_session* tr_sessionInit(char const* config_dir, bool messageQueuingEnabled, t
session->removed_torrents.clear();
/* nice to start logging at the very beginning */
auto i = int64_t{};
if (tr_variantDictFindInt(clientSettings, TR_KEY_message_level, &i))
if (auto i = int64_t{}; tr_variantDictFindInt(clientSettings, TR_KEY_message_level, &i))
{
tr_logSetLevel(tr_log_level(i));
}
@@ -2307,9 +2306,8 @@ static void loadBlocklists(tr_session* session)
if (!tr_sys_path_get_info(binname.c_str(), 0, &binname_info, nullptr)) /* create it */
{
tr_blocklistFile* b = tr_blocklistFileNew(binname.c_str(), isEnabled);
int const n = tr_blocklistFileSetContent(b, path.c_str());
if (n > 0)
if (int const n = tr_blocklistFileSetContent(b, path.c_str()); n > 0)
{
load = binname;
}
libtransmission/torrent-magnet.cc
+3 -4
View File
@@ -274,8 +274,7 @@ static bool useNewMetainfo(tr_torrent* tor, tr_incomplete_metadata const* m, tr_
}
// save it
auto const filename = tor->torrentFile();
if (!tr_saveFile(filename, benc, error))
if (auto const filename = tor->torrentFile(); !tr_saveFile(filename, benc, error))
{
return false;
}
@@ -370,9 +369,9 @@ bool tr_torrentGetNextMetadataRequest(tr_torrent* tor, time_t now, int* setme_pi
TR_ASSERT(tr_isTorrent(tor));
bool have_request = false;
struct tr_incomplete_metadata* m = tor->incompleteMetadata;
if (m != nullptr && m->piecesNeededCount > 0 && m->piecesNeeded[0].requestedAt + MinRepeatIntervalSecs < now)
if (auto* m = tor->incompleteMetadata;
m != nullptr && m->piecesNeededCount > 0 && m->piecesNeeded[0].requestedAt + MinRepeatIntervalSecs < now)
{
int const piece = m->piecesNeeded[0].piece;
tr_removeElementFromArray(m->piecesNeeded, 0, sizeof(struct metadata_node), m->piecesNeededCount);
libtransmission/torrent.cc
+3 -4
View File
@@ -336,8 +336,7 @@ static bool tr_torrentGetSeedRatioBytes(tr_torrent const* tor, uint64_t* setme_l
TR_ASSERT(tr_isTorrent(tor));
auto seed_ratio = double{};
if (tr_torrentGetSeedRatio(tor, &seed_ratio))
if (auto seed_ratio = double{}; tr_torrentGetSeedRatio(tor, &seed_ratio))
{
auto const uploaded = tor->uploadedCur + tor->uploadedPrev;
auto const baseline = tor->totalSize();
@@ -2340,8 +2339,8 @@ static void setLocationImpl(void* vdata)
auto const file_size = tor->fileSize(i);
char const* oldbase = nullptr;
char* sub = nullptr;
if (tr_torrentFindFile2(tor, i, &oldbase, &sub, nullptr))
if (char* sub = nullptr; tr_torrentFindFile2(tor, i, &oldbase, &sub, nullptr))
{
auto const oldpath = tr_strvPath(oldbase, sub);
auto const newpath = tr_strvPath(location, sub);
libtransmission/tr-dht.cc
+4 -5
View File
@@ -104,8 +104,8 @@ static void bootstrap_from_name(char const* name, tr_port port, int af)
tr_snprintf(pp, sizeof(pp), "%d", (int)port);
addrinfo* info = nullptr;
int const rc = getaddrinfo(name, pp, &hints, &info);
if (rc != 0)
if (int const rc = getaddrinfo(name, pp, &hints, &info); rc != 0)
{
tr_logAddNamedError("DHT", "%s:%s: %s", name, pp, gai_strerror(rc));
return;
@@ -527,7 +527,7 @@ int tr_dhtStatus(tr_session* session, int af, int* nodes_return)
return closure.status;
}
tr_port tr_dhtPort(tr_session* ss)
tr_port tr_dhtPort(tr_session const* ss)
{
return tr_dhtEnabled(ss) ? ss->udp_port : 0;
}
@@ -738,9 +738,8 @@ void tr_dhtCallback(unsigned char* buf, int buflen, struct sockaddr* from, sockl
}
time_t tosleep = 0;
int rc = dht_periodic(buf, buflen, from, fromlen, &tosleep, callback, nullptr);
if (rc < 0)
if (int rc = dht_periodic(buf, buflen, from, fromlen, &tosleep, callback, nullptr); rc < 0)
{
if (errno == EINTR)
{
libtransmission/tr-dht.h
+1 -1
View File
@@ -20,7 +20,7 @@ enum
int tr_dhtInit(tr_session*);
void tr_dhtUninit(tr_session*);
bool tr_dhtEnabled(tr_session const*);
tr_port tr_dhtPort(tr_session*);
tr_port tr_dhtPort(tr_session const*);
int tr_dhtStatus(tr_session*, int af, int* setme_nodeCount);
char const* tr_dhtPrintableStatus(int status);
bool tr_dhtAddNode(tr_session*, tr_address const*, tr_port, bool bootstrap);
libtransmission/tr-getopt.cc
+1 -2
View File
@@ -62,8 +62,7 @@ static void getopts_usage_line(tr_option const* opt, int longWidth, int shortWid
auto const strip_leading_whitespace = [](std::string_view text)
{
auto pos = text.find_first_not_of(' ');
if (pos != std::string_view::npos)
if (auto pos = text.find_first_not_of(' '); pos != std::string_view::npos)
{
text.remove_prefix(pos);
}
libtransmission/tr-lpd.cc
+1 -2
View File
@@ -138,10 +138,9 @@ static char const* lpd_extractHeader(char const* s, struct lpd_protocolVersion*
int major = -1;
int minor = -1;
size_t len = strlen(s);
/* something might be rotten with this chunk of data */
if (len == 0 || len > lpd_maxDatagramLength)
if (auto len = strlen(s); len == 0 || len > lpd_maxDatagramLength)
{
return nullptr;
}
libtransmission/transmission.h
+1 -1
View File
@@ -478,7 +478,7 @@ void tr_sessionSetLPDEnabled(tr_session* session, bool enabled);
void tr_sessionSetCacheLimit_MB(tr_session* session, int mb);
int tr_sessionGetCacheLimit_MB(tr_session const* session);
tr_encryption_mode tr_sessionGetEncryption(tr_session* session);
tr_encryption_mode tr_sessionGetEncryption(tr_session const* session);
void tr_sessionSetEncryption(tr_session* session, tr_encryption_mode mode);
/***********************************************************************
libtransmission/utils.h
+6 -6
View File
@@ -83,8 +83,7 @@ template<typename... T, typename std::enable_if_t<(std::is_convertible_v<T, std:
std::string& tr_buildBuf(std::string& setme, T... args)
{
setme.clear();
auto const n = (std::size(std::string_view{ args }) + ...);
if (setme.capacity() < n)
if (auto const n = (std::size(std::string_view{ args }) + ...); setme.capacity() < n)
{
setme.reserve(n);
}
@@ -301,8 +300,8 @@ std::string tr_strvPath(T... args)
{
auto setme = std::string{};
auto const n_args = sizeof...(args);
auto const n = n_args + (std::size(std::string_view{ args }) + ...);
if (setme.capacity() < n)
if (auto const n = n_args + (std::size(std::string_view{ args }) + ...); setme.capacity() < n)
{
setme.reserve(n);
}
@@ -321,11 +320,12 @@ template<typename... T, typename std::enable_if_t<(std::is_convertible_v<T, std:
std::string tr_strvJoin(T... args)
{
auto setme = std::string{};
auto const n = (std::size(std::string_view{ args }) + ...);
if (setme.capacity() < n)
if (auto const n = (std::size(std::string_view{ args }) + ...); setme.capacity() < n)
{
setme.reserve(n);
}
((setme += args), ...);
return setme;
}
libtransmission/variant-benc.cc
+1 -2
View File
@@ -59,8 +59,7 @@ std::optional<int64_t> ParseInt(std::string_view* benc)
// find the ending delimiter
walk.remove_prefix(std::size(Prefix));
auto const pos = walk.find(Suffix);
if (pos == std::string_view::npos)
if (auto const pos = walk.find(Suffix); pos == std::string_view::npos)
{
return {};
}
libtransmission/variant-json.cc
+5 -9
View File
@@ -448,17 +448,13 @@ static void jsonChildFunc(struct jsonWalk* data)
}
case TR_VARIANT_TYPE_LIST:
++pstate.childIndex;
if (bool const is_last = pstate.childIndex == pstate.childCount; !is_last)
{
++pstate.childIndex;
bool const is_last = pstate.childIndex == pstate.childCount;
if (!is_last)
{
evbuffer_add(data->out, ",", 1);
jsonIndent(data);
}
break;
evbuffer_add(data->out, ",", 1);
jsonIndent(data);
}
break;
default:
break;
libtransmission/variant.cc
+3 -7
View File
@@ -354,8 +354,7 @@ bool tr_variantGetBool(tr_variant const* v, bool* setme)
return true;
}
auto sv = std::string_view{};
if (tr_variantGetStrView(v, &sv))
if (auto sv = std::string_view{}; tr_variantGetStrView(v, &sv))
{
if (sv == "true"sv)
{
@@ -504,9 +503,7 @@ static tr_variant* containerReserve(tr_variant* v, size_t count)
{
TR_ASSERT(tr_variantIsContainer(v));
size_t const needed = v->val.l.count + count;
if (needed > v->val.l.alloc)
if (size_t const needed = v->val.l.count + count; needed > v->val.l.alloc)
{
/* scale the alloc size in powers-of-2 */
size_t n = v->val.l.alloc != 0 ? v->val.l.alloc : 8;
@@ -731,9 +728,8 @@ tr_variant* tr_variantDictSteal(tr_variant* dict, tr_quark const key, tr_variant
bool tr_variantDictRemove(tr_variant* dict, tr_quark const key)
{
bool removed = false;
int const i = dictIndexOf(dict, key);
if (i >= 0)
if (int const i = dictIndexOf(dict, key); i >= 0)
{
int const last = (int)dict->val.l.count - 1;
libtransmission/web-utils.cc
+1 -2
View File
@@ -304,8 +304,7 @@ std::optional<tr_url_parsed_t> tr_urlParse(std::string_view url)
// So many magnet links are malformed, e.g. not escaping text
// in the display name, that we're better off handling magnets
// as a special case before even scanning for invalid chars.
auto constexpr MagnetStart = "magnet:?"sv;
if (tr_strvStartsWith(url, MagnetStart))
if (auto constexpr MagnetStart = "magnet:?"sv; tr_strvStartsWith(url, MagnetStart))
{
parsed.scheme = "magnet"sv;
parsed.query = url.substr(std::size(MagnetStart));
libtransmission/web.cc
+1 -2
View File
@@ -422,8 +422,7 @@ static void tr_webThreadFunc(void* vsession)
tr_logAddNamedInfo("web", "NB: invalid certs will show up as 'Could not connect to tracker' like many other errors");
}
auto const str = tr_strvPath(session->config_dir, "cookies.txt");
if (tr_sys_path_exists(str.c_str(), nullptr))
if (auto const str = tr_strvPath(session->config_dir, "cookies.txt"); tr_sys_path_exists(str.c_str(), nullptr))
{
web->cookie_filename = tr_strvDup(str);
}
libtransmission/webseed.cc
+1 -2
View File
@@ -213,8 +213,7 @@ static void write_block_func(void* vdata)
struct tr_webseed* const w = data->webseed;
struct evbuffer* const buf = data->content;
auto* const tor = tr_torrentFindFromId(data->session, data->torrent_id);
if (tor != nullptr)
if (auto* const tor = tr_torrentFindFromId(data->session, data->torrent_id); tor != nullptr)
{
uint32_t const block_size = tor->blockSize();
uint32_t len = evbuffer_get_length(buf);
qt/Application.cc
+2 -7
View File
@@ -175,9 +175,7 @@ Application::Application(int& argc, char** argv)
// try to delegate the work to an existing copy of Transmission
// before starting ourselves...
InteropHelper interop_client;
if (interop_client.isConnected())
if (InteropHelper interop_client; interop_client.isConnected())
{
bool delegated = false;
@@ -605,9 +603,7 @@ void Application::raise() const
bool Application::notifyApp(QString const& title, QString const& body, QStringList const& actions) const
{
#ifdef QT_DBUS_LIB
QDBusConnection bus = QDBusConnection::sessionBus();
if (bus.isConnected())
if (auto bus = QDBusConnection::sessionBus(); bus.isConnected())
{
QDBusMessage m = QDBusMessage::createMethodCall(
fdo_notifications_service_name_,
@@ -633,7 +629,6 @@ bool Application::notifyApp(QString const& title, QString const& body, QStringLi
return true;
}
}
#endif
window_->trayIcon().showMessage(title, body);
qt/DBusInteropHelper.cc
+1 -2
View File
@@ -39,8 +39,7 @@ void DBusInteropHelper::registerObject(QObject* parent)
return;
}
auto const service_name = QStringLiteral("com.transmissionbt.Transmission");
if (!bus.registerService(service_name))
if (auto const service_name = QStringLiteral("com.transmissionbt.Transmission"); !bus.registerService(service_name))
{
qWarning() << "couldn't register" << qPrintable(service_name);
}
qt/DetailsDialog.cc
+1 -2
View File
@@ -84,8 +84,7 @@ QString collateAddress(QString const& address)
{
QString collated;
QHostAddress ip_address;
if (ip_address.setAddress(address))
if (QHostAddress ip_address; ip_address.setAddress(address))
{
if (ip_address.protocol() == QAbstractSocket::IPv4Protocol)
{
qt/FaviconCache.cc
+2 -4
View File
@@ -108,8 +108,7 @@ void FaviconCache::ensureCacheDirHasBeenScanned()
// remember which hosts we've asked for a favicon so that we
// don't re-ask them every time we start a new session
auto skip_file = QFile(getScrapedFile());
if (skip_file.open(QIODevice::ReadOnly | QIODevice::Text))
if (auto skip_file = QFile(getScrapedFile()); skip_file.open(QIODevice::ReadOnly | QIODevice::Text))
{
while (!skip_file.atEnd())
{
@@ -183,8 +182,7 @@ FaviconCache::Key FaviconCache::add(QString const& url_str)
ensureCacheDirHasBeenScanned();
// find or add this url's key
auto k_it = keys_.find(url_str);
if (k_it != keys_.end())
if (auto k_it = keys_.find(url_str); k_it != keys_.end())
{
return k_it->second;
}
qt/FileTreeItem.cc
+1 -3
View File
@@ -57,9 +57,7 @@ FileTreeItem* FileTreeItem::child(QString const& filename)
{
FileTreeItem* item(nullptr);
int const row = getMyChildRows().value(filename, -1);
if (row != -1)
if (int const row = getMyChildRows().value(filename, -1); row != -1)
{
item = child(row);
assert(filename == item->name());
qt/FileTreeView.cc
+1 -3
View File
@@ -173,9 +173,7 @@ void FileTreeView::mouseDoubleClickEvent(QMouseEvent* event)
void FileTreeView::contextMenuEvent(QContextMenuEvent* event)
{
QModelIndex const root_index = model_->index(0, 0);
if (!root_index.isValid())
if (auto const root_index = model_->index(0, 0); !root_index.isValid())
{
return;
}
qt/FilterBarComboBox.cc
+1 -3
View File
@@ -83,9 +83,7 @@ void FilterBarComboBox::paintEvent(QPaintEvent* e)
painter.drawComplexControl(QStyle::CC_ComboBox, opt);
// draw the icon and text
QModelIndex const model_index = model()->index(currentIndex(), 0, rootModelIndex());
if (model_index.isValid())
if (auto const model_index = model()->index(currentIndex(), 0, rootModelIndex()); model_index.isValid())
{
QStyle* s = style();
int const hmargin = getHSpacing(this);
qt/FreeSpaceLabel.cc
+1 -2
View File
@@ -76,8 +76,7 @@ void FreeSpaceLabel::onTimer()
[this](RpcResponse const& r)
{
// update the label
auto const bytes = dictFind<int64_t>(r.args.get(), TR_KEY_size_bytes);
if (bytes && *bytes > 1)
if (auto const bytes = dictFind<int64_t>(r.args.get(), TR_KEY_size_bytes); bytes && *bytes > 1)
{
setText(tr("%1 free").arg(Formatter::get().sizeToString(*bytes)));
}
qt/InteropObject.cc
+1 -3
View File
@@ -22,9 +22,7 @@ bool InteropObject::PresentWindow() const
// NOLINTNEXTLINE(readability-identifier-naming)
bool InteropObject::AddMetainfo(QString const& metainfo) const
{
AddData addme(metainfo);
if (addme.type != addme.NONE)
if (AddData addme(metainfo); addme.type != addme.NONE)
{
trApp->addTorrent(addme);
}
qt/MainWindow.cc
+1 -2
View File
@@ -658,9 +658,8 @@ void MainWindow::openFolder()
}
QString const first_file = files.at(0).filename;
int slash_index = first_file.indexOf(QLatin1Char('/'));
if (slash_index > -1)
if (int slash_index = first_file.indexOf(QLatin1Char('/')); slash_index > -1)
{
path = path + QLatin1Char('/') + first_file.left(slash_index);
}
qt/MakeDialog.cc
+1 -3
View File
@@ -216,9 +216,7 @@ void MakeDialog::onSourceChanged()
{
builder_.reset();
QString const filename = getSource();
if (!filename.isEmpty())
if (auto const filename = getSource(); !filename.isEmpty())
{
builder_.reset(tr_metaInfoBuilderCreate(filename.toUtf8().constData()));
}
qt/PathButton.cc
+1 -3
View File
@@ -112,9 +112,7 @@ void PathButton::onClicked() const
dialog->setNameFilter(name_filter_);
}
QFileInfo const path_info(path_);
if (!path_.isEmpty() && path_info.exists())
if (QFileInfo const path_info(path_); !path_.isEmpty() && path_info.exists())
{
if (path_info.isDir())
{
qt/Prefs.cc
+3 -4
View File
@@ -38,8 +38,8 @@ namespace
void ensureSoundCommandIsAList(tr_variant* dict)
{
tr_quark key = TR_KEY_torrent_complete_sound_command;
tr_variant* list = nullptr;
if (tr_variantDictFindList(dict, key, &list))
if (tr_variant* list = nullptr; tr_variantDictFindList(dict, key, &list))
{
return;
}
@@ -507,9 +507,8 @@ bool Prefs::getBool(int key) const
QString Prefs::getString(int key) const
{
assert(Items[key].type == QVariant::String);
QByteArray const b = values_[key].toByteArray();
if (isValidUtf8(b.constData()))
if (auto const b = values_[key].toByteArray(); isValidUtf8(b.constData()))
{
values_[key].setValue(QString::fromUtf8(b.constData()));
}
qt/PrefsDialog.cc
+7 -16
View File
@@ -206,36 +206,31 @@ void PrefsDialog::linkWidgetToPref(QWidget* widget, int pref_key)
updateWidgetValue(widget, pref_key);
widgets_.insert(pref_key, widget);
auto const* check_box = qobject_cast<QCheckBox*>(widget);
if (check_box != nullptr)
if (auto const* check_box = qobject_cast<QCheckBox*>(widget); check_box != nullptr)
{
connect(check_box, &QAbstractButton::toggled, this, &PrefsDialog::checkBoxToggled);
return;
}
auto const* time_edit = qobject_cast<QTimeEdit*>(widget);
if (time_edit != nullptr)
if (auto const* time_edit = qobject_cast<QTimeEdit*>(widget); time_edit != nullptr)
{
connect(time_edit, &QAbstractSpinBox::editingFinished, this, &PrefsDialog::timeEditingFinished);
return;
}
auto const* line_edit = qobject_cast<QLineEdit*>(widget);
if (line_edit != nullptr)
if (auto const* line_edit = qobject_cast<QLineEdit*>(widget); line_edit != nullptr)
{
connect(line_edit, &QLineEdit::editingFinished, this, &PrefsDialog::lineEditingFinished);
return;
}
auto const* path_button = qobject_cast<PathButton*>(widget);
if (path_button != nullptr)
if (auto const* path_button = qobject_cast<PathButton*>(widget); path_button != nullptr)
{
connect(path_button, &PathButton::pathChanged, this, &PrefsDialog::pathChanged);
return;
}
auto const* spin_box = qobject_cast<QAbstractSpinBox*>(widget);
if (spin_box != nullptr)
if (auto const* spin_box = qobject_cast<QAbstractSpinBox*>(widget); spin_box != nullptr)
{
connect(spin_box, &QAbstractSpinBox::editingFinished, this, &PrefsDialog::spinBoxEditingFinished);
}
@@ -281,9 +276,7 @@ void PrefsDialog::lineEditingFinished()
if (pref_widget.is<QLineEdit>())
{
auto const* const line_edit = pref_widget.as<QLineEdit>();
if (line_edit->isModified())
if (auto const* const line_edit = pref_widget.as<QLineEdit>(); line_edit->isModified())
{
setPref(pref_widget.getPrefKey(), line_edit->text());
}
@@ -681,9 +674,7 @@ void PrefsDialog::setPref(int key, QVariant const& v)
void PrefsDialog::sessionUpdated()
{
bool const is_local = session_.isLocal();
if (is_local_ != is_local)
if (bool const is_local = session_.isLocal(); is_local_ != is_local)
{
is_local_ = is_local;
updateDownloadingWidgetsLocality();
qt/RpcClient.cc
+2 -5
View File
@@ -301,16 +301,13 @@ RpcResponse RpcClient::parseResponseData(tr_variant& json) const
{
RpcResponse ret;
auto const result = dictFind<QString>(&json, TR_KEY_result);
if (result)
if (auto const result = dictFind<QString>(&json, TR_KEY_result); result)
{
ret.result = *result;
ret.success = *result == QStringLiteral("success");
}
tr_variant* args;
if (tr_variantDictFindDict(&json, TR_KEY_arguments, &args))
if (tr_variant * args; tr_variantDictFindDict(&json, TR_KEY_arguments, &args))
{
ret.args = createVariant();
*ret.args = *args;
qt/Session.cc
+24 -51
View File
@@ -199,26 +199,21 @@ void Session::updatePref(int key)
break;
case Prefs::ENCRYPTION:
switch (int const i = prefs_.variant(key).toInt(); i)
{
int const i = prefs_.variant(key).toInt();
case 0:
sessionSet(prefs_.getKey(key), QStringLiteral("tolerated"));
break;
switch (i)
{
case 0:
sessionSet(prefs_.getKey(key), QStringLiteral("tolerated"));
break;
case 1:
sessionSet(prefs_.getKey(key), QStringLiteral("preferred"));
break;
case 2:
sessionSet(prefs_.getKey(key), QStringLiteral("required"));
break;
}
case 1:
sessionSet(prefs_.getKey(key), QStringLiteral("preferred"));
break;
case 2:
sessionSet(prefs_.getKey(key), QStringLiteral("required"));
break;
}
break;
case Prefs::RPC_AUTH_REQUIRED:
if (session_ != nullptr)
@@ -907,68 +902,46 @@ void Session::updateInfo(tr_variant* d)
switch (prefs_.type(i))
{
case QVariant::Int:
if (auto const value = getValue<int>(b); value)
{
auto const value = getValue<int>(b);
if (value)
{
prefs_.set(i, *value);
}
break;
prefs_.set(i, *value);
}
break;
case QVariant::Double:
if (auto const value = getValue<double>(b); value)
{
auto const value = getValue<double>(b);
if (value)
{
prefs_.set(i, *value);
}
break;
prefs_.set(i, *value);
}
break;
case QVariant::Bool:
if (auto const value = getValue<bool>(b); value)
{
auto const value = getValue<bool>(b);
if (value)
{
prefs_.set(i, *value);
}
break;
prefs_.set(i, *value);
}
break;
case CustomVariantType::FilterModeType:
case CustomVariantType::SortModeType:
case QVariant::String:
if (auto const value = getValue<QString>(b); value)
{
auto const value = getValue<QString>(b);
if (value)
{
prefs_.set(i, *value);
}
break;
prefs_.set(i, *value);
}
break;
default:
break;
}
}
auto const b = dictFind<bool>(d, TR_KEY_seedRatioLimited);
if (b)
if (auto const b = dictFind<bool>(d, TR_KEY_seedRatioLimited); b)
{
prefs_.set(Prefs::RATIO_ENABLED, *b);
}
auto const x = dictFind<double>(d, TR_KEY_seedRatioLimit);
if (x)
if (auto const x = dictFind<double>(d, TR_KEY_seedRatioLimit); x)
{
prefs_.set(Prefs::RATIO, *x);
}
qt/VariantHelpers.h
+6 -6
View File
@@ -32,8 +32,8 @@ template<typename T, typename std::enable_if_t<std::is_same_v<T, bool>>* = nullp
auto getValue(tr_variant const* variant)
{
std::optional<T> ret;
auto value = T{};
if (tr_variantGetBool(variant, &value))
if (auto value = T{}; tr_variantGetBool(variant, &value))
{
ret = value;
}
@@ -49,8 +49,8 @@ template<
auto getValue(tr_variant const* variant)
{
std::optional<T> ret;
auto value = int64_t{};
if (tr_variantGetInt(variant, &value))
if (auto value = int64_t{}; tr_variantGetInt(variant, &value))
{
ret = value;
}
@@ -62,8 +62,8 @@ template<typename T, typename std::enable_if_t<std::is_same_v<T, double>>* = nul
auto getValue(tr_variant const* variant)
{
std::optional<T> ret;
auto value = T{};
if (tr_variantGetReal(variant, &value))
if (auto value = T{}; tr_variantGetReal(variant, &value))
{
ret = value;
}
utils/remote.cc
+2 -4
View File
@@ -544,8 +544,7 @@ static bool UseSSL = false;
static std::string getEncodedMetainfo(char const* filename)
{
auto contents = std::vector<char>{};
if (tr_loadFile(contents, filename))
if (auto contents = std::vector<char>{}; tr_loadFile(contents, filename))
{
return tr_base64_encode({ std::data(contents), std::size(contents) });
}
@@ -2158,8 +2157,7 @@ static int flush(char const* rpcurl, tr_variant** benc)
fprintf(stderr, "posting:\n--------\n%s\n--------\n", json.c_str());
}
auto const res = curl_easy_perform(curl);
if (res != CURLE_OK)
if (auto const res = curl_easy_perform(curl); res != CURLE_OK)
{
tr_logAddNamedError(MyName, " (%s) %s", rpcurl_http.c_str(), curl_easy_strerror(res));
status |= EXIT_FAILURE;