[MERGE] Simplify the check_file_version_parents API some more. This has already changed in this release cycle.
John Arbash Meinel
john at arbash-meinel.com
Tue Nov 27 22:53:01 GMT 2007
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Robert Collins wrote:
> On Tue, 2007-11-27 at 17:32 -0500, John Arbash Meinel wrote:
>> John Arbash Meinel has voted approve.
>> Status is now: Approved
>> Comment:
>> I don't see anything about VersionedFileChecker's api being rather
>> different in 1.0rc1 versus 0.92. The values used to be passed into
>> VFC.__init__() and then were changed to VFC.check_*, and now changed to
>> be interpreted from the weave object.
>>
>> I'm okay with that, but it does seem like a change from specifying a
>> subset to specifying everything. And I would at least like to see a
>> comment in NEWS that VFC has been changed since 0.92.
>
> I'd like to just mark it as non-public, and note that in NEWS then.
>
> I think that there is a mis-meme that has arisen somewhere about
> public/privacy of methods. And its because we don't have enough
> granularity.
>
> How do we express 'public to bzrlib only' ???
>
> the versioned_file_checker is:
> - only relevant to a particular repo format
> - internal to check and reconcile, which *are dependent on a repository
> factory*.
>
> So there is no publicity in this class and method at all, name
> notwithstanding.
>
> -Rob
I'm okay with that. But we've used _KnitData and _KnitIndex for that purpose.
So it seems we should certainly call this _VersionedFileChecker.
John
=:->
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.7 (Darwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iD8DBQFHTJ/NJdeBCYSNAAMRAncGAJ9KiuLOlthWNVkoMB1w/c5YJctQBACg2BFz
Qe7dHNlHvyxhPTr5PofHGbM=
=/YZc
-----END PGP SIGNATURE-----
More information about the bazaar
mailing list