Thread (5 messages) flat view 5 messages, 2 authors, 2016-06-15

Re: [PATCH] commit: Append commit_list prefix in two function names.

From: Jonathan Nieder <hidden>
Date: 2016-06-15 22:50:02

Thiago Farina wrote:
Append commit_lis prefix to insert_by_date function and to sort_by_date,
so it's clear that these functions refer to commit_list structure.
Nits: by "Append" I think you mean "Add"; missing "t" after commit_lis.

This gives the oft-used insert_by_name() function a fairly long name:

	commit_list_insert_by_name

The proposed name is long enough to be unwieldly.  It might have
the virtue of fitting better with some of the commit_list lib:

	commit_list_count
	commit_list_insert
	free_commit_list

Compare:

	sort_by_date
	pop_most_recent_commit
	sort_in_topological_order
	pop_commit

Perhaps this change would work better if some of the others were
renamed at the same time?

Some notes on commit_lists in api-revision-walking.txt would do even
more to make it intuitive. :)

Hope that helps,
Jonathan
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help