Thread (55 messages) 55 messages, 3 authors, 2021-03-15

Re: [PATCH v3 1/6] sequencer: export subject_length()

flat view

From: Eric Sunshine <hidden>
Date: 2021-03-01 20:29:46

On Mon, Mar 1, 2021 at 3:50 AM Charvi Mendiratta [off-list ref] wrote:
quoted hunk ↗ jump to hunk
This function can be used in other parts of git. Let's move the
function to commit.c.

Signed-off-by: Charvi Mendiratta <redacted>
---
diff --git a/commit.h b/commit.h
@@ -165,6 +165,9 @@ const void *detach_commit_buffer(struct commit *, unsigned long *sizep);
+/* Return length of the commit subject from commit log message. */
+size_t subject_length(const char *body);
Now that this function is public, is the name too generic? Most other
functions in this header have "commit" in the name. So,
commit_subject_length() might be one possibility (assuming the current
name is too generic).
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help