On Thu, May 20, 2010 at 13:04, <julianf...@apache.org> wrote: > Author: julianfoad > Date: Thu May 20 17:04:56 2010 > New Revision: 946713 > > URL: http://svn.apache.org/viewvc?rev=946713&view=rev > Log: > Move a function to where it can be shared, and give it a better name. > > * subversion/libsvn_wc/adm_files.h, > subversion/libsvn_wc/adm_files.c > (svn_wc__get_ultimate_base_md5_checksum): New function, based on
More naming terminology? When I see this, it makes me ask, "What does "ultimate" mean?" The docstring doesn't make it clear why this new term is present. The underlying db function is svn_wc__db_base_get_info(), so why wouldn't this simply be svn_wc__base_get_md5_checksum() ? >... Cheers, -g