Re: [PATCH] use strbuf_addstr() instead of strbuf_addf() with "%s"

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: Re: [PATCH] use strbuf_addstr() instead of strbuf_addf() with "%s"
From: Johannes Schindelin <Johannes.Schindelin@xxxxxx>
Date: Sun, 7 Aug 2016 10:49:27 +0200 (CEST)
Cc: Git List <git@xxxxxxxxxxxxxxx>, Junio C Hamano <gitster@xxxxxxxxx>
In-reply-to: <57A4F8F7.5020109@web.de>
References: <57A4F8F7.5020109@web.de>
User-agent: Alpine 2.20 (DEB 67 2015-01-07)

Hi René,

On Fri, 5 Aug 2016, René Scharfe wrote:

> Call strbuf_addstr() for adding a simple string to a strbuf instead of
> using the heavier strbuf_addf().  This is shorter and documents the
> intent more clearly.

Looks obviously good, thanks!

Ciao,
Dscho

References:
- [PATCH] use strbuf_addstr() instead of strbuf_addf() with "%s"
  - From: René Scharfe

Prev by Date: Re: patch submission process, was Re: [PATCH v6 06/16] merge_recursive: abort properly upon errors
Next by Date: Re: [PATCH] merge-recursive: use STRING_LIST_INIT_NODUP
Previous by thread: [PATCH] use strbuf_addstr() instead of strbuf_addf() with "%s"
Next by thread: [PATCH] use CHILD_PROCESS_INIT to initialize automatic variables
Index(es):
- Date
- Thread

[Index of Archives] [Linux Kernel Development] [Gcc Help] [IETF Annouce] [DCCP] [Netdev] [Networking] [Security] [V4L] [Bugtraq] [Yosemite] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Linux SCSI] [Fedora Users]