Date: Sat, 6 Dec 2008 09:27:26 -0800 (PST)
From: Linus Torvalds <torvalds@linux-foundation.org>
To: Nick Andrew <nick@nick-andrew.net>
cc: Andrew Morton <akpm@linux-foundation.org>, linux-kernel@vger.kernel.org
Subject: Re: [RFC PATCH v1 2/3] Add %v support to vsnprintf()
In-Reply-To: <20081206070003.29149.50293.stgit@marcab.local.tull.net>
Message-ID: <alpine.LFD.2.00.0812060917220.3425@nehalem.linux-foundation.org>
References: <20081206065922.29149.63380.stgit@marcab.local.tull.net> <20081206070003.29149.50293.stgit@marcab.local.tull.net>
User-Agent: Alpine 2.00 (LFD 1167 2008-08-23)
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1028
Lines: 27


On Sat, 6 Dec 2008, Nick Andrew wrote:
>
> Add %v support to vsnprintf()

No can do. gcc will always warn, and I don't want to see an extension that 
will generate unfixable warnings.

If you want to do this, you'd need to do it the same way we did some othe 
printf extensions, ie by not passing the valist entry itself, but 
passing the _pointer_ to it, and using "%pV" as the format modifier.

(Quite frankly, "va_list" can be rather large, so passing a pointer to it 
is probably a better idea anyway).

Of course, gcc may implement "va_list" as an array (or just a pointer) 
internally, and then it will be passed as a pointer anyway (and you'd not 
need the address-of), but that's certainly not in any way guaranteed, and 
is very possibly architecture-dependent.

			Linus
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/