On Wed, Apr 18, 2012 at 14:22, Stefan Sperling <s...@elego.de> wrote: > On Wed, Apr 18, 2012 at 12:33:23PM -0000, stef...@apache.org wrote: >> Author: stefan2 >> Date: Wed Apr 18 12:33:23 2012 >> New Revision: 1327477 >> >> URL: http://svn.apache.org/viewvc?rev=1327477&view=rev >> Log: >> Naming policy: Rename all svn_*.c to just *.c in /subversion/libsvn_subr. > > Was there a very important reason for this change?
It was in response to a commit review where Hyrum noted the "svn_" prefix is redundant. > This change can cause tree-conflicts during backports that affect > these files. It's not likely to cause a lot of harm since we're > looking at a small amount of files that rarely receive backport > changes. Still, a heads-up or question about concerns on dev@ would > have been nice since this creates extra work during backports > for no apparent gain. I applaud the corrected names :-P > > A minor nit: You forgot to list svn_temp_serializer.c in the log message. And svn_types.c Cheers, -g
