The set-like object returned by dirstate.dirs may be difficult for other
implementations of the dirstate to provide, and is unnecessary as it is
only ever used for contains. Instead, provide an explicit method for
testing for a directory.
.. api::
dirstate no longer provides a `dirs()` method. To test for the existence of a directory in the dirstate, use `dirstate.hasdir(dirname)`.
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
Looks good. For bonus points do one more upload with the right .. api:: annotation for the releasenotes extension.
| mercurial/dirstate.py | ||
|---|---|---|
| 393 | The external dirstate object only provides hasdir(), however the internal object needs to provide two objects as there are two behaviours it needs to expose: hastrackeddir() considers only files that are not marked as removed. This is used almost everywhere. hasdir() additionally considers directories that contain only removed files. This is used in one place below (line 803). I don't understand what _walkexplicit is trying to do, so I haven't changed it. I'm not convinced _walkexplicit is correct, though. | |