Re: [alsa-devel] [PATCH - AAF PCM plugin 1/7] aaf: Introduce plugin skeleton

On Thu, 04 Oct 2018 01:45:41 +0200, Andre Guedes wrote:

+static int aaf_close(snd_pcm_ioplug_t *io) +{

• snd_pcm_aaf_t *aaf = io->private_data;
• if (!aaf)

• return -EBADFD;
  

• free(aaf);
• aaf = NULL;

This makes no sense. The function can be simplified to

static int aaf_close(snd_pcm_ioplug_t *io) { free(io->private_data); return 0; }

thanks,

Takashi