* [PATCH] attr: fix instructions on how to check attrs
@ 2023-01-22 3:06 John Cai via GitGitGadget
2023-01-22 16:10 ` Junio C Hamano
2023-01-23 4:22 ` [PATCH v2] " John Cai via GitGitGadget
0 siblings, 2 replies; 6+ messages in thread
From: John Cai via GitGitGadget @ 2023-01-22 3:06 UTC (permalink / raw)
To: git; +Cc: John Cai, John Cai
From: John Cai <johncai86@gmail.com>
The instructions in attr.h describing what functions to call to check
attributes is missing the index as the first argument to git_check_attr.
Fix this to make it consistent with the actual function signature.
Signed-off-by: John Cai <johncai86@gmail.com>
---
attr: fix instructions on how to check attrs
The instructions in attr.h describing what functions to call to check
attributes is missing the index as the first argument to git_check_attr.
Fix this to make it consistent with the actual function signature.
Signed-off-by: John Cai johncai86@gmail.com
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-1441%2Fjohn-cai%2Fjc%2Ffix-attr-docs-v1
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-1441/john-cai/jc/fix-attr-docs-v1
Pull-Request: https://github.com/git/git/pull/1441
attr.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/attr.h b/attr.h
index 2f22dffadb3..47f1111f391 100644
--- a/attr.h
+++ b/attr.h
@@ -45,7 +45,7 @@
* const char *path;
*
* setup_check();
- * git_check_attr(path, check);
+ * git_check_attr(&the_index, path, check);
* ------------
*
* - Act on `.value` member of the result, left in `check->items[]`:
base-commit: 904d404274fef6695c78a6b055edd184b72e2f9b
--
gitgitgadget
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH] attr: fix instructions on how to check attrs 2023-01-22 3:06 [PATCH] attr: fix instructions on how to check attrs John Cai via GitGitGadget @ 2023-01-22 16:10 ` Junio C Hamano 2023-01-23 4:07 ` John Cai 2023-01-23 4:22 ` [PATCH v2] " John Cai via GitGitGadget 1 sibling, 1 reply; 6+ messages in thread From: Junio C Hamano @ 2023-01-22 16:10 UTC (permalink / raw) To: John Cai via GitGitGadget; +Cc: git, John Cai "John Cai via GitGitGadget" <gitgitgadget@gmail.com> writes: > From: John Cai <johncai86@gmail.com> > > The instructions in attr.h describing what functions to call to check > attributes is missing the index as the first argument to git_check_attr. > > Fix this to make it consistent with the actual function signature. Sounds quite sensible. It would have been very good to explain some research in the above, like When 7a400a2c (attr: remove an implicit dependency on the_index, 2018-08-13) started passing an index_state instance to git_check_attr(), it forgot to update the API documentation that was in Documentation/technical/api-gitattributes.txt. Later, 3a1b3415 (attr: move doc to attr.h, 2019-11-17) moved the API documentation to attr.h and made it to a comment, without realizing the earlier mistake. or something like that. Thanks. > diff --git a/attr.h b/attr.h > index 2f22dffadb3..47f1111f391 100644 > --- a/attr.h > +++ b/attr.h > @@ -45,7 +45,7 @@ > * const char *path; > * > * setup_check(); > - * git_check_attr(path, check); > + * git_check_attr(&the_index, path, check); > * ------------ > * > * - Act on `.value` member of the result, left in `check->items[]`: > > base-commit: 904d404274fef6695c78a6b055edd184b72e2f9b ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] attr: fix instructions on how to check attrs 2023-01-22 16:10 ` Junio C Hamano @ 2023-01-23 4:07 ` John Cai 0 siblings, 0 replies; 6+ messages in thread From: John Cai @ 2023-01-23 4:07 UTC (permalink / raw) To: Junio C Hamano; +Cc: John Cai via GitGitGadget, git Hi Junio, On 22 Jan 2023, at 11:10, Junio C Hamano wrote: > "John Cai via GitGitGadget" <gitgitgadget@gmail.com> writes: > >> From: John Cai <johncai86@gmail.com> >> >> The instructions in attr.h describing what functions to call to check >> attributes is missing the index as the first argument to git_check_attr. >> >> Fix this to make it consistent with the actual function signature. > > Sounds quite sensible. It would have been very good to explain some > research in the above, like > > When 7a400a2c (attr: remove an implicit dependency on the_index, > 2018-08-13) started passing an index_state instance to > git_check_attr(), it forgot to update the API documentation that > was in Documentation/technical/api-gitattributes.txt. Later, > 3a1b3415 (attr: move doc to attr.h, 2019-11-17) moved the API > documentation to attr.h and made it to a comment, without > realizing the earlier mistake. > > or something like that. good tip about including some history. I'll include that in the re-roll thanks! > > Thanks. > >> diff --git a/attr.h b/attr.h >> index 2f22dffadb3..47f1111f391 100644 >> --- a/attr.h >> +++ b/attr.h >> @@ -45,7 +45,7 @@ >> * const char *path; >> * >> * setup_check(); >> - * git_check_attr(path, check); >> + * git_check_attr(&the_index, path, check); >> * ------------ >> * >> * - Act on `.value` member of the result, left in `check->items[]`: >> >> base-commit: 904d404274fef6695c78a6b055edd184b72e2f9b ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2] attr: fix instructions on how to check attrs 2023-01-22 3:06 [PATCH] attr: fix instructions on how to check attrs John Cai via GitGitGadget 2023-01-22 16:10 ` Junio C Hamano @ 2023-01-23 4:22 ` John Cai via GitGitGadget 2023-01-26 21:21 ` [PATCH v3] " John Cai via GitGitGadget 1 sibling, 1 reply; 6+ messages in thread From: John Cai via GitGitGadget @ 2023-01-23 4:22 UTC (permalink / raw) To: git; +Cc: John Cai, John Cai From: John Cai <johncai86@gmail.com> The instructions in attr.h describing what functions to call to check attributes is missing the index as the first argument to git_check_attr(). When 7a400a2c (attr: remove an implicit dependency on the_index, 2018-08-13) started passing an index_state instance to git_check_attr(), it forgot to update the API documentation in Documentation/technical/api-gitattributes.txt. Later, 3a1b3415 (attr: move doc to attr.h, 2019-11-17) moved the API documentation to attr.h as a comment, but still left out the index_state as an argument. Fix this to make the documentation in the comment consistent with the actual function signature. Signed-off-by: John Cai <johncai86@gmail.com> --- attr: fix instructions on how to check attrs The instructions in attr.h describing what functions to call to check attributes is missing the index as the first argument to git_check_attr. Fix this to make it consistent with the actual function signature. Changes since V1: * updated commit message to include some history Signed-off-by: John Cai johncai86@gmail.com Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-1441%2Fjohn-cai%2Fjc%2Ffix-attr-docs-v2 Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-1441/john-cai/jc/fix-attr-docs-v2 Pull-Request: https://github.com/git/git/pull/1441 Range-diff vs v1: 1: 63bb84db487 ! 1: 8cfee55e48f attr: fix instructions on how to check attrs @@ Commit message attr: fix instructions on how to check attrs The instructions in attr.h describing what functions to call to check - attributes is missing the index as the first argument to git_check_attr. + attributes is missing the index as the first argument to + git_check_attr(). - Fix this to make it consistent with the actual function signature. + When 7a400a2c (attr: remove an implicit dependency on the_index, + 2018-08-13) started passing an index_state instance to git_check_attr(), + it forgot to update the API documentation in + Documentation/technical/api-gitattributes.txt. Later, 3a1b3415 + (attr: move doc to attr.h, 2019-11-17) moved the API documentation to + attr.h as a comment, but still left out the index_state as an argument. + + Fix this to make the documentation in the comment consistent with the + actual function signature. Signed-off-by: John Cai <johncai86@gmail.com> attr.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/attr.h b/attr.h index 2f22dffadb3..47f1111f391 100644 --- a/attr.h +++ b/attr.h @@ -45,7 +45,7 @@ * const char *path; * * setup_check(); - * git_check_attr(path, check); + * git_check_attr(&the_index, path, check); * ------------ * * - Act on `.value` member of the result, left in `check->items[]`: base-commit: 904d404274fef6695c78a6b055edd184b72e2f9b -- gitgitgadget ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH v3] attr: fix instructions on how to check attrs 2023-01-23 4:22 ` [PATCH v2] " John Cai via GitGitGadget @ 2023-01-26 21:21 ` John Cai via GitGitGadget 2023-01-26 22:17 ` Junio C Hamano 0 siblings, 1 reply; 6+ messages in thread From: John Cai via GitGitGadget @ 2023-01-26 21:21 UTC (permalink / raw) To: git; +Cc: John Cai, John Cai From: John Cai <johncai86@gmail.com> The instructions in attr.h describing what functions to call to check attributes is missing the index as the first argument to git_check_attr(), as well as tree_oid as the second argument. When 7a400a2c (attr: remove an implicit dependency on the_index, 2018-08-13) started passing an index_state instance to git_check_attr(), it forgot to update the API documentation in Documentation/technical/api-gitattributes.txt. Later, 3a1b3415 (attr: move doc to attr.h, 2019-11-17) moved the API documentation to attr.h as a comment, but still left out the index_state as an argument. In 47cfc9b (attr: add flag `--source` to work with tree-ish 2023-01-14) added tree_oid as an optional parameter but was not added to the docs in attr.h Fix this to make the documentation in the comment consistent with the actual function signature. Signed-off-by: John Cai <johncai86@gmail.com> --- attr: fix instructions on how to check attrs The instructions in attr.h describing what functions to call to check attributes is missing the index as the first argument to git_check_attr. Fix this to make it consistent with the actual function signature. Changes since V2: * updated with adding second argument after rebasing against master Changes since V1: * updated commit message to include some history Signed-off-by: John Cai johncai86@gmail.com Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-1441%2Fjohn-cai%2Fjc%2Ffix-attr-docs-v3 Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-1441/john-cai/jc/fix-attr-docs-v3 Pull-Request: https://github.com/git/git/pull/1441 Range-diff vs v2: 1: 8cfee55e48f ! 1: cf6f456af47 attr: fix instructions on how to check attrs @@ Commit message The instructions in attr.h describing what functions to call to check attributes is missing the index as the first argument to - git_check_attr(). + git_check_attr(), as well as tree_oid as the second argument. When 7a400a2c (attr: remove an implicit dependency on the_index, 2018-08-13) started passing an index_state instance to git_check_attr(), @@ Commit message (attr: move doc to attr.h, 2019-11-17) moved the API documentation to attr.h as a comment, but still left out the index_state as an argument. + In 47cfc9b (attr: add flag `--source` to work with tree-ish 2023-01-14) + added tree_oid as an optional parameter but was not added to the docs in + attr.h + Fix this to make the documentation in the comment consistent with the actual function signature. @@ attr.h * * setup_check(); - * git_check_attr(path, check); -+ * git_check_attr(&the_index, path, check); ++ * git_check_attr(&the_index, tree_oid, path, check); * ------------ * * - Act on `.value` member of the result, left in `check->items[]`: attr.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/attr.h b/attr.h index 58a2bc1344f..9884ea2bc60 100644 --- a/attr.h +++ b/attr.h @@ -45,7 +45,7 @@ * const char *path; * * setup_check(); - * git_check_attr(path, check); + * git_check_attr(&the_index, tree_oid, path, check); * ------------ * * - Act on `.value` member of the result, left in `check->items[]`: base-commit: 5dec958dcf965fc75e0f459f8e8ccf9c9f495b15 -- gitgitgadget ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v3] attr: fix instructions on how to check attrs 2023-01-26 21:21 ` [PATCH v3] " John Cai via GitGitGadget @ 2023-01-26 22:17 ` Junio C Hamano 0 siblings, 0 replies; 6+ messages in thread From: Junio C Hamano @ 2023-01-26 22:17 UTC (permalink / raw) To: John Cai via GitGitGadget; +Cc: git, John Cai "John Cai via GitGitGadget" <gitgitgadget@gmail.com> writes: > Changes since V2: > > * updated with adding second argument after rebasing against master Ahh, thanks for being so careful. I forgot all about that other topic changing the function signature. ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2023-01-26 22:17 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2023-01-22 3:06 [PATCH] attr: fix instructions on how to check attrs John Cai via GitGitGadget 2023-01-22 16:10 ` Junio C Hamano 2023-01-23 4:07 ` John Cai 2023-01-23 4:22 ` [PATCH v2] " John Cai via GitGitGadget 2023-01-26 21:21 ` [PATCH v3] " John Cai via GitGitGadget 2023-01-26 22:17 ` Junio C Hamano
Code repositories for project(s) associated with this public inbox https://80x24.org/mirrors/git.git This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).