jack: don't set audioOutput->data=NULL

There is only one caller of freeJackData() left: jack_finishDriver(). This function is called by the mpd core, and is called exactly once for every successful jack_initDriver(). We do not need to clear audioOutput->data, since this variable is invalidated anyway.

jack: don't set audioOutput->data=NULL
edd7e2f9 · Max Kellermann · 806a9f02 · edd7e2f9
Commit edd7e2f9 authored Aug 26, 2008 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 5 deletions

audioOutput_jack.c src/audioOutputs/audioOutput_jack.c +5 -5

No files found.
--- a/src/audioOutputs/audioOutput_jack.c
+++ b/src/audioOutputs/audioOutput_jack.c
@@ -87,11 +87,11 @@ static void freeJackClient(JackData *jd)
 static void freeJackData(AudioOutput *audioOutput)
 {
 	JackData *jd = audioOutput->data;
-	if (jd) {
-		freeJackClient(jd);
-		free(jd);
-		audioOutput->data = NULL;
-	}
+
+	assert(jd != NULL);
+
+	freeJackClient(jd);
+	free(jd);
 }

 static void jack_finishDriver(AudioOutput *audioOutput)