audio: replaced copyAudioFormat() with simple assignment

The "!src" check in copyAudioFormat() used to hide bugs - one should never pass NULL to it. There is one caller which might pass NULL, add a check in this caller. Instead of doing mempcy(), we can simply assign the structures, which looks more natural.

audio: replaced copyAudioFormat() with simple assignment
7f1cccb3 · Max Kellermann · f9316fbb · 7f1cccb3 · 7f1cccb3 · 7f1cccb3
Commit 7f1cccb3 authored Sep 09, 2008 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 9 additions and 21 deletions

audio.c src/audio.c +5 -13

audio.h src/audio.h +0 -2

output_control.c src/output_control.c +3 -5

output_init.c src/output_init.c +1 -1

No files found.
--- a/src/audio.c
+++ b/src/audio.c
@@ -66,14 +66,6 @@ static unsigned int audio_output_count(void)
 	return nr;
 }

-void copyAudioFormat(struct audio_format *dest, const struct audio_format *src)
-{
-	if (!src)
-		return;
-
-	memcpy(dest, src, sizeof(*dest));
-}
-
 int cmpAudioFormat(const struct audio_format *f1, const struct audio_format *f2)
 {
 	if (f1 && f2 && (f1->sampleRate == f2->sampleRate) &&
@@ -130,10 +122,9 @@ void initAudioDriver(void)
 void getOutputAudioFormat(const struct audio_format *inAudioFormat,
 			  struct audio_format *outAudioFormat)
 {
-	if (audio_configFormat) {
-		copyAudioFormat(outAudioFormat, audio_configFormat);
-	} else
-		copyAudioFormat(outAudioFormat, inAudioFormat);
+	*outAudioFormat = audio_configFormat != NULL
+		? *audio_configFormat
+		: *inAudioFormat;
 }

 void initAudioConfig(void)
@@ -316,7 +307,8 @@ int openAudioDevice(const struct audio_format *audioFormat)

 	if (!audioOpened || !isCurrentAudioFormat(audioFormat)) {
 		flushAudioBuffer();
-		copyAudioFormat(&audio_format, audioFormat);
+		if (audioFormat != NULL)
+			audio_format = *audioFormat;
 		audioBufferSize = (audio_format.bits >> 3) *
 		    audio_format.channels;
 		audioBufferSize *= audio_format.sampleRate >> 5;

--- a/src/audio.h
+++ b/src/audio.h
@@ -27,8 +27,6 @@ struct audio_format;
 struct tag;
 struct client;

-void copyAudioFormat(struct audio_format *dest, const struct audio_format *src);
-
 int cmpAudioFormat(const struct audio_format *dest, const struct audio_format *src);

 void getOutputAudioFormat(const struct audio_format *inFormat,

--- a/src/output_control.c
+++ b/src/output_control.c
@@ -32,14 +32,12 @@ int audio_output_open(struct audio_output *audioOutput,
 		return 0;
 	}

-	copyAudioFormat(&audioOutput->inAudioFormat, audioFormat);
+	audioOutput->inAudioFormat = *audioFormat;

 	if (audioOutput->convertAudioFormat) {
-		copyAudioFormat(&audioOutput->outAudioFormat,
-		                &audioOutput->reqAudioFormat);
+		audioOutput->outAudioFormat = audioOutput->reqAudioFormat;
 	} else {
-		copyAudioFormat(&audioOutput->outAudioFormat,
-		                &audioOutput->inAudioFormat);
+		audioOutput->outAudioFormat = audioOutput->inAudioFormat;
 		if (audioOutput->open)
 			audio_output_close(audioOutput);
 	}

--- a/src/output_init.c
+++ b/src/output_init.c
@@ -103,7 +103,7 @@ int audio_output_init(struct audio_output *ao, ConfigParam * param)
 			FATAL("error parsing format at line %i\n", bp->line);
 		}

-		copyAudioFormat(&ao->outAudioFormat, &ao->reqAudioFormat);
+		ao->outAudioFormat = ao->reqAudioFormat;
 	}

 	if (plugin->init(ao, param) != 0)