Based on the tests and the decrease of roundtrips: approved !
16 + def _vfs_checkout_metadir(self): 17 + self._ensure_real() 18 + return self._real_bzrdir.checkout_metadir() 19 + 20 + def checkout_metadir(self): 21 + medium = self._client._medium 22 + if medium._is_remote_before((2, 5)): 23 + return self._vfs_checkout_metadir()
_vfs_checkout_metadir -> checkout_metadir -> _vfs_checkout_metadir ?
Is that a loop or do I miss something obvious ?
Up to you to wait for spiv's review.
« Back to merge proposal
Based on the tests and the decrease of roundtrips: approved !
16 + def _vfs_checkout_ metadir( self): bzrdir. checkout_ metadir( ) metadir( self): _medium _is_remote_ before( (2, 5)): checkout_ metadir( )
17 + self._ensure_real()
18 + return self._real_
19 +
20 + def checkout_
21 + medium = self._client.
22 + if medium.
23 + return self._vfs_
_vfs_checkout_ metadir -> checkout_metadir -> _vfs_checkout_ metadir ?
Is that a loop or do I miss something obvious ?
Up to you to wait for spiv's review.