[PATCH] imaadp32.acm: Avoid ARRAY_SIZE-like macros

Michael Stefaniuc mstefani at winehq.org
Mon Jul 9 13:44:39 CDT 2018


On 07/09/2018 03:08 PM, Andrew Eikum wrote:
> On Thu, Jul 05, 2018 at 10:07:22PM +0200, Michael Stefaniuc wrote:
>> @@ -93,11 +90,11 @@ static	DWORD	ADPCM_GetFormatIndex(const WAVEFORMATEX *wfx)
>>      switch (wfx->wFormatTag)
>>      {
>>      case WAVE_FORMAT_PCM:
>> -	hi = NUM_PCM_FORMATS;
>> +        hi = ARRAY_SIZE(PCM_Formats);
>>  	fmts = PCM_Formats;
>>  	break;
>>      case WAVE_FORMAT_IMA_ADPCM:
>> -	hi = NUM_ADPCM_FORMATS;
>> +        hi = ARRAY_SIZE(ADPCM_Formats);
>>  	fmts = ADPCM_Formats;
>>  	break;
>>      default:
> 
> I'd prefer to keep this as tabs, to stay (more) consistent.
I have changed it because it is already inconsistent. The "case" uses
spaces for indentation and that cannot be changed to tabs.

bye
	michael




More information about the wine-devel mailing list