MyFavMirrors/owntone-server
1
0
Fork 0
mirror of https://github.com/owntone/owntone-server.git synced 2025-03-31 01:33:44 -04:00
Code Issues Packages Projects Releases Wiki Activity

[input] Change input abstraction to use media_quality instead of input_quality

Browse Source
This commit is contained in:
ejurgensen 2019-02-08 19:03:09 +01:00
parent 462d787fab
commit a9085e978d
2 changed files with 17 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
src/input.c
View File

@ -41,7 +41,7 @@
// Disallow further writes to the buffer when its size is larger than this threshold // Disallow further writes to the buffer when its size is larger than this threshold
// TODO untie from 44100 // TODO untie from 44100
#define INPUT_BUFFER_THRESHOLD STOB(88200) #define INPUT_BUFFER_THRESHOLD STOB(88200, 16, 2)
// How long (in sec) to wait for player read before looping in playback thread // How long (in sec) to wait for player read before looping in playback thread
#define INPUT_LOOP_TIMEOUT 1 #define INPUT_LOOP_TIMEOUT 1
@ -68,7 +68,7 @@ static struct input_definition *inputs[] = {
struct marker struct marker
{ {
uint64_t pos; // Position of marker measured in bytes uint64_t pos; // Position of marker measured in bytes
struct input_quality quality; struct media_quality quality;
enum input_flags flags; enum input_flags flags;
// Reverse linked list, yay! // Reverse linked list, yay!
@ -89,8 +89,8 @@ struct input_buffer
input_cb full_cb; input_cb full_cb;
// Quality of write/read data // Quality of write/read data
struct input_quality cur_write_quality; struct media_quality cur_write_quality;
struct input_quality cur_read_quality; struct media_quality cur_read_quality;
size_t bytes_written; size_t bytes_written;
size_t bytes_read; size_t bytes_read;
@ -236,7 +236,7 @@ input_wait(void)
// Called by input modules from within the playback loop // Called by input modules from within the playback loop
int int
input_write(struct evbuffer *evbuf, struct input_quality *quality, short flags) input_write(struct evbuffer *evbuf, struct media_quality *quality, short flags)
{ {
struct timespec ts; struct timespec ts;
int ret; int ret;
@ -270,7 +270,7 @@ input_write(struct evbuffer *evbuf, struct input_quality *quality, short flags)
// Change of quality. Note, the marker is placed at the last position of the // Change of quality. Note, the marker is placed at the last position of the
// last byte we wrote, even though that of course doesn't have the new quality // last byte we wrote, even though that of course doesn't have the new quality
// yet. Not intuitive, but input_read() will understand. // yet. Not intuitive, but input_read() will understand.
if (quality && memcmp(quality, &input_buffer.cur_write_quality, sizeof(struct input_quality)) != 0) if (quality && !quality_is_equal(quality, &input_buffer.cur_write_quality))
{ {
input_buffer.cur_write_quality = *quality; input_buffer.cur_write_quality = *quality;
marker_add(INPUT_FLAG_QUALITY); marker_add(INPUT_FLAG_QUALITY);
@ -346,17 +346,20 @@ input_read(void *data, size_t size, short *flags)
#ifdef DEBUG #ifdef DEBUG
// Logs if flags present or each 10 seconds // Logs if flags present or each 10 seconds
size_t one_sec_size = STOB(input_buffer.cur_read_quality.sample_rate, input_buffer.cur_read_quality.bits_per_sample, input_buffer.cur_read_quality.channels);
debug_elapsed += len; debug_elapsed += len;
if (*flags || (debug_elapsed / STOB(input_buffer.cur_read_quality.sample_rate) > 10)) if (*flags || (debug_elapsed > 10 * one_sec_size))
{ {
debug_elapsed = 0; debug_elapsed = 0;
DPRINTF(E_SPAM, L_PLAYER, "READ %zu bytes (%d/%d), WROTE %zu bytes (%d/%d), SIZE %zu (=%zu), FLAGS %04x\n", DPRINTF(E_SPAM, L_PLAYER, "READ %zu bytes (%d/%d/%d), WROTE %zu bytes (%d/%d/%d), SIZE %zu (=%zu), FLAGS %04x\n",
input_buffer.bytes_read, input_buffer.bytes_read,
input_buffer.cur_read_quality.sample_rate, input_buffer.cur_read_quality.sample_rate,
input_buffer.cur_read_quality.bits_per_sample, input_buffer.cur_read_quality.bits_per_sample,
input_buffer.cur_read_quality.channels,
input_buffer.bytes_written, input_buffer.bytes_written,
input_buffer.cur_write_quality.sample_rate, input_buffer.cur_write_quality.sample_rate,
input_buffer.cur_write_quality.bits_per_sample, input_buffer.cur_write_quality.bits_per_sample,
input_buffer.cur_write_quality.channels,
evbuffer_get_length(input_buffer.evbuf), evbuffer_get_length(input_buffer.evbuf),
input_buffer.bytes_written - input_buffer.bytes_read, input_buffer.bytes_written - input_buffer.bytes_read,
*flags); *flags);
@ -518,8 +521,8 @@ input_flush(short *flags)
evbuffer_drain(input_buffer.evbuf, len); evbuffer_drain(input_buffer.evbuf, len);
memset(&input_buffer.cur_read_quality, 0, sizeof(struct input_quality)); memset(&input_buffer.cur_read_quality, 0, sizeof(struct media_quality));
memset(&input_buffer.cur_write_quality, 0, sizeof(struct input_quality)); memset(&input_buffer.cur_write_quality, 0, sizeof(struct media_quality));
input_buffer.bytes_read = 0; input_buffer.bytes_read = 0;
input_buffer.bytes_written = 0; input_buffer.bytes_written = 0;
@ -534,7 +537,7 @@ input_flush(short *flags)
} }
int int
input_quality_get(struct input_quality *quality) input_quality_get(struct media_quality *quality)
{ {
// No mutex, other threads should not be able to affect cur_read_quality // No mutex, other threads should not be able to affect cur_read_quality
*quality = input_buffer.cur_read_quality; *quality = input_buffer.cur_read_quality;

12
src/input.h
View File

@ -6,6 +6,7 @@
# include <config.h> # include <config.h>
#endif #endif
#include <event2/buffer.h> #include <event2/buffer.h>
#include "misc.h"
#include "transcode.h" #include "transcode.h"
// Must be in sync with inputs[] in input.c // Must be in sync with inputs[] in input.c
@ -83,13 +84,6 @@ struct player_source
typedef int (*input_cb)(void); typedef int (*input_cb)(void);
struct input_quality
{
int sample_rate;
int bits_per_sample;
// Maybe some day also add channels here
};
struct input_metadata struct input_metadata
{ {
uint32_t item_id; uint32_t item_id;
@ -161,7 +155,7 @@ int input_loop_break;
* -1 on error * -1 on error
*/ */
int int
input_write(struct evbuffer *evbuf, struct input_quality *quality, short flags); input_write(struct evbuffer *evbuf, struct media_quality *quality, short flags);
/* /*
* Input modules can use this to wait in the playback loop (like input_write() * Input modules can use this to wait in the playback loop (like input_write()
@ -236,7 +230,7 @@ input_flush(short *flags);
* Returns the current quality of data returned by intput_read(). * Returns the current quality of data returned by intput_read().
*/ */
int int
input_quality_get(struct input_quality *quality); input_quality_get(struct media_quality *quality);
/* /*
* Gets metadata from the input, returns 0 if metadata is set, otherwise -1 * Gets metadata from the input, returns 0 if metadata is set, otherwise -1