diff options
| author |  Kjetil Barvik <barvik@broadpark.no> | 2009-02-09 21:54:06 +0100 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2009-02-09 20:59:26 -0800 |
| commit | 571998921d8fd4ee674545406aabb86987921252 (patch) | |
| tree | 519fb9c8f6d7a9e1e3fcbc1d79fb22420a4b3582 /Documentation/CodingGuidelines | |
| parent | lstat_cache(): generalise longest_match_lstat_cache() (diff) | |
| download | tgif-571998921d8fd4ee674545406aabb86987921252.tar.xz | |
lstat_cache(): swap func(length, string) into func(string, length)
Swap function argument pair (length, string) into (string, length) to
conform with the commonly used order inside the GIT source code.
Also, add a note about this fact into the coding guidelines.
Signed-off-by: Kjetil Barvik <barvik@broadpark.no>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'Documentation/CodingGuidelines')
| -rw-r--r-- | Documentation/CodingGuidelines | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/Documentation/CodingGuidelines b/Documentation/CodingGuidelines index 0d7fa9cca9..b8bf618a30 100644 --- a/Documentation/CodingGuidelines +++ b/Documentation/CodingGuidelines @@ -129,3 +129,6 @@ For C programs: used in the git core command set (unless your command is clearly separate from it, such as an importer to convert random-scm-X repositories to git). + + - When we pass <string, length> pair to functions, we should try to + pass them in that order. |