merge - #1

Fix for #446 and #447.

Although this can already be set via mpd_host, having a separate option
is cleaner, and doesn't require setting the host when ncmpcpp already
detects it correctly.

* fix multiple values for vorbis comment

* add vorbis comment support for opus

* In #259 users request the ability to have ncmpcpp remain open when the
  terminal has a small size. Previously ncmpcpp would exit when there
  were less than 30 columns or 5 rows.
* After removing this constraint, ncmpcpp would freeze when terminal
  sizes went below those thresholds.
* Upon debugging, it became clear that ncmpcpp wasn't frozen, it was
  stuck in a very long for loop because it thought the display was
  unreasonably large (resulting from integer overflow).
* Preventing that overflow allows ncmpcpp to handle small sizes
  gracefully. No other changes to the rendering code were necessary.

Fixes #259

…rtist instead of artist (#472)

* add configurable column widths

* reorder to match config file

* fix possibly misleading documentation

* fix crash when integer out of bounds

* parse config string during initial processing

* use std::bind with length of list

* fix division by zero error

* use list_of

* change escaped_list_separator variable names

modern browsers don't support http

* Update settings.cpp

* Update settings.cpp

…t blocking further execution by throwing

Fix unresponsive stall when server error occurs

Terminals that don't restore the terminal state when a program exits display stdin incorrectly after quitting. This is not a bug in readline.h as its examples use termios in the same way.

* Replace "hack" with hacky chars

* Remove unnecessary spaces

* Add config option visualizer_spectrum_use_legacy_chars

* use visualizer_spectrum_log_scale_{x,y} to control log scaling

* change sound ellipse character to the circle instead of the bar

* oops, should be determined by x-axis scaling

* rebase to upstream

* use values from Config directly

* fix that weird first-column screen-wrapping issue when drawing

* rebase to upstream

---------

Co-authored-by: whooie <[email protected]>
Co-authored-by: Andrzej Rybczak <[email protected]>

updated links

Adaptation of #482.

Fixes #607.

Signed-off-by: Filip Kobierski <[email protected]>

Adaptation of https://github.com/spvkgn/ncmpcpp/blob/2148c716c7c0a2047d10cc231d825a44eefb6b7c/.github/workflows/build.yml

Fixes #616.

Their layout is problematic to match with regexes and changes often.

…#652)

Co-authored-by: Devon Kirk <[email protected]>

…654)

Co-authored-by: Devon Kirk <[email protected]>

libc++ 22 passes it[n] instead of *(it + n) to the comparator in std::sort.
For a non-POD value type, boost::transform_iterator returns a proxy object
from operator[]. Template argument deduction fails on the proxy, so the
templated overloads of LocaleBasedSorting::operator() do not match.

This is a libc++ bug, reported upstream as
llvm/llvm-project#223290. Until it is fixed,
replace boost::transform_iterator in NC::Menu with TransformIterator, which
returns a real reference from operator[].

Fixes #663.