Re: [PATCH 11/15] Add strbuf_vaddf(), use it in strbuf_addf(), and add strbuf_initf()
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Jun 29, 2008, 08:07 UTC
- Message-ID
- <7vskuwek9w.fsf@gitster.siamese.dyndns.org>
- In-Reply-To
- <alpine.DEB.1.00.0806281832510.9925@racer>
Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
> Heh. I suggested it, as I thought this was a perfect excuse to introduce > strbuf_initf(), which has been on my wishlist since long.
We may want to consider the change for its own merit, if there is a reason for the vaddf patch, such as "this is needed because platform X and Y do not have working vsnprintf()". Perhaps we add compat/* something.
> Anyway, I'm fine with having them only in my personal fork.
It will cost you some "trust point" next time you try to sneak something in as a part of a largely unrelated topic. Please don't.