C: Почему глобальные переменные в моей библиотеке не обновляются? (Использование ALSA Lib) - PullRequest
0 голосов
/ 23 мая 2019

Предисловие: Я посмотрел другие ответы и считаю, что это другой тип вопроса.Возможно, это связано с указателем, но ни один из других ответов не помог.Я также, возможно, просто что-то упускаю.

Мне удалось создать работающую программу, которая транслирует необработанные аудиоданные на стандартный вывод, используя библиотеку ALSA.Сейчас я пытаюсь обобщить этот код в библиотеку, и глобальные переменные в файле .c библиотеки не обновляются.Насколько я могу судить, установка ничем не отличается от исходной программы, поэтому я думаю, что она должна работать ... но это не так.Я был бы очень признателен, если бы кто-то объяснил, как именно я все испортил.Благодарю вас в первую очередь!

EDIT 1:

Все глобальные переменные в верхней части wm8782.c не могут быть установлены при вызове wm8782_open_audio().Например, когда snd_pcm_hw_params_get_period_time(params, &val, &dir); вызывается в wm8782_us_to_loops() после вызова wm8782_open_audio(), происходит сбой, поскольку params не был инициализирован.Тем не менее, wm8782_open_audio() завершается без ошибок!

РЕДАКТИРОВАТЬ 2:

Я добавил extern объявлений в заголовок, без кубиков.Смотрите обновленные файлы ниже.

РЕДАКТИРОВАТЬ 3:

Я воспроизвел проблему в одном файле C без заголовка.Просто разбить код на функции достаточно, чтобы разбить его.Я заменил 2 библиотечных файла этим монолитным C-файлом, чтобы исключить ошибки компоновщика и т. П.

Вот исходная программа, которую я пытаюсь обобщить в библиотеку.Это работает так, как я хочу.

#define ALSA_PCM_NEW_HW_PARAMS_API
#include <alsa/asoundlib.h>

long loops;
int rc;
int size;
snd_pcm_t *handle;
snd_pcm_hw_params_t *params;
unsigned int val;
int dir;
snd_pcm_uframes_t frames;
char *buffer;

int main()
{
    /* Open PCM device for recording (capture). */
    rc = snd_pcm_open(&handle, "hw:1",
                      SND_PCM_STREAM_CAPTURE, 0);
    if (rc < 0)
    {
        fprintf(stderr,
                "unable to open pcm device: %s\n",
                snd_strerror(rc));
        exit(1);
    }

    /* Allocate a hardware parameters object. */
    snd_pcm_hw_params_alloca(&params);

    /* Fill it in with default values. */
    snd_pcm_hw_params_any(handle, params);

    /* Set the desired hardware parameters. */

    /* Interleaved mode */
    snd_pcm_hw_params_set_access(handle, params,
                                 SND_PCM_ACCESS_RW_INTERLEAVED);

    /* Signed 32-bit little-endian format */
    snd_pcm_hw_params_set_format(handle, params,
                                 SND_PCM_FORMAT_S32_LE);

    /* Two channels (stereo) */
    snd_pcm_hw_params_set_channels(handle, params, 2);

    /* 44100 bits/second sampling rate (CD quality) */
    val = 44100;
    snd_pcm_hw_params_set_rate_near(handle, params,
                                    &val, &dir);

    /* Set period size to 64 frames. */
    frames = 64;
    snd_pcm_hw_params_set_period_size_near(handle,
                                           params, &frames, &dir);

    /* Write the parameters to the driver */
    rc = snd_pcm_hw_params(handle, params);
    if (rc < 0)
    {
        fprintf(stderr,
                "unable to set hw parameters: %s\n",
                snd_strerror(rc));
        exit(1);
    }

    /* Use a buffer large enough to hold one period */
    snd_pcm_hw_params_get_period_size(params,
                                      &frames, &dir);
    size = frames * 8; /* 4 bytes/sample, 2 channels */
    buffer = (char *) malloc(size);

    /* We want to loop for 5 seconds */
    snd_pcm_hw_params_get_period_time(params,
                                      &val, &dir);
    //fprintf(stderr, "%u\n", val);
    loops = 5000000 / val;

    while (loops > 0)
    {
        loops--;
        rc = snd_pcm_readi(handle, buffer, frames);
        if (rc == -EPIPE)
        {
            /* EPIPE means overrun */
            fprintf(stderr, "overrun occurred\n");
            snd_pcm_prepare(handle);
        }
        else if (rc < 0)
        {
            fprintf(stderr,
                    "error from read: %s\n",
                    snd_strerror(rc));
        }
        else if (rc != (int)frames)
        {
            fprintf(stderr, "short read, read %d frames\n", rc);
        }
        rc = write(1, buffer, size);
        if (rc != size)
            fprintf(stderr,
                    "short write: wrote %d bytes\n", rc);
    }

    snd_pcm_drain(handle);
    snd_pcm_close(handle);
    free(buffer);

    return 0;
}

Теперь вот модульная версия, которая в конечном итоге станет библиотекой.Это не с ERROR: Failed to get number of loops.:

#define ALSA_PCM_NEW_HW_PARAMS_API

#include <stdbool.h>
#include <stdint.h>
#include <stdio.h>
#include <alsa/asoundlib.h>

char *wm8782_buffer;
int wm8782_buffer_size;

snd_pcm_t *wm8782_handle;
snd_pcm_hw_params_t *wm8782_params;
int wm8782_dir;
snd_pcm_uframes_t wm8782_frames;
unsigned int wm8782_val;
int wm8782_rc;

typedef enum
{
    WM8782_READ_OVERRUN,
    WM8782_READ_ERROR,
    WM8782_READ_SHORT,
    WM8782_READ_GOOD,
} wm8782read_t;

typedef enum
{
    WM8782_RATE_8K = 8000,
    WM8782_RATE_16K = 16000,
    WM8782_RATE_32K = 32000,
    WM8782_RATE_44_1K = 44100,
    WM8782_RATE_48K = 48000,
    WM8782_RATE_96K = 96000,
    WM8782_RATE_192K = 192000,
} wm8782rate_t;

bool wm8782_open_audio(wm8782rate_t rate_in, snd_pcm_uframes_t wm8782_frames_in)
{
    // Open PCM device for recording (capture).
    wm8782_rc = snd_pcm_open(&wm8782_handle, "hw:1",
                      SND_PCM_STREAM_CAPTURE, 0);
    if (wm8782_rc < 0)
    {
        fprintf(stderr,
                "ERROR: Unable to open pcm device: %s\n",
                snd_strerror(wm8782_rc));
        return false;
    }

    // Allocate a hardware parameters object.
    snd_pcm_hw_params_alloca(&wm8782_params);

    // Fill it in with default values
    snd_pcm_hw_params_any(wm8782_handle, wm8782_params);

    // Set the desired hardware parameters. */

    // Interleaved mode
    snd_pcm_hw_params_set_access(wm8782_handle, wm8782_params, SND_PCM_ACCESS_RW_INTERLEAVED);

    // Signed 32-bit little-endian format
    snd_pcm_hw_params_set_format(wm8782_handle, wm8782_params, SND_PCM_FORMAT_S32_LE);

    // Two channels (stereo)
    snd_pcm_hw_params_set_channels(wm8782_handle, wm8782_params, 2);

    // Set rate
    wm8782_val = rate_in;
    snd_pcm_hw_params_set_rate_near(wm8782_handle, wm8782_params, &wm8782_val, &wm8782_dir);

    // Set period size.
    wm8782_frames = wm8782_frames_in;
    snd_pcm_hw_params_set_period_size_near(wm8782_handle, wm8782_params, &wm8782_frames, &wm8782_dir);

    // Write the parameters to the driver
    wm8782_rc = snd_pcm_hw_params(wm8782_handle, wm8782_params);
    if (wm8782_rc < 0)
    {
        fprintf(stderr,
                "ERROR: Unable to set hw parameters: %s\n",
                snd_strerror(wm8782_rc));
        return false;
    }

    // Use a buffer large enough to hold one period
    snd_pcm_hw_params_get_period_size(wm8782_params, &wm8782_frames, &wm8782_dir);
    wm8782_buffer_size = wm8782_frames * 8; // 4 bytes/sample, 2 channels
    wm8782_buffer = (char *) malloc(wm8782_buffer_size);

    return true;
}

void wm8782_close_audio()
{
    snd_pcm_drain(wm8782_handle);
    snd_pcm_close(wm8782_handle);
    free(wm8782_buffer);
}

wm8782read_t wm8782_read_audio()
{
    wm8782_rc = snd_pcm_readi(wm8782_handle, wm8782_buffer, wm8782_frames);
    if (wm8782_rc == -EPIPE)
    {
        // EPIPE means overrun
        fprintf(stderr, "ERROR: Overrun occurred\n");
        snd_pcm_prepare(wm8782_handle);
        return WM8782_READ_OVERRUN;
    }
    else if (wm8782_rc < 0)
    {
        fprintf(stderr, "ERROR: From read: %s\n", snd_strerror(wm8782_rc));
        return WM8782_READ_ERROR;
    }
    else if (wm8782_rc != (int)wm8782_frames)
    {
        fprintf(stderr, "ERROR: Short read, read %d wm8782_frames\n", wm8782_rc);
        return WM8782_READ_SHORT;
    }

    return WM8782_READ_GOOD;
}

unsigned int wm8782_us_to_loops(unsigned int us_in)
{
    if(snd_pcm_hw_params_get_period_time(wm8782_params, &wm8782_val, &wm8782_dir) < 0)
    {
        fprintf(stderr, "ERROR: Failed to get number of loops.\n");
        return 0;
    }
    //fprintf(stderr, "%u\n", wm8782_val);
    return (us_in / wm8782_val);
}

int main()
{
    if(!wm8782_open_audio(44100, 64))
    {
        fprintf(stderr, "unable to open pcm device");
    }

    unsigned int loops =  wm8782_us_to_loops(5000000);

    while (loops > 0)
    {
        loops--;
        wm8782_read_audio();

        wm8782_rc = write(1, wm8782_buffer, wm8782_buffer_size);
        if (wm8782_rc != wm8782_buffer_size)
            fprintf(stderr,
                    "short write: wrote %d bytes\n", wm8782_rc);
    }

    wm8782_close_audio();

    return 0;
}

1 Ответ

1 голос
/ 23 мая 2019

Хорошо, так что оказывается, что это вообще не проблема глобалов ... извините всех! Это какой-то странный неверный аргумент, который передается snd_pcm_hw_params_get_period_size(), в частности. Код фактически работает с определенным вручную числом циклов, поэтому я закрываю этот вопрос.

...