On 08/14/2012 04:08 PM, Tanu Kaskinen wrote:
Also add a cast to void pointer to get rid of one instance of "warning: cast increases required alignment of target type". The warning is a false positive. --- src/pulsecore/resampler.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)diff --git a/src/pulsecore/resampler.c b/src/pulsecore/resampler.c index 9f19559..9d50093 100644 --- a/src/pulsecore/resampler.c +++ b/src/pulsecore/resampler.c @@ -1729,7 +1729,7 @@ static void ffmpeg_resample(pa_resampler *r, const pa_memchunk *input, unsigned previous_consumed_frames = consumed_frames; /* And place the results in the output buffer */ - s = (short*) ((uint8_t*) pa_memblock_acquire(output->memblock) + output->index) + c; + s = (int16_t *) ((void *) ((uint8_t *) pa_memblock_acquire(output->memblock) + output->index)) + c;
Maybe clearer if you use a temporary variable here, i e split to two lines? Otherwise acked. -- David Henningsson, Canonical Ltd. https://launchpad.net/~diwic _______________________________________________ pulseaudio-discuss mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss
