<feed xmlns='http://www.w3.org/2005/Atom'>
<title>git/Documentation/CodingGuidelines, branch v2.2.1</title>
<subtitle>Mirror of https://git.kernel.org/pub/scm/git/git.git/
</subtitle>
<id>https://git.shady.money/git/atom?h=v2.2.1</id>
<link rel='self' href='https://git.shady.money/git/atom?h=v2.2.1'/>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/'/>
<updated>2014-07-16T18:33:03Z</updated>
<entry>
<title>Merge branch 'po/error-message-style'</title>
<updated>2014-07-16T18:33:03Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2014-07-16T18:33:03Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=7591e2c53c6f02626e376a98b5f95745fbfe108f'/>
<id>urn:sha1:7591e2c53c6f02626e376a98b5f95745fbfe108f</id>
<content type='text'>
* po/error-message-style:
  doc: give some guidelines for error messages
</content>
</entry>
<entry>
<title>doc: give some guidelines for error messages</title>
<updated>2014-07-10T20:31:55Z</updated>
<author>
<name>Philip Oakley</name>
<email>philipoakley@iee.org</email>
</author>
<published>2014-06-16T12:55:57Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=0ae0e882b2dd8048e2096138561f0b2ab0cad454'/>
<id>urn:sha1:0ae0e882b2dd8048e2096138561f0b2ab0cad454</id>
<content type='text'>
Clarify error message puntuation to reduce review workload.

Signed-off-by: Philip Oakley &lt;philipoakley@iee.org&gt;
Helped-by: Jeff King &lt;peff@peff.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>CodingGuidelines: avoid "test &lt;cond&gt; -a/-o &lt;cond&gt;"</title>
<updated>2014-05-20T18:19:43Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2014-05-20T18:12:02Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=897f964c0dce8e7cc2cc53bb19b83cadce106773'/>
<id>urn:sha1:897f964c0dce8e7cc2cc53bb19b83cadce106773</id>
<content type='text'>
The construct is error-prone; "test" being built-in in most modern
shells, the reason to avoid "test &lt;cond&gt; &amp;&amp; test &lt;cond&gt;" spawning
one extra process by using a single "test &lt;cond&gt; -a &lt;cond&gt;" no
longer exists.

Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>CodingGuidelines: on splitting a long line</title>
<updated>2014-05-02T21:08:16Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2014-05-02T20:42:39Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=f26443da0449c459dab8b91d963bd91dd4335657'/>
<id>urn:sha1:f26443da0449c459dab8b91d963bd91dd4335657</id>
<content type='text'>
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>CodingGuidelines: on comparison</title>
<updated>2014-05-02T20:44:46Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2014-04-30T21:26:23Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=5db9ab82b94fab16e69b0228aaf1e972520bd04a'/>
<id>urn:sha1:5db9ab82b94fab16e69b0228aaf1e972520bd04a</id>
<content type='text'>
There are arguments for writing a conditional as "a &lt; b" rather than
"b &gt; a", or vice versa.  Let's give guidance on which we prefer.

See http://thread.gmane.org/gmane.comp.version-control.git/3903/focus=4126
for the original discussion.

Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>CodingGuidelines: do not call the conditional statement "if()"</title>
<updated>2014-05-02T20:26:07Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2014-04-30T21:25:47Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=691d0dd0a9c901286c2a0a28c30ec4d13bcd2032'/>
<id>urn:sha1:691d0dd0a9c901286c2a0a28c30ec4d13bcd2032</id>
<content type='text'>
The point immediately before it is about having SP after the control
keyword.  Spell it out as 'an "if" statement' instead.

Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>CodingGuidelines: give an example for shell function preamble</title>
<updated>2014-05-02T20:24:57Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2014-04-30T21:25:11Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=6117a3d4946e35c7d4a38c0b443891c81808a838'/>
<id>urn:sha1:6117a3d4946e35c7d4a38c0b443891c81808a838</id>
<content type='text'>
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>CodingGuidelines: give an example for control statements</title>
<updated>2014-05-02T20:24:57Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2014-04-30T21:24:52Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=9dbe780174892a35d6f1e19495c13b9edb9d88fd'/>
<id>urn:sha1:9dbe780174892a35d6f1e19495c13b9edb9d88fd</id>
<content type='text'>
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>CodingGuidelines: give an example for redirection</title>
<updated>2014-05-02T20:24:57Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2014-04-30T21:24:24Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=6a49909b52b48592234da6a53bfe74ea34c302c6'/>
<id>urn:sha1:6a49909b52b48592234da6a53bfe74ea34c302c6</id>
<content type='text'>
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>CodingGuidelines: give an example for case/esac statement</title>
<updated>2014-05-02T20:24:57Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2014-04-30T21:24:08Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=79fc3ca1232099fd21e5bdef14c1b6fe1d2dbc1c'/>
<id>urn:sha1:79fc3ca1232099fd21e5bdef14c1b6fe1d2dbc1c</id>
<content type='text'>
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
</feed>
