[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r11410 - trunk/subversion/libsvn_fs_fs

From: <kfogel_at_collab.net>
Date: 2004-10-14 22:00:27 CEST

bliss@tigris.org writes:
> Log:
> Followup to r11364 to fix a problem pointed out by ghudson, namely
> that an empty file is not a valid revprop file. While changing this,
> move the creation of the revprop file back up to where the svn:date
> revprop is set.
>
> * subversion/libsvn_fs_fs/fs_fs.c
> (svn_fs_fs__set_revision_proplist): Make it create the revprop file
> if it does not exist.
>
> * subversion/libsvn_fs_fs/dag.c
> (svn_fs_fs__dag_init_fs): Use svn_fs_fs__set_revision_proplist to
> create the revprop file for revision zero.

Minor log message nit:

The important effect here was not merely that you moved *where* the
creation of the revprop file happens, but that you changed its initial
content. That's an important point for the summarizing paragraph, I
would think.

-Karl

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Oct 14 23:50:06 2004

This is an archived mail posted to the Subversion Dev mailing list.