Re: naming issue: SVN_FS_PROP_TXN_*

On Wed, Sep 19, 2007 at 11:26:20AM -0700, David Glasser wrote:
> Three property names are defined in libsvn_fs/fs-loader.h starting
> with SVN_FS_PROP_TXN_: SVN_FS_PROP_TXN_CHECK_LOCKS,
> SVN_FS_PROP_TXN_CHECK_OOD, and SVN_FS_PROP_TXN_CONTAINS_MERGEINFO.
>
> Shouldn't they be SVN_FS__PROP_TXN_*?
>

No, I think the first two are part of a public API, though I haven't
time to check just now.

> (Also, I don't think contains-mergeinfo is necessary: it's only used

(agreed).

> by FSFS, and you should be able to just see if the mergeinfo file
> exists on disk instead, no?)
>

Sure. I'd actually like to move the mergeinfo information -- currently
stored in a 'mergeinfo transaction property' file -- into the
already-existing transaction properties hash (since I see no reason why
we need a separate file). So we'd need to read that instead, but sure.

Regards,
Malcolm