Fix -Wcast-function-type warnings in GTK+ app code

https://gitlab.gnome.org/GNOME/gnome-terminal/issues/96 talks about both
the issue and its solution.

GCC 8's -Wcast-function-type, enabled by -Wextra, is problematic in glib
applications because it's idiomatic there to recast function signatures,
e.g. `g_slist_free(list, (GFunc)g_free, NULL);`.

Disabling the warning with pragmas causes "unrecognized pragma" warnings
on clang and older versions of gcc, and disabling the warning could miss
actual bugs. GCC defines `void (*)(void)` as a special case that matches
anything so we can silence warnings by double-casting through GCallback.

In the previous example, the warning is silenced by changing the code to
read `g_slist_free(list, (GFunc)(GCallback)g_free, NULL);`).

fixup! fix 'increases required alignment' warning

make building quieter so warnings are more visible

fix net.c 'increases required alignment' warning

The code passes in a `struct sockaddr_storage*` which is a padded struct
large enough for the necessary alignment. Unfortunately it was recast as
a `struct sockaddr*` which has less padding and a smaller alignment. The
warning occrred because of these differing alignments.

fix 'increases required alignment' warning

Caused from storing int16_t's in a char array.

fix "unused in lambda capture" warnings by clang++

fix: add DISABLE_WARNINGS option for SunPro Studio

chore: uncrustify

fix: pthread_create's start_routine's return value

This was defined as `void` on non-Windows but should have been `void*`

refactor: fix libtransmission deprecation warnings

Merge branch 'fix-gcc-warnings' of github.com:transmission/transmission into fix-gcc-warnings

Merge branch 'master' into fix-gcc-warnings

Merge branch 'master' into fix-gcc-warnings

Update Uncrustify config to 0.69

Use explicit boolean conversions (follow-up)

Use explicit boolean conversions

Fixup code style

Support OFD locks and missing flock

Introduce `tr_str_is_empty` to relay intent better

Fully cover `tr_metainfo_sanitize_path_component`

Sanitize suspicious path components instead of rejecting them

Apply the same rules on all the supported platforms to avoid issues
with network shares and alien file systems.

For future compatibility, explicitly mark adjusted paths as renamed.

Fixes: #294

Add missing declaration for `tr_strcasestr`

Load CA certs from system store on Windows / OpenSSL

Fixes: #446

Update curl and openssl to those in 10.10 SDK (macOS)

Only add revision to filename for nightly builds (MSI package)

Include Windows Vista Qt style, remove dbus revision suffix and expat (MSI package)

Support OpenSSL 1.1 x64 file names (MSI package)

Support OpenSSL 1.1 file names (MSI package)

Support MSVS 2017 VC CRT MSM file locations (MSI package)

Fixup invalid daemon foreground mode check (2cc996cb77)

Merge branch 'master' into fix-gcc-warnings

use -w for DISABLE_WARNINGS in Clang

fixup! silence spurious alignment warning

Update libtransmission/CMakeLists.txt

Co-Authored-By: ckerr <ckerr@github.com>

Parse session-id header case-insensitively (#765)

RFC 2616 defines headers as case-insensitive, so if rpc is behind a
reverse proxy that lowers the case of headers, transmission will not
parse them correctly.

A new wrapper function, `tr_strcasestr` is added to
libtransmission/utils.c to allow for comparisons of headers case
insensitively, and checks in cmake and autogen are included.

Mark a couple of MSI properties as secure

They are being discarded by server side with "ignoring disallowed
property" messages otherwise which leads to failed checks using those
properties.

Fixes: #451

Remove useless parentheses

Define each identifier in a dedicated statement

* MISRA C++:2008, 8-0-1 - An init-declarator-list or a member-declarator-list
  shall consist of a single init-declarator or member-declarator respectively
* CERT, DCL52-J. - Do not declare more than one variable per declaration
* CERT, DCL04-C. - Do not declare more than one variable per declaration

Remove side effects from right hand operands of && or ||

* MISRA C:2004, 12.4 - The right-hand operand of a logical && or || operator
  shall not contain side effects.
* MISRA C++:2008, 5-14-1 - The right hand operand of a logical && or ||
  operator shall not contain side effects.
* MISRA C:2012, 13.5 - The right hand operand of a logical && or || operator
  shall not contain persistent side effects
* CERT, EXP02-C. - Be aware of the short-circuit behavior of the logical AND
  and OR operators

Extract assignments from expression

Assignments explicitly enclosed in parentheses are ignored.

* MISRA C:2004, 13.1 - Assignment operators shall not be used in expressions
  that yield a Boolean value
* MISRA C++:2008, 6-2-1 - Assignment operators shall not be used in
  sub-expressions
* MISRA C:2012, 13.4 - The result of an assignment operator should not be used
* MITRE, CWE-481 - Assigning instead of Comparing
* CERT, EXP45-C. - Do not perform assignments in selection statements
* CERT, EXP51-J. - Do not perform assignments in conditional expressions

Uppercase the literal suffixes

* MISRA C++:2008, 2-13-4 - Literal suffixes shall be upper case
* MISRA C:2012, 7.3 - The lowercase character "l" shall not be used in a
  literal suffix
* CERT DCL16-C. - Use "L," not "l," to indicate a long value
* CERT, DCL50-J. - Use visually distinct identifiers

Merge pull request #859 from fishead/fix-typo

fix typo

typo

Refactor daemon startup a bit

Fix exit code to be zero when dumping settings along the way.

Closes: #487

Actually add new languages to the build

Attempt to explain something to Coverity

I'm seeing 7 potentially false reported issues re. use of uninitialized buffer
data here. It seems that `read` function should be properly supported by
Coverity, but maybe it can't figure out the postconditions of it properly...
Walking in the dark here.

Let compiler calculate static quark lenghts for us

Fix build break from previous commit on old Clangs

Remove static assert altogether: it should have worked everywhere anyway,
otherwise there is no point.

Add a few casts here and there to ensure proper type for directory handle.

Fix build on E2K with LCC (patch by Michael Shigorin, ALT Linux)

Add new translations from Transifex (95% and higher)

GTK client: Portuguese (Portugal).

Qt client: Afrikaans, Catalan, Danish, Greek, Norwegian Bokmål, Slovenian.

Mac client: none (there're suitable languages, but they're lacking XIB files
and we haven't switched to base localization yet).

Sync existing translations with Transifex

Merge pull request #584 from userwithuid/rmintltool

cmake: replace intltool with gettext

fixup! fix: __attribute__(__printf__) warnings

Merge pull request #850 from transmission/code-style-script

Add code style script and Dockerfile

Add code style script and Dockerfile

remove uncrustify-test.sh

that's probably off-topic for this PR

make uncrustify happy

fixup! fix: implicit fallthrough warning

fixup! fix: disable warnings for 3rd party code

Merge branch 'master' into rmintltool

silence spurious alignment warning

Xrefs
Discussion: https://stackoverflow.com/a/35554349
Macro inspiration: https://pagure.io/SSSD/sssd/blob/90ac46f71068d131391492360a8553bdd005b5a7/f/src/util/util_safealign.h#_35

fixup! fix: disable warnings for 3rd party code

Merge branch 'fix-gcc-warnings' of github.com:transmission/transmission into fix-gcc-warnings

fixup! fix: disable warnings for 3rd party code

Merge branch 'master' into fix-gcc-warnings

widen the torrent-id column for cleaner formatting (#840)

Under the current code, if there are over 9,999 torrents the
format will shift over one space when bumping from 9999 to 10000.

fix: Coverity warnings (#842)

* Silence coverity CHECKED_RETURN on added.f load

The existing code behaved alright since added.f is optional.
However, by testing for success we can both silence the warning
and prevent a useless initialization of NULL/0 to added_f and
added_f_length.

* Silence coverity CHECKED_RETURN on added6.f load

ipv6 variant of previous commit.

* Silence coverity CHECKED_RETURN writing benc strs

saveStringFunc() gets the target string by calling tr_variantGetStr().
It previously didn't check to see if this function succeeded because
saveStringFunc() isn't reached without the type already being known.
However, checking the return value costs nothing and makes Coverity happy.

* Silence coverity CHECKED_RETURN on ut metadata

Like earlier few Coverity commits in this PR, we're handling optional
values by declaring stack locals set to the default (e.g. -1) and then
trying to read the variant.

Unlike the earlier commits, there is a two-part step to thise read:
checking for the metadata, then checking for the individual fields.
The earlier fixes' aproach -- e.g. initializing to -1 only if the reads
failed -- would involve new nested conditionals. I find the new complexity
to outweigh the benefit of removing the dead store, so in this case I'm
casting the return value to `(void)` to tell Coverity to shush.

* Silence coverity CHECKED_RETURN on scrape

Check the return value of tr_variantGetInt() when showing
seeder and leecher counts in transmission-show.

* Silence CHECKED_RETURN on rpc recently-active

When building a list of removed torrent IDs from variants, confirm that
we can read the IDs from the variants before adding them to the list.
I don't _think_ this would have failed before, but Coverity's right that
it's reasonable to add a safeguard here.

* fix: better fix to serializing benc strings

The approach in 33e2ece7e5bc261566ae9e8db57be0b3247508d1 was
a little problematic: GetString() shouldn't fail here; but if
it somehow did, we still want to encode a zero-length benc string here.

* chore: make uncrustify happy

fix: disable warnings for 3rd party code

Since we want to leave upstream code as-is

fixup! fix: implicit fallthrough warning

fix: implicit fallthrough warning

fix: __attribute__(__printf__) warnings

Fixup some formatting leftovers (JS)

Add labels feature (#822)

* applied changes from https://github.com/Elbandi/transmission/tree/elbandi/labels to official transmission repo

* Fix compilation errors

* Address review comments

Changed `tr_ptrArray* labels` to `tr_ptrArray labels`;
Removed tr_ptrArrayNew() tr_ptrArrayDup() tr_ptrArrayFree()
Use tr_strsep() to split string by delimiters
Update transmission-remote.1
Update rpc-spec.txt

* Fix warning, address comments

* Rebase, fix formatting and address comments

Use uncrustify to format changed files
Fix "const <type>" -> "<type> const"
Fix small comments

* Lock torrent for setLabels, check for duplicates

* Check for empty labels in daemon

* Stop on first error

feat: make multiscrape limits adaptive (#837)

* feat: make multiscrape limits adaptive

Previously hardcoded by TR_MULTISCRAPE_MAX. This change makes
that the initial value, then incrementally lowers the value
when multiscrapes get "req too long" style errors.

* fix: don't log personal announce url

* chore: treat HTTP 400 as scrape-too-long

* chore: copyediting

* chore: copyediting

* move 'const' to the right of the type

* make conditional tests explicitly boolean

* make 'key' const

* don't lookup a value we already have

* make an array for known too-big scrape error messages

* improved multiscrape throttle logging

* fix: multiscraping of low numbers of torrents

Handle the case of getting a 'multiscrape too big' error message
back even if the user fewer than TR_MULTISCRAPE_MAX torrents.

* uncrustify

* fix oops

* refactor: remove TR_MULTISCRAPE_MIN

Is there any reason to have a minimum batch size?

* make test explicit boolean

Co-Authored-By: ckerr <ckerr@github.com>
* improve declaration of too_long_errors

Co-Authored-By: ckerr <ckerr@github.com>
* make test explicitly boolean

Co-Authored-By: ckerr <ckerr@github.com>
* make test explicitly boolean

Co-Authored-By: ckerr <ckerr@github.com>
* improve looping decl of too_long_errors

Fixup some formatting leftovers

Merge pull request #838 from transmission/uncrustify-update

Update to Uncrustify 0.68.1

Update to Uncrustify 0.68.1

Tweak a few rules in the process. Now all code in cli, daemon, gtk,
libtransmission, qt, and utils is properly formatted with no manual
intervention.

Merge pull request #825 from transmission/update-resume-file-before-invoking-torrent-scripts

fix: update the resume file before running scripts

Merge pull request #826 from Coeur/QuickLookPluginPath

Fix QuickLookPlugin reference path to be relative to group

Merge branch 'master' into update-resume-file-before-invoking-torrent-scripts

Fix QuickLookPlugin reference path to be relative to group

address dirty/save sequencing issue in prev commit

Merge pull request #741 from Artoria2e5/parseline-cidr

Support CIDR-notated blocklists

Merge pull request #816 from PeterDaveHello/improve-shellscript

fix shell script syntax errors and spacing inconsistencies

Merge pull request #795 from jelly/systemd_hardening

daemon: harden transmission-daemon.service

Merge pull request #767 from takiz/transm_show

Add option "--unsorted" for transmission-show

Merge pull request #807 from fetzu/master

Updated copyright strings to 2019

Merge pull request #811 from louib/cli_manpage_typo

Fix typo in CLI manpage.

fix: update the resume file before running scripts

This is useful if the client script is going to access
the .resume file. Ensuring an up-to-date .resume file
means that user scripts will be able to access it.

Refactor shell scripts with syntax and styling fixes

Fix typo in CLI manpage.

Updated copyright strings to 2019

Updated Transmission Project copyright dates to 2005-2019 in headers and code.

daemon: harden transmission-daemon.service

Systemd 227 introduced the option to make a service disallow elevating
privileges.

Update some selector-based alerts to block-based

More dark mode fixes

add Sparkle update to NEWS

missed one

Update Sparkle to 1.21.2

Start of updating NEWS for 3.00

Enable hardened runtime