Skip to content

Commit a7d4c49

Browse files
committed
builtin/apply: remove misleading comment on lock_file field
Just like pointer field like prefix, the piece of memory pointed at by lock_file field is not owned by the apply_state structure. It is true that the caller needs to be careful about the lifetime rule for lockfile instances, but that is none of this API's business. Signed-off-by: Junio C Hamano <gitster@pobox.com>
1 parent a1bc3dd commit a7d4c49

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

builtin/apply.c

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -52,10 +52,7 @@ struct apply_state {
5252
const char *prefix;
5353
int prefix_length;
5454

55-
/*
56-
* Since lockfile.c keeps a linked list of all created
57-
* lock_file structures, it isn't safe to free(lock_file).
58-
*/
55+
/* These are lock_file related */
5956
struct lock_file *lock_file;
6057
int newfd;
6158

0 commit comments

Comments
 (0)