Revert "Use if constexpr to make things a lot more readable"

This reverts commit d62ca9f87906be6f046fe9d07afb8bfb69c166e3.

Use if constexpr to make things a lot more readable

Use std::optional<bool> instead of OptionalBool

C++17

update docs to mention identd disable choice

follow coding style

optional identd

Add a few \n in the e2e error messages

Remove an unused function

Little doc typo

And this one (hostname -> address)

ref #3273

Rename a few “hostname” into “address”, regarding last commit

ref #3273

Allow to override the addresses used to connect to an IRC network

fix #3273

Fix the INSERT query for types that don’t have any Id column

Add a <item/> node in the presence of a leaving participant

fix #3339

Remove the virtual channel feature altogether

Actually display the error message from postgresql when a query failed

Log the warn/error messages coming from libpq in a correct format

Sort archive messages by date (and only then by id)

fix #3337

Because apparently the IDs are not always incrementing. We still use them to
know the order of two messages that were received at the same second (in
this case, the ID will always be incrementing).

Fix the actual_bind versions for integrals