Re: [alsa-devel] [PATCH - AAF PCM plugin 3/7] aaf: Implement Playback mode support

On Wed, 2018-10-10 at 12:34 +0200, Takashi Iwai wrote:

...

+static int aaf_set_hw_constraint(snd_pcm_aaf_t *aaf) +{

• int res;
  

• snd_pcm_ioplug_t *io = &aaf->io;
  

• const unsigned int accesses[] = {
  

•         SND_PCM_ACCESS_RW_INTERLEAVED,
  

• };
  

• const unsigned int formats[] = {
  

•         SND_PCM_FORMAT_S16_BE,
  

•         SND_PCM_FORMAT_S24_3BE,
  

•         SND_PCM_FORMAT_S32_BE,
  

•         SND_PCM_FORMAT_FLOAT_BE,
  

• };
  

• const unsigned int rates[] = {
  

•         8000,
  

•         16000,
  

•         24000,
  

•         32000,
  

•         44100,
  

•         48000,
  

•         88200,
  

•         96000,
  

•         176400,
  

•         192000,
  

• };
  

Use static arrays for the above.

It isn't clear to me what is the point in allocating these variables in the global scope. May I ask what is the benefit in adding the 'static' modifier to these local variables?

Thanks,

Andre