Fix segfault due to missing null check...

NOP. Reformat code consistently, improve comments.

Use appropriate warn/inform/debug levels.

Actually implement auto-connection, and document it.

Move away from hardcoded channels. Centralizes information, simplifies the code, and might be helpful for future surround support.

forgot to add new option autoconnect_pattern to default conf and common.h

Add dummy option jack_autoconnect_option.

Partly revert c00199d664a1042af78ede253f45799e8bca3aa0, some 0 are actually longs. It's late :(

Declare all file-global variables as static, remove unused thread pointer.

Add myself to (C)omplain-to tag :-D

Always use NULL instead of 0 for pointers for clarity.

Use more conservative JACK client name (a simple token without whitespace).

Move jack base latency computation out of delay() into graph() to save cycles.

Add graph reorder callback, this is where we now compute the output latencies.

Make callback names simpler and more consistent. Always use &callback in setter functions to make the function pointer explicit, even though it's not syntactically necessary.

Revert "Remove client_mutex, since JACK opening and closing is now single-threaded."
That was a bit too optimistic, I guess. Must understand more about the rest of shairport-sync.
Fixes spurious segfault on program termination with CTRL-C that does not happen in gdb :)
This reverts commit 8e46c727d59334b87ea0f639160e92349f5c81d1.

Remove unused headers. <getopt.h> will come back once we implement command line parsing, the others may well be gone for good.

Remove client_mutex, since JACK opening and closing is now single-threaded.

Remove global client_is_open flag, always execute former conditional statements. We know we're open.

Remove optional is_running() API call, since it doesn't do anything useful.
We are either already running along just fine, or we are throughly dead.

Remove indirections open_if_needed() and close(), can now be integrated into API calls init and deinit.

Remove jack_stop() function pointer from API struct, since it does nothing. Checked, no segfault on program termination.

Remove audio_client_disconnect option. This is not good behaviour for a JACK client, and adds nothing.

Remove auto_client_open_interval option, always open JACK connection immediately or fail.
Remove client creation thread and wrapper function.

Fail early and fail deadly (AFAIK there is no fallback to another audio backend, so any severe JACK issue should be fatal).

remove unused globals

Remove option to configure port names. This is very unusual for a JACK client, nothing is gained by offering this, and it might confuse users.

NOP: move external API functions to the end, ordered by struct audio_output

Declare all JACK callbacks static (they are accessed via function pointers only).

Declare all functions static that are not callbacks and not in audio_jack API struct.
If there are any calls from the outside I haven't thought of, that should make them fail.

comment out some debug statements not relevant to users

Declare deinterleaving function static, it's nobody else's business.

Do not query the JACK port latency on every process() call. It can only ever change when the graph is reordered.

remove more dead code and obsolete comments

Add parentheses around macro to avoid breakage with stronger-binding operators.

NOP: remove commented-out code segments that were re-implemented

Fix logic error in counting the frames written and pending. RESTORES AUDIO!

NOP: comment flush implementation

Do not re-check JACK graph latency every time delay() is called. Do not register a latency callback with jack.

Remove dubious latency check from jack_is_running(). For now, we assume JACK is always running, which makes this function pointless, as it always returns possitive. FIXME: understand why it is needed.

remove comment - the "rolling" status of jack transport is irrelevant to shairport-sync.

move setting of time_of_latest_transfer to play(), we're now looking at ringbuffer writes, not jack output buffers, as those can't be locked to be tested atomically.

audio_occupancy_now in delay() needs to be in frames, not bytes. Still no luck...

Unset flush_please after flushing. Tentatively re-enable latency queries. Same problems...

try a solution for flush(). Compiled but not tested.

First step of replacing existing buffer with jack_ringbuffer. Compiles, but does not work, of course.

die() on failure to create ringbuffer.

Add lock-free ringbuffer. Preparation to make jack_stream_write_cb realtime safe.

Update RELEASENOTES.md

Update RELEASENOTES.md

Update RELEASENOTES.md

Update RELEASENOTES.md

Merge branch 'unstable' into development

use the term'state' rather than 'mode' when taling about the active state. Use 'while_active' rather than 'while active'

Merge branch 'unstable' into development

Quieten two compilation warnings from tinysvcmdns

Merge branch 'unstable' into development

Fix some errors and warnings from OpenBSD

Hook up the active mode to the disable standby facility, enabling a 'while active' setting for it. Expose them to D-Bus.

quieten a few debug messages

Add call to check if the dbus service is started.

make including of reference to dbus-service conditional on CONFIG_DBUS_INTERFACE

Continue hooking up active mode and disable standby stuff

fix a few minor compilation errors

Starting to connect active mode with disable standby.

check that the systems are started before trying to stop them, duh.

remove dependency on libdaemon -- leave it in by default, but allow the new configuration option --with-libdaemon.

remove dependency on libdaemon -- leave it in by default, but allow the new configuration option --with-libdaemon.

Quieten an incorrect compiler warning

Tidy code and slightly reorder settings listing.

Clean up audio help texts, add in the volume_range_hw_priority setting.

Stop a segmentation error if a bsackend has no help routine

make mdns-external-avahi a conditional compilation

Include the mdns_external interface even if avahi is not included.

Include some more settings in the debug listing.

allow configuration numbers to convert between floats and integers automatically.

Include the mbedtls version.h file so that conditional compilation can be done right

allow for updated or old mbedtls library

Fix warnings about deprecated mbedtls API calls

New volume code, new mute code, fix cancellation memnory leaks, defer alsa device initialisation until needed.

add a volume control mutex to serialise multiple attempts to change the volume, stop setting the fixed volume to max if using hybrid volume setup

add a volume control mutex to serialise multiple attempts to change the volume

add a volume control mutex to serialise multiple attempts to change the volume

lengthen some mutex timeouts to allow valgrind to work okay on a Pi 3B. Make operations that open the mixer uncancellable until it's closed again to stop a few small memory leaks when a player is cancelled.

Fix a situation where cancellation could lose a malloced buffer.

add a special mixer-only mutex, do make open/close mixer stuff safer and probably quicker, not waiting for the main alsa mutex.

Don't do a hardware mute when doing a flush -- if disable_standby is true, the mode goes to connected and then if player goes back to play immediately, you might hear some of the sound still moving through the output buffer. This happens due to a bug in IOS player.

Make mute work only if not diconnected. Clewan up some debug messages.

Rebuild the hardware mute code -- clear up the logic. Needs to check if disconnected though.

Make it work with deferred initialisation of the alsa device -- deferred to first use. Investigate hardware mute again.

Rebuild the volume control calculations. Implement software mute. Try to delay alsa initialisation as long as possible.

Remove the preflight call - don't seem to need it.

add hooks for active mode start and end and add in the timeout into the configuration file.

Add a few litte changes from thre transit branch.

bring up to date with unstable branch

gracefully handle situation when the output device is already in use; release it properly when finished.

Merge branch 'unstable' of https://github.com/mikebrady/shairport-sync into unstable

Merging two unstable branchses that have diverged

Fix a number of memory leaks and one #include error when using the OpenSSL library.

Remove some unnecessary code "protecting" some use of the OpenSSL library.

use the correct label -- CONFIG_OPENSSL -- rather than the incorrect CONFIG_SSL for including code to create a filename using the OpenSSL library.