From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from de01egw01.freescale.net (de01egw01.freescale.net [192.88.165.102]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client CN "de01egw01.freescale.net", Issuer "Thawte Premium Server CA" (verified OK)) by ozlabs.org (Postfix) with ESMTP id 48452DDE17 for ; Mon, 7 Jan 2008 09:46:40 +1100 (EST) Date: Sun, 6 Jan 2008 16:46:24 -0600 From: Scott Wood To: jdl@jdl.com, linuxppc-dev@ozlabs.org Subject: Re: [PATCH 2/3] Handle absolute pathnames correctly in dtc_open_file. Message-ID: <20080106224623.GA8239@ld0162-tx32.am.freescale.net> References: <20080104211045.GB28991@ld0162-tx32.am.freescale.net> <20080106033716.GL4326@localhost.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20080106033716.GL4326@localhost.localdomain> List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Sun, Jan 06, 2008 at 02:37:16PM +1100, David Gibson wrote: > > out: > > + free((void *)file->dir); > > That cast shouldn't be there. It generates a constness warning otherwise. I was on the fence over whether to just remove the constness (even though the data is never modified during its lifetime); would that be preferable? -Scott