diff options
author | Mariusz Kozlowski <m.kozlowski@tuxland.pl> | 2007-08-11 11:06:09 +0200 |
---|---|---|
committer | Jaroslav Kysela <perex@perex.cz> | 2007-10-16 15:58:48 +0200 |
commit | 51882453a729675aa7266aa87d99f33eca310c86 (patch) | |
tree | bf948854782092a0111058224bdfbec6cbcc36c6 /CREDITS | |
parent | 6ca2cdcc5e40286e6bf5286056b50a56c8ce327e (diff) |
[ALSA] This patch removes memset() from snd_emu10k1_fx8010_info() which apparently
isn't needed there. Upatched code uses:
memset(info, 0, sizeof(info));
where 'info' is a pointer and therefore only first 4 bytes of 'info' gets
cleared on a 32bit machine. Anyway looking at the code zeoring this memory
region isn't needed at all because the snd_emu10k1_fx8010_info() function
initializes all the 'info' fields on its own. So that's why this code works
at all in its original form.
This patch removes this redundant code. Also snd_emu10k1_fx8010_info() can't
fail so lets save some bytes and change its return type to void.
Signed-off-by: Mariusz Kozlowski <m.kozlowski@tuxland.pl>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Jaroslav Kysela <perex@suse.cz>
Diffstat (limited to 'CREDITS')
0 files changed, 0 insertions, 0 deletions