From mboxrd@z Thu Jan 1 00:00:00 1970 From: Keir Fraser Subject: Re: mini-os: C programming Date: Thu, 15 Mar 2007 13:54:06 +0000 Message-ID: References: <45F94C89.4010603@cea.fr> Mime-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <45F94C89.4010603@cea.fr> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: PUCCETTI Armand , xen-devel@lists.xensource.com List-Id: xen-devel@lists.xenproject.org On 15/3/07 13:39, "PUCCETTI Armand" wrote: > extern char *stack; Yes, this one is bogus. > 2. In file xen-3.0.3/extras/mini-os/gnttab.c:140: the const variable > gnttabop_error_msgs > is declared as > > static const char *gnttabop_error_msgs[] = GNTTABOP_error_msgs; > > shouldn't that instead be declared: > > static const char * const gnttabop_error_msgs[] = GNTTABOP_error_msgs; It doesn't really matter, does it? Personally I hate scattering 'const' all over the place. -- Keir