[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 27/52] audio: use size_t where makes sense
From: |
Pavel Dovgalyuk |
Subject: |
Re: [Qemu-devel] [PATCH v2 27/52] audio: use size_t where makes sense |
Date: |
Mon, 24 Dec 2018 09:19:44 +0300 |
> From: Kővágó, Zoltán [mailto:address@hidden
> diff --git a/replay/replay-audio.c b/replay/replay-audio.c
> index b113836de4..efe1628727 100644
> --- a/replay/replay-audio.c
> +++ b/replay/replay-audio.c
> @@ -16,18 +16,18 @@
> #include "sysemu/sysemu.h"
> #include "audio/audio.h"
>
> -void replay_audio_out(int *played)
> +void replay_audio_out(size_t *played)
> {
> if (replay_mode == REPLAY_MODE_RECORD) {
> g_assert(replay_mutex_locked());
> replay_save_instructions();
> replay_put_event(EVENT_AUDIO_OUT);
> - replay_put_dword(*played);
> + replay_put_qword(*played);
> } else if (replay_mode == REPLAY_MODE_PLAY) {
> g_assert(replay_mutex_locked());
> replay_account_executed_instructions();
> if (replay_next_event_is(EVENT_AUDIO_OUT)) {
> - *played = replay_get_dword();
> + *played = replay_get_qword();
> replay_finish_event();
> } else {
> error_report("Missing audio out event in the replay log");
> @@ -36,7 +36,7 @@ void replay_audio_out(int *played)
> }
> }
>
> -void replay_audio_in(int *recorded, void *samples, int *wpos, int size)
> +void replay_audio_in(size_t *recorded, void *samples, size_t *wpos, size_t
> size)
> {
> int pos;
> uint64_t left, right;
> @@ -44,8 +44,8 @@ void replay_audio_in(int *recorded, void *samples, int
> *wpos, int size)
> g_assert(replay_mutex_locked());
> replay_save_instructions();
> replay_put_event(EVENT_AUDIO_IN);
> - replay_put_dword(*recorded);
> - replay_put_dword(*wpos);
> + replay_put_qword(*recorded);
> + replay_put_qword(*wpos);
> for (pos = (*wpos - *recorded + size) % size ; pos != *wpos
> ; pos = (pos + 1) % size) {
> audio_sample_to_uint64(samples, pos, &left, &right);
> @@ -56,8 +56,8 @@ void replay_audio_in(int *recorded, void *samples, int
> *wpos, int size)
> g_assert(replay_mutex_locked());
> replay_account_executed_instructions();
> if (replay_next_event_is(EVENT_AUDIO_IN)) {
> - *recorded = replay_get_dword();
> - *wpos = replay_get_dword();
> + *recorded = replay_get_qword();
> + *wpos = replay_get_qword();
> for (pos = (*wpos - *recorded + size) % size ; pos != *wpos
> ; pos = (pos + 1) % size) {
> left = replay_get_qword();
Please also increment REPLAY_VERSION in replay.c
It is required because log format is changed and newer version should not
be able reading old logs.
Pavel Dovgalyuk
- [Qemu-devel] [PATCH v2 21/52] paaudio: do not move stream when sink/source name is specified, (continued)
- [Qemu-devel] [PATCH v2 21/52] paaudio: do not move stream when sink/source name is specified, Kővágó, Zoltán, 2018/12/23
- [Qemu-devel] [PATCH v2 22/52] paaudio: properly disconnect streams in fini_*, Kővágó, Zoltán, 2018/12/23
- [Qemu-devel] [PATCH v2 20/52] audio: audiodev= parameters no longer optional when -audiodev present, Kővágó, Zoltán, 2018/12/23
- [Qemu-devel] [PATCH v2 18/52] audio: basic support for multi backend audio, Kővágó, Zoltán, 2018/12/23
- [Qemu-devel] [PATCH v2 24/52] audio: do not run each backend in audio_run, Kővágó, Zoltán, 2018/12/23
- [Qemu-devel] [PATCH v2 19/52] audio: add audiodev properties to frontends, Kővágó, Zoltán, 2018/12/23
- [Qemu-devel] [PATCH v2 25/52] paaudio: fix playback glitches, Kővágó, Zoltán, 2018/12/23
- [Qemu-devel] [PATCH v2 26/52] audio: remove read and write pcm_ops, Kővágó, Zoltán, 2018/12/23
- [Qemu-devel] [PATCH v2 28/52] audio: api for mixeng code free backends, Kővágó, Zoltán, 2018/12/23
- [Qemu-devel] [PATCH v2 27/52] audio: use size_t where makes sense, Kővágó, Zoltán, 2018/12/23
- [Qemu-devel] [PATCH v2 23/52] audio: remove audio_MIN, audio_MAX, Kővágó, Zoltán, 2018/12/23
- Re: [Qemu-devel] [PATCH v2 23/52] audio: remove audio_MIN, audio_MAX, Philippe Mathieu-Daudé, 2018/12/23
- Re: [Qemu-devel] [PATCH v2 23/52] audio: remove audio_MIN, audio_MAX, Zoltán Kővágó, 2018/12/23
- Re: [Qemu-devel] [PATCH v2 23/52] audio: remove audio_MIN, audio_MAX, Philippe Mathieu-Daudé, 2018/12/24
- Re: [Qemu-devel] [PATCH v2 23/52] audio: remove audio_MIN, audio_MAX, Kővágó Zoltán, 2018/12/24
- Re: [Qemu-devel] [PATCH v2 23/52] audio: remove audio_MIN, audio_MAX, Philippe Mathieu-Daudé, 2018/12/25
- Re: [Qemu-devel] [PATCH v2 23/52] audio: remove audio_MIN, audio_MAX, Kővágó Zoltán, 2018/12/27
[Qemu-devel] [PATCH v2 36/52] spiceaudio: port to the new audio backend api, Kővágó, Zoltán, 2018/12/23
[Qemu-devel] [PATCH v2 32/52] noaudio: port to the new audio backend api, Kővágó, Zoltán, 2018/12/23