From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joe Perches Subject: Re: [PATCH 24/25] sound: Use static const char arrays Date: Mon, 13 Sep 2010 13:47:45 -0700 Message-ID: <1284410865.26719.88.camel@Joe-Laptop> References: Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Takashi Iwai Cc: linux-kernel@vger.kernel.org, Jaroslav Kysela , alsa-devel List-Id: alsa-devel@alsa-project.org On Mon, 2010-09-13 at 22:30 +0200, Takashi Iwai wrote: > At Mon, 13 Sep 2010 12:48:02 -0700, > Joe Perches wrote: > > diff --git a/sound/core/misc.c b/sound/core/misc.c > > - char tmp[] = "<0>"; > > + char tmp[sizeof("<0>")]; > Would this case save something really...? Not really. It's the same x86 code size with different opcodes. It's just a stupid checkpatch future warning avoidance change. Ignore it at your pleasure. cheers, Joe