From: Miklos Vajna Date: Fri, 26 Feb 2010 00:50:20 GMT Subject: Re: [PATCH v2 2/3] git-core: Support retrieving passwords with GIT_ASKPASS Message-ID: <20100226005020.GR12429@genesis.frugalware.org> In-Reply-To: <1267143154-5020-1-git-send-email-lznuaa@gmail.com> On Fri, Feb 26, 2010 at 08:12:34AM +0800, Frank Li wrote: > + for (i = 0; i < buffer.len; i++) > + if (buffer.buf[i] == '\n' || buffer.buf[i] == '\r') { > + buffer.buf[i] = '\0'; > + buffer.len = i; > + } > + return strbuf_detach(&buffer, NULL); > + > + } else { > + return getpass(prompt); > + } According to Documentation/CodingGuidelines, this would be: for (i = 0; i < buffer.len; i++) if (buffer.buf[i] == '\n' || buffer.buf[i] == '\r') { buffer.buf[i] = '\0'; buffer.len = i; } return strbuf_detach(&buffer, NULL); } else return getpass(prompt);