~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/dirstate.py

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2010-03-25 00:02:51 UTC
  • mfrom: (5106.1.1 version-bump)
  • Revision ID: pqm@pqm.ubuntu.com-20100325000251-bwsv5c5d3l9x3lnn
(Jelmer) Bump API version for 2.2.0.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2006-2011 Canonical Ltd
 
1
# Copyright (C) 2006-2010 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
20
20
lines by NL. The field delimiters are ommitted in the grammar, line delimiters
21
21
are not - this is done for clarity of reading. All string data is in utf8.
22
22
 
23
 
::
24
 
 
25
 
    MINIKIND = "f" | "d" | "l" | "a" | "r" | "t";
26
 
    NL = "\\n";
27
 
    NULL = "\\0";
28
 
    WHOLE_NUMBER = {digit}, digit;
29
 
    BOOLEAN = "y" | "n";
30
 
    REVISION_ID = a non-empty utf8 string;
31
 
    
32
 
    dirstate format = header line, full checksum, row count, parent details,
33
 
     ghost_details, entries;
34
 
    header line = "#bazaar dirstate flat format 3", NL;
35
 
    full checksum = "crc32: ", ["-"], WHOLE_NUMBER, NL;
36
 
    row count = "num_entries: ", WHOLE_NUMBER, NL;
37
 
    parent_details = WHOLE NUMBER, {REVISION_ID}* NL;
38
 
    ghost_details = WHOLE NUMBER, {REVISION_ID}*, NL;
39
 
    entries = {entry};
40
 
    entry = entry_key, current_entry_details, {parent_entry_details};
41
 
    entry_key = dirname,  basename, fileid;
42
 
    current_entry_details = common_entry_details, working_entry_details;
43
 
    parent_entry_details = common_entry_details, history_entry_details;
44
 
    common_entry_details = MINIKIND, fingerprint, size, executable
45
 
    working_entry_details = packed_stat
46
 
    history_entry_details = REVISION_ID;
47
 
    executable = BOOLEAN;
48
 
    size = WHOLE_NUMBER;
49
 
    fingerprint = a nonempty utf8 sequence with meaning defined by minikind.
50
 
 
51
 
Given this definition, the following is useful to know::
52
 
 
53
 
    entry (aka row) - all the data for a given key.
54
 
    entry[0]: The key (dirname, basename, fileid)
55
 
    entry[0][0]: dirname
56
 
    entry[0][1]: basename
57
 
    entry[0][2]: fileid
58
 
    entry[1]: The tree(s) data for this path and id combination.
59
 
    entry[1][0]: The current tree
60
 
    entry[1][1]: The second tree
61
 
 
62
 
For an entry for a tree, we have (using tree 0 - current tree) to demonstrate::
63
 
 
64
 
    entry[1][0][0]: minikind
65
 
    entry[1][0][1]: fingerprint
66
 
    entry[1][0][2]: size
67
 
    entry[1][0][3]: executable
68
 
    entry[1][0][4]: packed_stat
69
 
 
70
 
OR (for non tree-0)::
71
 
 
72
 
    entry[1][1][4]: revision_id
 
23
MINIKIND = "f" | "d" | "l" | "a" | "r" | "t";
 
24
NL = "\n";
 
25
NULL = "\0";
 
26
WHOLE_NUMBER = {digit}, digit;
 
27
BOOLEAN = "y" | "n";
 
28
REVISION_ID = a non-empty utf8 string;
 
29
 
 
30
dirstate format = header line, full checksum, row count, parent details,
 
31
 ghost_details, entries;
 
32
header line = "#bazaar dirstate flat format 3", NL;
 
33
full checksum = "crc32: ", ["-"], WHOLE_NUMBER, NL;
 
34
row count = "num_entries: ", WHOLE_NUMBER, NL;
 
35
parent_details = WHOLE NUMBER, {REVISION_ID}* NL;
 
36
ghost_details = WHOLE NUMBER, {REVISION_ID}*, NL;
 
37
entries = {entry};
 
38
entry = entry_key, current_entry_details, {parent_entry_details};
 
39
entry_key = dirname,  basename, fileid;
 
40
current_entry_details = common_entry_details, working_entry_details;
 
41
parent_entry_details = common_entry_details, history_entry_details;
 
42
common_entry_details = MINIKIND, fingerprint, size, executable
 
43
working_entry_details = packed_stat
 
44
history_entry_details = REVISION_ID;
 
45
executable = BOOLEAN;
 
46
size = WHOLE_NUMBER;
 
47
fingerprint = a nonempty utf8 sequence with meaning defined by minikind.
 
48
 
 
49
Given this definition, the following is useful to know:
 
50
entry (aka row) - all the data for a given key.
 
51
entry[0]: The key (dirname, basename, fileid)
 
52
entry[0][0]: dirname
 
53
entry[0][1]: basename
 
54
entry[0][2]: fileid
 
55
entry[1]: The tree(s) data for this path and id combination.
 
56
entry[1][0]: The current tree
 
57
entry[1][1]: The second tree
 
58
 
 
59
For an entry for a tree, we have (using tree 0 - current tree) to demonstrate:
 
60
entry[1][0][0]: minikind
 
61
entry[1][0][1]: fingerprint
 
62
entry[1][0][2]: size
 
63
entry[1][0][3]: executable
 
64
entry[1][0][4]: packed_stat
 
65
OR (for non tree-0)
 
66
entry[1][1][4]: revision_id
73
67
 
74
68
There may be multiple rows at the root, one per id present in the root, so the
75
 
in memory root row is now::
76
 
 
77
 
    self._dirblocks[0] -> ('', [entry ...]),
78
 
 
79
 
and the entries in there are::
80
 
 
81
 
    entries[0][0]: ''
82
 
    entries[0][1]: ''
83
 
    entries[0][2]: file_id
84
 
    entries[1][0]: The tree data for the current tree for this fileid at /
85
 
    etc.
86
 
 
87
 
Kinds::
88
 
 
89
 
    'r' is a relocated entry: This path is not present in this tree with this
90
 
        id, but the id can be found at another location. The fingerprint is
91
 
        used to point to the target location.
92
 
    'a' is an absent entry: In that tree the id is not present at this path.
93
 
    'd' is a directory entry: This path in this tree is a directory with the
94
 
        current file id. There is no fingerprint for directories.
95
 
    'f' is a file entry: As for directory, but it's a file. The fingerprint is
96
 
        the sha1 value of the file's canonical form, i.e. after any read
97
 
        filters have been applied to the convenience form stored in the working
98
 
        tree.
99
 
    'l' is a symlink entry: As for directory, but a symlink. The fingerprint is
100
 
        the link target.
101
 
    't' is a reference to a nested subtree; the fingerprint is the referenced
102
 
        revision.
 
69
in memory root row is now:
 
70
self._dirblocks[0] -> ('', [entry ...]),
 
71
and the entries in there are
 
72
entries[0][0]: ''
 
73
entries[0][1]: ''
 
74
entries[0][2]: file_id
 
75
entries[1][0]: The tree data for the current tree for this fileid at /
 
76
etc.
 
77
 
 
78
Kinds:
 
79
'r' is a relocated entry: This path is not present in this tree with this id,
 
80
    but the id can be found at another location. The fingerprint is used to
 
81
    point to the target location.
 
82
'a' is an absent entry: In that tree the id is not present at this path.
 
83
'd' is a directory entry: This path in this tree is a directory with the
 
84
    current file id. There is no fingerprint for directories.
 
85
'f' is a file entry: As for directory, but it's a file. The fingerprint is the
 
86
    sha1 value of the file's canonical form, i.e. after any read filters have
 
87
    been applied to the convenience form stored in the working tree.
 
88
'l' is a symlink entry: As for directory, but a symlink. The fingerprint is the
 
89
    link target.
 
90
't' is a reference to a nested subtree; the fingerprint is the referenced
 
91
    revision.
103
92
 
104
93
Ordering:
105
94
 
106
 
The entries on disk and in memory are ordered according to the following keys::
 
95
The entries on disk and in memory are ordered according to the following keys:
107
96
 
108
97
    directory, as a list of components
109
98
    filename
110
99
    file-id
111
100
 
112
101
--- Format 1 had the following different definition: ---
113
 
 
114
 
::
115
 
 
116
 
    rows = dirname, NULL, basename, NULL, MINIKIND, NULL, fileid_utf8, NULL,
117
 
        WHOLE NUMBER (* size *), NULL, packed stat, NULL, sha1|symlink target,
118
 
        {PARENT ROW}
119
 
    PARENT ROW = NULL, revision_utf8, NULL, MINIKIND, NULL, dirname, NULL,
120
 
        basename, NULL, WHOLE NUMBER (* size *), NULL, "y" | "n", NULL,
121
 
        SHA1
 
102
rows = dirname, NULL, basename, NULL, MINIKIND, NULL, fileid_utf8, NULL,
 
103
    WHOLE NUMBER (* size *), NULL, packed stat, NULL, sha1|symlink target,
 
104
    {PARENT ROW}
 
105
PARENT ROW = NULL, revision_utf8, NULL, MINIKIND, NULL, dirname, NULL,
 
106
    basename, NULL, WHOLE NUMBER (* size *), NULL, "y" | "n", NULL,
 
107
    SHA1
122
108
 
123
109
PARENT ROW's are emitted for every parent that is not in the ghosts details
124
110
line. That is, if the parents are foo, bar, baz, and the ghosts are bar, then
149
135
----
150
136
 
151
137
Design priorities:
152
 
 1. Fast end to end use for bzr's top 5 uses cases. (commmit/diff/status/merge/???)
153
 
 2. fall back current object model as needed.
154
 
 3. scale usably to the largest trees known today - say 50K entries. (mozilla
 
138
 1) Fast end to end use for bzr's top 5 uses cases. (commmit/diff/status/merge/???)
 
139
 2) fall back current object model as needed.
 
140
 3) scale usably to the largest trees known today - say 50K entries. (mozilla
155
141
    is an example of this)
156
142
 
157
143
 
158
144
Locking:
159
 
 
160
145
 Eventually reuse dirstate objects across locks IFF the dirstate file has not
161
146
 been modified, but will require that we flush/ignore cached stat-hit data
162
147
 because we won't want to restat all files on disk just because a lock was
163
148
 acquired, yet we cannot trust the data after the previous lock was released.
164
149
 
165
 
Memory representation::
166
 
 
 
150
Memory representation:
167
151
 vector of all directories, and vector of the childen ?
168
152
   i.e.
169
153
     root_entrie = (direntry for root, [parent_direntries_for_root]),
183
167
    - What's the risk of error here? Once we have the base format being processed
184
168
      we should have a net win regardless of optimality. So we are going to
185
169
      go with what seems reasonable.
186
 
 
187
170
open questions:
188
171
 
189
172
Maybe we should do a test profile of the core structure - 10K simulated
218
201
 
219
202
"""
220
203
 
221
 
from __future__ import absolute_import
222
 
 
223
204
import bisect
 
205
import binascii
224
206
import errno
225
207
import operator
226
208
import os
227
209
from stat import S_IEXEC
228
210
import stat
 
211
import struct
229
212
import sys
230
213
import time
231
214
import zlib
232
215
 
233
216
from bzrlib import (
234
217
    cache_utf8,
235
 
    config,
236
218
    debug,
237
219
    errors,
238
220
    inventory,
239
221
    lock,
240
222
    osutils,
241
 
    static_tuple,
242
223
    trace,
243
 
    urlutils,
244
224
    )
245
225
 
246
226
 
251
231
ERROR_DIRECTORY = 267
252
232
 
253
233
 
 
234
if not getattr(struct, '_compile', None):
 
235
    # Cannot pre-compile the dirstate pack_stat
 
236
    def pack_stat(st, _encode=binascii.b2a_base64, _pack=struct.pack):
 
237
        """Convert stat values into a packed representation."""
 
238
        return _encode(_pack('>LLLLLL', st.st_size, int(st.st_mtime),
 
239
            int(st.st_ctime), st.st_dev, st.st_ino & 0xFFFFFFFF,
 
240
            st.st_mode))[:-1]
 
241
else:
 
242
    # compile the struct compiler we need, so as to only do it once
 
243
    from _struct import Struct
 
244
    _compiled_pack = Struct('>LLLLLL').pack
 
245
    def pack_stat(st, _encode=binascii.b2a_base64, _pack=_compiled_pack):
 
246
        """Convert stat values into a packed representation."""
 
247
        # jam 20060614 it isn't really worth removing more entries if we
 
248
        # are going to leave it in packed form.
 
249
        # With only st_mtime and st_mode filesize is 5.5M and read time is 275ms
 
250
        # With all entries, filesize is 5.9M and read time is maybe 280ms
 
251
        # well within the noise margin
 
252
 
 
253
        # base64 encoding always adds a final newline, so strip it off
 
254
        # The current version
 
255
        return _encode(_pack(st.st_size, int(st.st_mtime), int(st.st_ctime),
 
256
            st.st_dev, st.st_ino & 0xFFFFFFFF, st.st_mode))[:-1]
 
257
        # This is 0.060s / 1.520s faster by not encoding as much information
 
258
        # return _encode(_pack('>LL', int(st.st_mtime), st.st_mode))[:-1]
 
259
        # This is not strictly faster than _encode(_pack())[:-1]
 
260
        # return '%X.%X.%X.%X.%X.%X' % (
 
261
        #      st.st_size, int(st.st_mtime), int(st.st_ctime),
 
262
        #      st.st_dev, st.st_ino, st.st_mode)
 
263
        # Similar to the _encode(_pack('>LL'))
 
264
        # return '%X.%X' % (int(st.st_mtime), st.st_mode)
 
265
 
 
266
 
254
267
class SHA1Provider(object):
255
268
    """An interface for getting sha1s of a file."""
256
269
 
341
354
    NOT_IN_MEMORY = 0
342
355
    IN_MEMORY_UNMODIFIED = 1
343
356
    IN_MEMORY_MODIFIED = 2
344
 
    IN_MEMORY_HASH_MODIFIED = 3 # Only hash-cache updates
345
357
 
346
358
    # A pack_stat (the x's) that is just noise and will never match the output
347
359
    # of base64 encode.
348
360
    NULLSTAT = 'x' * 32
349
 
    NULL_PARENT_DETAILS = static_tuple.StaticTuple('a', '', 0, False, '')
 
361
    NULL_PARENT_DETAILS = ('a', '', 0, False, '')
350
362
 
351
363
    HEADER_FORMAT_2 = '#bazaar dirstate flat format 2\n'
352
364
    HEADER_FORMAT_3 = '#bazaar dirstate flat format 3\n'
353
365
 
354
 
    def __init__(self, path, sha1_provider, worth_saving_limit=0):
 
366
    def __init__(self, path, sha1_provider):
355
367
        """Create a  DirState object.
356
368
 
357
369
        :param path: The path at which the dirstate file on disk should live.
358
370
        :param sha1_provider: an object meeting the SHA1Provider interface.
359
 
        :param worth_saving_limit: when the exact number of hash changed
360
 
            entries is known, only bother saving the dirstate if more than
361
 
            this count of entries have changed.
362
 
            -1 means never save hash changes, 0 means always save hash changes.
363
371
        """
364
372
        # _header_state and _dirblock_state represent the current state
365
373
        # of the dirstate metadata and the per-row data respectiely.
402
410
        # during commit.
403
411
        self._last_block_index = None
404
412
        self._last_entry_index = None
405
 
        # The set of known hash changes
406
 
        self._known_hash_changes = set()
407
 
        # How many hash changed entries can we have without saving
408
 
        self._worth_saving_limit = worth_saving_limit
409
 
        self._config_stack = config.LocationStack(urlutils.local_path_to_url(
410
 
            path))
411
413
 
412
414
    def __repr__(self):
413
415
        return "%s(%r)" % \
414
416
            (self.__class__.__name__, self._filename)
415
417
 
416
 
    def _mark_modified(self, hash_changed_entries=None, header_modified=False):
417
 
        """Mark this dirstate as modified.
418
 
 
419
 
        :param hash_changed_entries: if non-None, mark just these entries as
420
 
            having their hash modified.
421
 
        :param header_modified: mark the header modified as well, not just the
422
 
            dirblocks.
423
 
        """
424
 
        #trace.mutter_callsite(3, "modified hash entries: %s", hash_changed_entries)
425
 
        if hash_changed_entries:
426
 
            self._known_hash_changes.update([e[0] for e in hash_changed_entries])
427
 
            if self._dirblock_state in (DirState.NOT_IN_MEMORY,
428
 
                                        DirState.IN_MEMORY_UNMODIFIED):
429
 
                # If the dirstate is already marked a IN_MEMORY_MODIFIED, then
430
 
                # that takes precedence.
431
 
                self._dirblock_state = DirState.IN_MEMORY_HASH_MODIFIED
432
 
        else:
433
 
            # TODO: Since we now have a IN_MEMORY_HASH_MODIFIED state, we
434
 
            #       should fail noisily if someone tries to set
435
 
            #       IN_MEMORY_MODIFIED but we don't have a write-lock!
436
 
            # We don't know exactly what changed so disable smart saving
437
 
            self._dirblock_state = DirState.IN_MEMORY_MODIFIED
438
 
        if header_modified:
439
 
            self._header_state = DirState.IN_MEMORY_MODIFIED
440
 
 
441
 
    def _mark_unmodified(self):
442
 
        """Mark this dirstate as unmodified."""
443
 
        self._header_state = DirState.IN_MEMORY_UNMODIFIED
444
 
        self._dirblock_state = DirState.IN_MEMORY_UNMODIFIED
445
 
        self._known_hash_changes = set()
446
 
 
447
418
    def add(self, path, file_id, kind, stat, fingerprint):
448
419
        """Add a path to be tracked.
449
420
 
575
546
        if kind == 'directory':
576
547
           # insert a new dirblock
577
548
           self._ensure_block(block_index, entry_index, utf8path)
578
 
        self._mark_modified()
 
549
        self._dirblock_state = DirState.IN_MEMORY_MODIFIED
579
550
        if self._id_index:
580
 
            self._add_to_id_index(self._id_index, entry_key)
 
551
            self._id_index.setdefault(entry_key[2], set()).add(entry_key)
581
552
 
582
553
    def _bisect(self, paths):
583
554
        """Bisect through the disk structure for specific rows.
1047
1018
 
1048
1019
        self._ghosts = []
1049
1020
        self._parents = [parents[0]]
1050
 
        self._mark_modified(header_modified=True)
 
1021
        self._dirblock_state = DirState.IN_MEMORY_MODIFIED
 
1022
        self._header_state = DirState.IN_MEMORY_MODIFIED
1051
1023
 
1052
1024
    def _empty_parent_info(self):
1053
1025
        return [DirState.NULL_PARENT_DETAILS] * (len(self._parents) -
1182
1154
    def _fields_per_entry(self):
1183
1155
        """How many null separated fields should be in each entry row.
1184
1156
 
1185
 
        Each line now has an extra '\\n' field which is not used
 
1157
        Each line now has an extra '\n' field which is not used
1186
1158
        so we just skip over it
1187
 
 
1188
 
        entry size::
 
1159
        entry size:
1189
1160
            3 fields for the key
1190
1161
            + number of fields per tree_data (5) * tree count
1191
1162
            + newline
1292
1263
                    parent_trees.append((parent_id, parent_tree))
1293
1264
                    parent_tree.lock_read()
1294
1265
                result.set_parent_trees(parent_trees, [])
1295
 
                result.set_state_from_inventory(tree.root_inventory)
 
1266
                result.set_state_from_inventory(tree.inventory)
1296
1267
            finally:
1297
1268
                for revid, parent_tree in parent_trees:
1298
1269
                    parent_tree.unlock()
1304
1275
            raise
1305
1276
        return result
1306
1277
 
1307
 
    def _check_delta_is_valid(self, delta):
1308
 
        return list(inventory._check_delta_unique_ids(
1309
 
                    inventory._check_delta_unique_old_paths(
1310
 
                    inventory._check_delta_unique_new_paths(
1311
 
                    inventory._check_delta_ids_match_entry(
1312
 
                    inventory._check_delta_ids_are_valid(
1313
 
                    inventory._check_delta_new_path_entry_both_or_None(delta)))))))
1314
 
 
1315
1278
    def update_by_delta(self, delta):
1316
1279
        """Apply an inventory delta to the dirstate for tree 0
1317
1280
 
1335
1298
        new_ids = set()
1336
1299
        # This loop transforms the delta to single atomic operations that can
1337
1300
        # be executed and validated.
1338
 
        delta = sorted(self._check_delta_is_valid(delta), reverse=True)
1339
 
        for old_path, new_path, file_id, inv_entry in delta:
 
1301
        for old_path, new_path, file_id, inv_entry in sorted(
 
1302
            inventory._check_delta_unique_old_paths(
 
1303
            inventory._check_delta_unique_new_paths(
 
1304
            inventory._check_delta_ids_match_entry(
 
1305
            inventory._check_delta_ids_are_valid(
 
1306
            inventory._check_delta_new_path_entry_both_or_None(delta))))),
 
1307
            reverse=True):
1340
1308
            if (file_id in insertions) or (file_id in removals):
1341
 
                self._raise_invalid(old_path or new_path, file_id,
 
1309
                raise errors.InconsistentDelta(old_path or new_path, file_id,
1342
1310
                    "repeated file_id")
1343
1311
            if old_path is not None:
1344
1312
                old_path = old_path.encode('utf-8')
1347
1315
                new_ids.add(file_id)
1348
1316
            if new_path is not None:
1349
1317
                if inv_entry is None:
1350
 
                    self._raise_invalid(new_path, file_id,
 
1318
                    raise errors.InconsistentDelta(new_path, file_id,
1351
1319
                        "new_path with no entry")
1352
1320
                new_path = new_path.encode('utf-8')
1353
1321
                dirname_utf8, basename = osutils.split(new_path)
1394
1362
            # _get_entry raises BzrError when a request is inconsistent; we
1395
1363
            # want such errors to be shown as InconsistentDelta - and that 
1396
1364
            # fits the behaviour we trigger.
1397
 
            raise errors.InconsistentDeltaDelta(delta,
1398
 
                "error from _get_entry. %s" % (e,))
 
1365
            raise errors.InconsistentDeltaDelta(delta, "error from _get_entry.")
1399
1366
 
1400
1367
    def _apply_removals(self, removals):
1401
1368
        for file_id, path in sorted(removals, reverse=True,
1406
1373
            try:
1407
1374
                entry = self._dirblocks[block_i][1][entry_i]
1408
1375
            except IndexError:
1409
 
                self._raise_invalid(path, file_id,
 
1376
                self._changes_aborted = True
 
1377
                raise errors.InconsistentDelta(path, file_id,
1410
1378
                    "Wrong path for old path.")
1411
1379
            if not f_present or entry[1][0][0] in 'ar':
1412
 
                self._raise_invalid(path, file_id,
 
1380
                self._changes_aborted = True
 
1381
                raise errors.InconsistentDelta(path, file_id,
1413
1382
                    "Wrong path for old path.")
1414
1383
            if file_id != entry[0][2]:
1415
 
                self._raise_invalid(path, file_id,
 
1384
                self._changes_aborted = True
 
1385
                raise errors.InconsistentDelta(path, file_id,
1416
1386
                    "Attempt to remove path has wrong id - found %r."
1417
1387
                    % entry[0][2])
1418
1388
            self._make_absent(entry)
1428
1398
                # be due to it being in a parent tree, or a corrupt delta.
1429
1399
                for child_entry in self._dirblocks[block_i][1]:
1430
1400
                    if child_entry[1][0][0] not in ('r', 'a'):
1431
 
                        self._raise_invalid(path, entry[0][2],
 
1401
                        self._changes_aborted = True
 
1402
                        raise errors.InconsistentDelta(path, entry[0][2],
1432
1403
                            "The file id was deleted but its children were "
1433
1404
                            "not deleted.")
1434
1405
 
1438
1409
                self.update_minimal(key, minikind, executable, fingerprint,
1439
1410
                                    path_utf8=path_utf8)
1440
1411
        except errors.NotVersionedError:
1441
 
            self._raise_invalid(path_utf8.decode('utf8'), key[2],
 
1412
            self._changes_aborted = True
 
1413
            raise errors.InconsistentDelta(path_utf8.decode('utf8'), key[2],
1442
1414
                "Missing parent")
1443
1415
 
1444
1416
    def update_basis_by_delta(self, delta, new_revid):
1452
1424
        Note that an exception during the operation of this method will leave
1453
1425
        the dirstate in a corrupt state where it should not be saved.
1454
1426
 
 
1427
        Finally, we expect all changes to be synchronising the basis tree with
 
1428
        the working tree.
 
1429
 
1455
1430
        :param new_revid: The new revision id for the trees parent.
1456
1431
        :param delta: An inventory delta (see apply_inventory_delta) describing
1457
1432
            the changes from the current left most parent revision to new_revid.
1469
1444
 
1470
1445
        self._parents[0] = new_revid
1471
1446
 
1472
 
        delta = sorted(self._check_delta_is_valid(delta), reverse=True)
 
1447
        delta = sorted(delta, reverse=True)
1473
1448
        adds = []
1474
1449
        changes = []
1475
1450
        deletes = []
1496
1471
        new_ids = set()
1497
1472
        for old_path, new_path, file_id, inv_entry in delta:
1498
1473
            if inv_entry is not None and file_id != inv_entry.file_id:
1499
 
                self._raise_invalid(new_path, file_id,
 
1474
                raise errors.InconsistentDelta(new_path, file_id,
1500
1475
                    "mismatched entry file_id %r" % inv_entry)
1501
 
            if new_path is None:
1502
 
                new_path_utf8 = None
1503
 
            else:
 
1476
            if new_path is not None:
1504
1477
                if inv_entry is None:
1505
 
                    self._raise_invalid(new_path, file_id,
 
1478
                    raise errors.InconsistentDelta(new_path, file_id,
1506
1479
                        "new_path with no entry")
1507
1480
                new_path_utf8 = encode(new_path)
1508
1481
                # note the parent for validation
1510
1483
                if basename_utf8:
1511
1484
                    parents.add((dirname_utf8, inv_entry.parent_id))
1512
1485
            if old_path is None:
1513
 
                old_path_utf8 = None
1514
 
            else:
1515
 
                old_path_utf8 = encode(old_path)
1516
 
            if old_path is None:
1517
 
                adds.append((None, new_path_utf8, file_id,
 
1486
                adds.append((None, encode(new_path), file_id,
1518
1487
                    inv_to_entry(inv_entry), True))
1519
1488
                new_ids.add(file_id)
1520
1489
            elif new_path is None:
1521
 
                deletes.append((old_path_utf8, None, file_id, None, True))
1522
 
            elif (old_path, new_path) == root_only:
1523
 
                # change things in-place
1524
 
                # Note: the case of a parent directory changing its file_id
1525
 
                #       tends to break optimizations here, because officially
1526
 
                #       the file has actually been moved, it just happens to
1527
 
                #       end up at the same path. If we can figure out how to
1528
 
                #       handle that case, we can avoid a lot of add+delete
1529
 
                #       pairs for objects that stay put.
1530
 
                # elif old_path == new_path:
1531
 
                changes.append((old_path_utf8, new_path_utf8, file_id,
1532
 
                                inv_to_entry(inv_entry)))
1533
 
            else:
 
1490
                deletes.append((encode(old_path), None, file_id, None, True))
 
1491
            elif (old_path, new_path) != root_only:
1534
1492
                # Renames:
1535
1493
                # Because renames must preserve their children we must have
1536
1494
                # processed all relocations and removes before hand. The sort
1546
1504
                self._update_basis_apply_deletes(deletes)
1547
1505
                deletes = []
1548
1506
                # Split into an add/delete pair recursively.
1549
 
                adds.append((old_path_utf8, new_path_utf8, file_id,
1550
 
                             inv_to_entry(inv_entry), False))
 
1507
                adds.append((None, new_path_utf8, file_id,
 
1508
                    inv_to_entry(inv_entry), False))
1551
1509
                # Expunge deletes that we've seen so that deleted/renamed
1552
1510
                # children of a rename directory are handled correctly.
1553
 
                new_deletes = reversed(list(
1554
 
                    self._iter_child_entries(1, old_path_utf8)))
 
1511
                new_deletes = reversed(list(self._iter_child_entries(1,
 
1512
                    encode(old_path))))
1555
1513
                # Remove the current contents of the tree at orig_path, and
1556
1514
                # reinsert at the correct new path.
1557
1515
                for entry in new_deletes:
1558
 
                    child_dirname, child_basename, child_file_id = entry[0]
1559
 
                    if child_dirname:
1560
 
                        source_path = child_dirname + '/' + child_basename
 
1516
                    if entry[0][0]:
 
1517
                        source_path = entry[0][0] + '/' + entry[0][1]
1561
1518
                    else:
1562
 
                        source_path = child_basename
 
1519
                        source_path = entry[0][1]
1563
1520
                    if new_path_utf8:
1564
 
                        target_path = \
1565
 
                            new_path_utf8 + source_path[len(old_path_utf8):]
 
1521
                        target_path = new_path_utf8 + source_path[len(old_path):]
1566
1522
                    else:
1567
 
                        if old_path_utf8 == '':
 
1523
                        if old_path == '':
1568
1524
                            raise AssertionError("cannot rename directory to"
1569
 
                                                 " itself")
1570
 
                        target_path = source_path[len(old_path_utf8) + 1:]
 
1525
                                " itself")
 
1526
                        target_path = source_path[len(old_path) + 1:]
1571
1527
                    adds.append((None, target_path, entry[0][2], entry[1][1], False))
1572
1528
                    deletes.append(
1573
1529
                        (source_path, target_path, entry[0][2], None, False))
1574
1530
                deletes.append(
1575
 
                    (old_path_utf8, new_path_utf8, file_id, None, False))
1576
 
 
 
1531
                    (encode(old_path), new_path, file_id, None, False))
 
1532
            else:
 
1533
                # changes to just the root should not require remove/insertion
 
1534
                # of everything.
 
1535
                changes.append((encode(old_path), encode(new_path), file_id,
 
1536
                    inv_to_entry(inv_entry)))
1577
1537
        self._check_delta_ids_absent(new_ids, delta, 1)
1578
1538
        try:
1579
1539
            # Finish expunging deletes/first half of renames.
1589
1549
            if 'integrity error' not in str(e):
1590
1550
                raise
1591
1551
            # _get_entry raises BzrError when a request is inconsistent; we
1592
 
            # want such errors to be shown as InconsistentDelta - and that
1593
 
            # fits the behaviour we trigger.
1594
 
            raise errors.InconsistentDeltaDelta(delta,
1595
 
                "error from _get_entry. %s" % (e,))
 
1552
            # want such errors to be shown as InconsistentDelta - and that 
 
1553
            # fits the behaviour we trigger. Partof this is driven by dirstate
 
1554
            # only supporting deltas that turn the basis into a closer fit to
 
1555
            # the active tree.
 
1556
            raise errors.InconsistentDeltaDelta(delta, "error from _get_entry.")
1596
1557
 
1597
 
        self._mark_modified(header_modified=True)
 
1558
        self._dirblock_state = DirState.IN_MEMORY_MODIFIED
 
1559
        self._header_state = DirState.IN_MEMORY_MODIFIED
1598
1560
        self._id_index = None
1599
1561
        return
1600
1562
 
1604
1566
            return
1605
1567
        id_index = self._get_id_index()
1606
1568
        for file_id in new_ids:
1607
 
            for key in id_index.get(file_id, ()):
 
1569
            for key in id_index.get(file_id, []):
1608
1570
                block_i, entry_i, d_present, f_present = \
1609
1571
                    self._get_block_entry_index(key[0], key[1], tree_index)
1610
1572
                if not f_present:
1614
1576
                if entry[0][2] != file_id:
1615
1577
                    # Different file_id, so not what we want.
1616
1578
                    continue
1617
 
                self._raise_invalid(("%s/%s" % key[0:2]).decode('utf8'), file_id,
 
1579
                # NB: No changes made before this helper is called, so no need
 
1580
                # to set the _changes_aborted flag.
 
1581
                raise errors.InconsistentDelta(
 
1582
                    ("%s/%s" % key[0:2]).decode('utf8'), file_id,
1618
1583
                    "This file_id is new in the delta but already present in "
1619
1584
                    "the target")
1620
1585
 
1621
 
    def _raise_invalid(self, path, file_id, reason):
1622
 
        self._changes_aborted = True
1623
 
        raise errors.InconsistentDelta(path, file_id, reason)
1624
 
 
1625
1586
    def _update_basis_apply_adds(self, adds):
1626
1587
        """Apply a sequence of adds to tree 1 during update_basis_by_delta.
1627
1588
 
1635
1596
        """
1636
1597
        # Adds are accumulated partly from renames, so can be in any input
1637
1598
        # order - sort it.
1638
 
        # TODO: we may want to sort in dirblocks order. That way each entry
1639
 
        #       will end up in the same directory, allowing the _get_entry
1640
 
        #       fast-path for looking up 2 items in the same dir work.
1641
 
        adds.sort(key=lambda x: x[1])
 
1599
        adds.sort()
1642
1600
        # adds is now in lexographic order, which places all parents before
1643
1601
        # their children, so we can process it linearly.
1644
1602
        absent = 'ar'
1645
 
        st = static_tuple.StaticTuple
1646
1603
        for old_path, new_path, file_id, new_details, real_add in adds:
1647
 
            dirname, basename = osutils.split(new_path)
1648
 
            entry_key = st(dirname, basename, file_id)
1649
 
            block_index, present = self._find_block_index_from_key(entry_key)
1650
 
            if not present:
1651
 
                # The block where we want to put the file is not present.
1652
 
                # However, it might have just been an empty directory. Look for
1653
 
                # the parent in the basis-so-far before throwing an error.
1654
 
                parent_dir, parent_base = osutils.split(dirname)
1655
 
                parent_block_idx, parent_entry_idx, _, parent_present = \
1656
 
                    self._get_block_entry_index(parent_dir, parent_base, 1)
1657
 
                if not parent_present:
1658
 
                    self._raise_invalid(new_path, file_id,
1659
 
                        "Unable to find block for this record."
1660
 
                        " Was the parent added?")
1661
 
                self._ensure_block(parent_block_idx, parent_entry_idx, dirname)
1662
 
 
1663
 
            block = self._dirblocks[block_index][1]
1664
 
            entry_index, present = self._find_entry_index(entry_key, block)
1665
 
            if real_add:
1666
 
                if old_path is not None:
1667
 
                    self._raise_invalid(new_path, file_id,
1668
 
                        'considered a real add but still had old_path at %s'
1669
 
                        % (old_path,))
1670
 
            if present:
1671
 
                entry = block[entry_index]
1672
 
                basis_kind = entry[1][1][0]
1673
 
                if basis_kind == 'a':
1674
 
                    entry[1][1] = new_details
1675
 
                elif basis_kind == 'r':
1676
 
                    raise NotImplementedError()
1677
 
                else:
1678
 
                    self._raise_invalid(new_path, file_id,
1679
 
                        "An entry was marked as a new add"
1680
 
                        " but the basis target already existed")
1681
 
            else:
1682
 
                # The exact key was not found in the block. However, we need to
1683
 
                # check if there is a key next to us that would have matched.
1684
 
                # We only need to check 2 locations, because there are only 2
1685
 
                # trees present.
1686
 
                for maybe_index in range(entry_index-1, entry_index+1):
1687
 
                    if maybe_index < 0 or maybe_index >= len(block):
1688
 
                        continue
1689
 
                    maybe_entry = block[maybe_index]
1690
 
                    if maybe_entry[0][:2] != (dirname, basename):
1691
 
                        # Just a random neighbor
1692
 
                        continue
1693
 
                    if maybe_entry[0][2] == file_id:
1694
 
                        raise AssertionError(
1695
 
                            '_find_entry_index didnt find a key match'
1696
 
                            ' but walking the data did, for %s'
1697
 
                            % (entry_key,))
1698
 
                    basis_kind = maybe_entry[1][1][0]
1699
 
                    if basis_kind not in 'ar':
1700
 
                        self._raise_invalid(new_path, file_id,
1701
 
                            "we have an add record for path, but the path"
1702
 
                            " is already present with another file_id %s"
1703
 
                            % (maybe_entry[0][2],))
1704
 
 
1705
 
                entry = (entry_key, [DirState.NULL_PARENT_DETAILS,
1706
 
                                     new_details])
1707
 
                block.insert(entry_index, entry)
1708
 
 
1709
 
            active_kind = entry[1][0][0]
1710
 
            if active_kind == 'a':
1711
 
                # The active record shows up as absent, this could be genuine,
1712
 
                # or it could be present at some other location. We need to
1713
 
                # verify.
1714
 
                id_index = self._get_id_index()
1715
 
                # The id_index may not be perfectly accurate for tree1, because
1716
 
                # we haven't been keeping it updated. However, it should be
1717
 
                # fine for tree0, and that gives us enough info for what we
1718
 
                # need
1719
 
                keys = id_index.get(file_id, ())
1720
 
                for key in keys:
1721
 
                    block_i, entry_i, d_present, f_present = \
1722
 
                        self._get_block_entry_index(key[0], key[1], 0)
1723
 
                    if not f_present:
1724
 
                        continue
1725
 
                    active_entry = self._dirblocks[block_i][1][entry_i]
1726
 
                    if (active_entry[0][2] != file_id):
1727
 
                        # Some other file is at this path, we don't need to
1728
 
                        # link it.
1729
 
                        continue
1730
 
                    real_active_kind = active_entry[1][0][0]
1731
 
                    if real_active_kind in 'ar':
1732
 
                        # We found a record, which was not *this* record,
1733
 
                        # which matches the file_id, but is not actually
1734
 
                        # present. Something seems *really* wrong.
1735
 
                        self._raise_invalid(new_path, file_id,
1736
 
                            "We found a tree0 entry that doesnt make sense")
1737
 
                    # Now, we've found a tree0 entry which matches the file_id
1738
 
                    # but is at a different location. So update them to be
1739
 
                    # rename records.
1740
 
                    active_dir, active_name = active_entry[0][:2]
1741
 
                    if active_dir:
1742
 
                        active_path = active_dir + '/' + active_name
1743
 
                    else:
1744
 
                        active_path = active_name
1745
 
                    active_entry[1][1] = st('r', new_path, 0, False, '')
1746
 
                    entry[1][0] = st('r', active_path, 0, False, '')
1747
 
            elif active_kind == 'r':
1748
 
                raise NotImplementedError()
1749
 
 
1750
 
            new_kind = new_details[0]
1751
 
            if new_kind == 'd':
1752
 
                self._ensure_block(block_index, entry_index, new_path)
 
1604
            # the entry for this file_id must be in tree 0.
 
1605
            entry = self._get_entry(0, file_id, new_path)
 
1606
            if entry[0] is None or entry[0][2] != file_id:
 
1607
                self._changes_aborted = True
 
1608
                raise errors.InconsistentDelta(new_path, file_id,
 
1609
                    'working tree does not contain new entry')
 
1610
            if real_add and entry[1][1][0] not in absent:
 
1611
                self._changes_aborted = True
 
1612
                raise errors.InconsistentDelta(new_path, file_id,
 
1613
                    'The entry was considered to be a genuinely new record,'
 
1614
                    ' but there was already an old record for it.')
 
1615
            # We don't need to update the target of an 'r' because the handling
 
1616
            # of renames turns all 'r' situations into a delete at the original
 
1617
            # location.
 
1618
            entry[1][1] = new_details
1753
1619
 
1754
1620
    def _update_basis_apply_changes(self, changes):
1755
1621
        """Apply a sequence of changes to tree 1 during update_basis_by_delta.
1760
1626
        absent = 'ar'
1761
1627
        for old_path, new_path, file_id, new_details in changes:
1762
1628
            # the entry for this file_id must be in tree 0.
1763
 
            entry = self._get_entry(1, file_id, new_path)
1764
 
            if entry[0] is None or entry[1][1][0] in 'ar':
1765
 
                self._raise_invalid(new_path, file_id,
1766
 
                    'changed entry considered not present')
 
1629
            entry = self._get_entry(0, file_id, new_path)
 
1630
            if entry[0] is None or entry[0][2] != file_id:
 
1631
                self._changes_aborted = True
 
1632
                raise errors.InconsistentDelta(new_path, file_id,
 
1633
                    'working tree does not contain new entry')
 
1634
            if (entry[1][0][0] in absent or
 
1635
                entry[1][1][0] in absent):
 
1636
                self._changes_aborted = True
 
1637
                raise errors.InconsistentDelta(new_path, file_id,
 
1638
                    'changed considered absent')
1767
1639
            entry[1][1] = new_details
1768
1640
 
1769
1641
    def _update_basis_apply_deletes(self, deletes):
1781
1653
        null = DirState.NULL_PARENT_DETAILS
1782
1654
        for old_path, new_path, file_id, _, real_delete in deletes:
1783
1655
            if real_delete != (new_path is None):
1784
 
                self._raise_invalid(old_path, file_id, "bad delete delta")
 
1656
                self._changes_aborted = True
 
1657
                raise AssertionError("bad delete delta")
1785
1658
            # the entry for this file_id must be in tree 1.
1786
1659
            dirname, basename = osutils.split(old_path)
1787
1660
            block_index, entry_index, dir_present, file_present = \
1788
1661
                self._get_block_entry_index(dirname, basename, 1)
1789
1662
            if not file_present:
1790
 
                self._raise_invalid(old_path, file_id,
 
1663
                self._changes_aborted = True
 
1664
                raise errors.InconsistentDelta(old_path, file_id,
1791
1665
                    'basis tree does not contain removed entry')
1792
1666
            entry = self._dirblocks[block_index][1][entry_index]
1793
 
            # The state of the entry in the 'active' WT
1794
 
            active_kind = entry[1][0][0]
1795
1667
            if entry[0][2] != file_id:
1796
 
                self._raise_invalid(old_path, file_id,
 
1668
                self._changes_aborted = True
 
1669
                raise errors.InconsistentDelta(old_path, file_id,
1797
1670
                    'mismatched file_id in tree 1')
1798
 
            dir_block = ()
1799
 
            old_kind = entry[1][1][0]
1800
 
            if active_kind in 'ar':
1801
 
                # The active tree doesn't have this file_id.
1802
 
                # The basis tree is changing this record. If this is a
1803
 
                # rename, then we don't want the record here at all
1804
 
                # anymore. If it is just an in-place change, we want the
1805
 
                # record here, but we'll add it if we need to. So we just
1806
 
                # delete it
1807
 
                if active_kind == 'r':
1808
 
                    active_path = entry[1][0][1]
1809
 
                    active_entry = self._get_entry(0, file_id, active_path)
1810
 
                    if active_entry[1][1][0] != 'r':
1811
 
                        self._raise_invalid(old_path, file_id,
1812
 
                            "Dirstate did not have matching rename entries")
1813
 
                    elif active_entry[1][0][0] in 'ar':
1814
 
                        self._raise_invalid(old_path, file_id,
1815
 
                            "Dirstate had a rename pointing at an inactive"
1816
 
                            " tree0")
1817
 
                    active_entry[1][1] = null
 
1671
            if real_delete:
 
1672
                if entry[1][0][0] != 'a':
 
1673
                    self._changes_aborted = True
 
1674
                    raise errors.InconsistentDelta(old_path, file_id,
 
1675
                            'This was marked as a real delete, but the WT state'
 
1676
                            ' claims that it still exists and is versioned.')
1818
1677
                del self._dirblocks[block_index][1][entry_index]
1819
 
                if old_kind == 'd':
1820
 
                    # This was a directory, and the active tree says it
1821
 
                    # doesn't exist, and now the basis tree says it doesn't
1822
 
                    # exist. Remove its dirblock if present
1823
 
                    (dir_block_index,
1824
 
                     present) = self._find_block_index_from_key(
1825
 
                        (old_path, '', ''))
1826
 
                    if present:
1827
 
                        dir_block = self._dirblocks[dir_block_index][1]
1828
 
                        if not dir_block:
1829
 
                            # This entry is empty, go ahead and just remove it
1830
 
                            del self._dirblocks[dir_block_index]
1831
1678
            else:
1832
 
                # There is still an active record, so just mark this
1833
 
                # removed.
1834
 
                entry[1][1] = null
1835
 
                block_i, entry_i, d_present, f_present = \
1836
 
                    self._get_block_entry_index(old_path, '', 1)
1837
 
                if d_present:
1838
 
                    dir_block = self._dirblocks[block_i][1]
1839
 
            for child_entry in dir_block:
1840
 
                child_basis_kind = child_entry[1][1][0]
1841
 
                if child_basis_kind not in 'ar':
1842
 
                    self._raise_invalid(old_path, file_id,
1843
 
                        "The file id was deleted but its children were "
1844
 
                        "not deleted.")
 
1679
                if entry[1][0][0] == 'a':
 
1680
                    self._changes_aborted = True
 
1681
                    raise errors.InconsistentDelta(old_path, file_id,
 
1682
                        'The entry was considered a rename, but the source path'
 
1683
                        ' is marked as absent.')
 
1684
                    # For whatever reason, we were asked to rename an entry
 
1685
                    # that was originally marked as deleted. This could be
 
1686
                    # because we are renaming the parent directory, and the WT
 
1687
                    # current state has the file marked as deleted.
 
1688
                elif entry[1][0][0] == 'r':
 
1689
                    # implement the rename
 
1690
                    del self._dirblocks[block_index][1][entry_index]
 
1691
                else:
 
1692
                    # it is being resurrected here, so blank it out temporarily.
 
1693
                    self._dirblocks[block_index][1][entry_index][1][1] = null
1845
1694
 
1846
1695
    def _after_delta_check_parents(self, parents, index):
1847
1696
        """Check that parents required by the delta are all intact.
1856
1705
            # has the right file id.
1857
1706
            entry = self._get_entry(index, file_id, dirname_utf8)
1858
1707
            if entry[1] is None:
1859
 
                self._raise_invalid(dirname_utf8.decode('utf8'),
 
1708
                self._changes_aborted = True
 
1709
                raise errors.InconsistentDelta(dirname_utf8.decode('utf8'),
1860
1710
                    file_id, "This parent is not present.")
1861
1711
            # Parents of things must be directories
1862
1712
            if entry[1][index][0] != 'd':
1863
 
                self._raise_invalid(dirname_utf8.decode('utf8'),
 
1713
                self._changes_aborted = True
 
1714
                raise errors.InconsistentDelta(dirname_utf8.decode('utf8'),
1864
1715
                    file_id, "This parent is not a directory.")
1865
1716
 
1866
1717
    def _observed_sha1(self, entry, sha1, stat_value,
1867
 
        _stat_to_minikind=_stat_to_minikind):
 
1718
        _stat_to_minikind=_stat_to_minikind, _pack_stat=pack_stat):
1868
1719
        """Note the sha1 of a file.
1869
1720
 
1870
1721
        :param entry: The entry the sha1 is for.
1876
1727
        except KeyError:
1877
1728
            # Unhandled kind
1878
1729
            return None
 
1730
        packed_stat = _pack_stat(stat_value)
1879
1731
        if minikind == 'f':
1880
1732
            if self._cutoff_time is None:
1881
1733
                self._sha_cutoff_time()
1882
1734
            if (stat_value.st_mtime < self._cutoff_time
1883
1735
                and stat_value.st_ctime < self._cutoff_time):
1884
 
                entry[1][0] = ('f', sha1, stat_value.st_size, entry[1][0][3],
1885
 
                               pack_stat(stat_value))
1886
 
                self._mark_modified([entry])
 
1736
                entry[1][0] = ('f', sha1, entry[1][0][2], entry[1][0][3],
 
1737
                    packed_stat)
 
1738
                self._dirblock_state = DirState.IN_MEMORY_MODIFIED
1887
1739
 
1888
1740
    def _sha_cutoff_time(self):
1889
1741
        """Return cutoff time.
1933
1785
            # paths are produced by UnicodeDirReader on purpose.
1934
1786
            abspath = abspath.encode(fs_encoding)
1935
1787
        target = os.readlink(abspath)
1936
 
        if fs_encoding not in ('utf-8', 'ascii'):
 
1788
        if fs_encoding not in ('UTF-8', 'US-ASCII', 'ANSI_X3.4-1968'):
1937
1789
            # Change encoding if needed
1938
1790
            target = target.decode(fs_encoding).encode('UTF-8')
1939
1791
        return target
1947
1799
        """Serialise the entire dirstate to a sequence of lines."""
1948
1800
        if (self._header_state == DirState.IN_MEMORY_UNMODIFIED and
1949
1801
            self._dirblock_state == DirState.IN_MEMORY_UNMODIFIED):
1950
 
            # read what's on disk.
 
1802
            # read whats on disk.
1951
1803
            self._state_file.seek(0)
1952
1804
            return self._state_file.readlines()
1953
1805
        lines = []
1954
1806
        lines.append(self._get_parents_line(self.get_parent_ids()))
1955
1807
        lines.append(self._get_ghosts_line(self._ghosts))
1956
 
        lines.extend(self._get_entry_lines())
 
1808
        # append the root line which is special cased
 
1809
        lines.extend(map(self._entry_to_line, self._iter_entries()))
1957
1810
        return self._get_output_lines(lines)
1958
1811
 
1959
1812
    def _get_ghosts_line(self, ghost_ids):
1964
1817
        """Create a line for the state file for parents information."""
1965
1818
        return '\0'.join([str(len(parent_ids))] + parent_ids)
1966
1819
 
1967
 
    def _get_entry_lines(self):
1968
 
        """Create lines for entries."""
1969
 
        return map(self._entry_to_line, self._iter_entries())
1970
 
 
1971
1820
    def _get_fields_to_entry(self):
1972
1821
        """Get a function which converts entry fields into a entry record.
1973
1822
 
2091
1940
            entry_index += 1
2092
1941
        return block_index, entry_index, True, False
2093
1942
 
2094
 
    def _get_entry(self, tree_index, fileid_utf8=None, path_utf8=None,
2095
 
                   include_deleted=False):
 
1943
    def _get_entry(self, tree_index, fileid_utf8=None, path_utf8=None, include_deleted=False):
2096
1944
        """Get the dirstate entry for path in tree tree_index.
2097
1945
 
2098
1946
        If either file_id or path is supplied, it is used as the key to lookup.
2132
1980
                                          ' tree_index, file_id and path')
2133
1981
            return entry
2134
1982
        else:
2135
 
            possible_keys = self._get_id_index().get(fileid_utf8, ())
 
1983
            possible_keys = self._get_id_index().get(fileid_utf8, None)
2136
1984
            if not possible_keys:
2137
1985
                return None, None
2138
1986
            for key in possible_keys:
2240
2088
            executable = False
2241
2089
        else:
2242
2090
            raise Exception("can't pack %s" % inv_entry)
2243
 
        return static_tuple.StaticTuple(minikind, fingerprint, size,
2244
 
                                        executable, tree_data)
 
2091
        return (minikind, fingerprint, size, executable, tree_data)
2245
2092
 
2246
2093
    def _iter_child_entries(self, tree_index, path_utf8):
2247
2094
        """Iterate over all the entries that are children of path_utf.
2296
2143
                yield entry
2297
2144
 
2298
2145
    def _get_id_index(self):
2299
 
        """Get an id index of self._dirblocks.
2300
 
 
2301
 
        This maps from file_id => [(directory, name, file_id)] entries where
2302
 
        that file_id appears in one of the trees.
2303
 
        """
 
2146
        """Get an id index of self._dirblocks."""
2304
2147
        if self._id_index is None:
2305
2148
            id_index = {}
2306
2149
            for key, tree_details in self._iter_entries():
2307
 
                self._add_to_id_index(id_index, key)
 
2150
                id_index.setdefault(key[2], set()).add(key)
2308
2151
            self._id_index = id_index
2309
2152
        return self._id_index
2310
2153
 
2311
 
    def _add_to_id_index(self, id_index, entry_key):
2312
 
        """Add this entry to the _id_index mapping."""
2313
 
        # This code used to use a set for every entry in the id_index. However,
2314
 
        # it is *rare* to have more than one entry. So a set is a large
2315
 
        # overkill. And even when we do, we won't ever have more than the
2316
 
        # number of parent trees. Which is still a small number (rarely >2). As
2317
 
        # such, we use a simple tuple, and do our own uniqueness checks. While
2318
 
        # the 'in' check is O(N) since N is nicely bounded it shouldn't ever
2319
 
        # cause quadratic failure.
2320
 
        file_id = entry_key[2]
2321
 
        entry_key = static_tuple.StaticTuple.from_sequence(entry_key)
2322
 
        if file_id not in id_index:
2323
 
            id_index[file_id] = static_tuple.StaticTuple(entry_key,)
2324
 
        else:
2325
 
            entry_keys = id_index[file_id]
2326
 
            if entry_key not in entry_keys:
2327
 
                id_index[file_id] = entry_keys + (entry_key,)
2328
 
 
2329
 
    def _remove_from_id_index(self, id_index, entry_key):
2330
 
        """Remove this entry from the _id_index mapping.
2331
 
 
2332
 
        It is an programming error to call this when the entry_key is not
2333
 
        already present.
2334
 
        """
2335
 
        file_id = entry_key[2]
2336
 
        entry_keys = list(id_index[file_id])
2337
 
        entry_keys.remove(entry_key)
2338
 
        id_index[file_id] = static_tuple.StaticTuple.from_sequence(entry_keys)
2339
 
 
2340
2154
    def _get_output_lines(self, lines):
2341
2155
        """Format lines for final output.
2342
2156
 
2362
2176
        """The number of parent entries in each record row."""
2363
2177
        return len(self._parents) - len(self._ghosts)
2364
2178
 
2365
 
    @classmethod
2366
 
    def on_file(cls, path, sha1_provider=None, worth_saving_limit=0):
 
2179
    @staticmethod
 
2180
    def on_file(path, sha1_provider=None):
2367
2181
        """Construct a DirState on the file at path "path".
2368
2182
 
2369
2183
        :param path: The path at which the dirstate file on disk should live.
2370
2184
        :param sha1_provider: an object meeting the SHA1Provider interface.
2371
2185
            If None, a DefaultSHA1Provider is used.
2372
 
        :param worth_saving_limit: when the exact number of hash changed
2373
 
            entries is known, only bother saving the dirstate if more than
2374
 
            this count of entries have changed. -1 means never save.
2375
2186
        :return: An unlocked DirState object, associated with the given path.
2376
2187
        """
2377
2188
        if sha1_provider is None:
2378
2189
            sha1_provider = DefaultSHA1Provider()
2379
 
        result = cls(path, sha1_provider,
2380
 
                     worth_saving_limit=worth_saving_limit)
 
2190
        result = DirState(path, sha1_provider)
2381
2191
        return result
2382
2192
 
2383
2193
    def _read_dirblocks_if_needed(self):
2441
2251
            raise errors.BzrError('missing num_entries line')
2442
2252
        self._num_entries = int(num_entries_line[len('num_entries: '):-1])
2443
2253
 
2444
 
    def sha1_from_stat(self, path, stat_result):
 
2254
    def sha1_from_stat(self, path, stat_result, _pack_stat=pack_stat):
2445
2255
        """Find a sha1 given a stat lookup."""
2446
 
        return self._get_packed_stat_index().get(pack_stat(stat_result), None)
 
2256
        return self._get_packed_stat_index().get(_pack_stat(stat_result), None)
2447
2257
 
2448
2258
    def _get_packed_stat_index(self):
2449
2259
        """Get a packed_stat index of self._dirblocks."""
2475
2285
            trace.mutter('Not saving DirState because '
2476
2286
                    '_changes_aborted is set.')
2477
2287
            return
2478
 
        # TODO: Since we now distinguish IN_MEMORY_MODIFIED from
2479
 
        #       IN_MEMORY_HASH_MODIFIED, we should only fail quietly if we fail
2480
 
        #       to save an IN_MEMORY_HASH_MODIFIED, and fail *noisily* if we
2481
 
        #       fail to save IN_MEMORY_MODIFIED
2482
 
        if not self._worth_saving():
2483
 
            return
 
2288
        if (self._header_state == DirState.IN_MEMORY_MODIFIED or
 
2289
            self._dirblock_state == DirState.IN_MEMORY_MODIFIED):
2484
2290
 
2485
 
        grabbed_write_lock = False
2486
 
        if self._lock_state != 'w':
2487
 
            grabbed_write_lock, new_lock = self._lock_token.temporary_write_lock()
2488
 
            # Switch over to the new lock, as the old one may be closed.
2489
 
            # TODO: jam 20070315 We should validate the disk file has
2490
 
            #       not changed contents, since temporary_write_lock may
2491
 
            #       not be an atomic operation.
2492
 
            self._lock_token = new_lock
2493
 
            self._state_file = new_lock.f
2494
 
            if not grabbed_write_lock:
2495
 
                # We couldn't grab a write lock, so we switch back to a read one
2496
 
                return
2497
 
        try:
2498
 
            lines = self.get_lines()
2499
 
            self._state_file.seek(0)
2500
 
            self._state_file.writelines(lines)
2501
 
            self._state_file.truncate()
2502
 
            self._state_file.flush()
2503
 
            self._maybe_fdatasync()
2504
 
            self._mark_unmodified()
2505
 
        finally:
2506
 
            if grabbed_write_lock:
2507
 
                self._lock_token = self._lock_token.restore_read_lock()
2508
 
                self._state_file = self._lock_token.f
 
2291
            grabbed_write_lock = False
 
2292
            if self._lock_state != 'w':
 
2293
                grabbed_write_lock, new_lock = self._lock_token.temporary_write_lock()
 
2294
                # Switch over to the new lock, as the old one may be closed.
2509
2295
                # TODO: jam 20070315 We should validate the disk file has
2510
 
                #       not changed contents. Since restore_read_lock may
2511
 
                #       not be an atomic operation.                
2512
 
 
2513
 
    def _maybe_fdatasync(self):
2514
 
        """Flush to disk if possible and if not configured off."""
2515
 
        if self._config_stack.get('dirstate.fdatasync'):
2516
 
            osutils.fdatasync(self._state_file.fileno())
2517
 
 
2518
 
    def _worth_saving(self):
2519
 
        """Is it worth saving the dirstate or not?"""
2520
 
        if (self._header_state == DirState.IN_MEMORY_MODIFIED
2521
 
            or self._dirblock_state == DirState.IN_MEMORY_MODIFIED):
2522
 
            return True
2523
 
        if self._dirblock_state == DirState.IN_MEMORY_HASH_MODIFIED:
2524
 
            if self._worth_saving_limit == -1:
2525
 
                # We never save hash changes when the limit is -1
2526
 
                return False
2527
 
            # If we're using smart saving and only a small number of
2528
 
            # entries have changed their hash, don't bother saving. John has
2529
 
            # suggested using a heuristic here based on the size of the
2530
 
            # changed files and/or tree. For now, we go with a configurable
2531
 
            # number of changes, keeping the calculation time
2532
 
            # as low overhead as possible. (This also keeps all existing
2533
 
            # tests passing as the default is 0, i.e. always save.)
2534
 
            if len(self._known_hash_changes) >= self._worth_saving_limit:
2535
 
                return True
2536
 
        return False
 
2296
                #       not changed contents. Since temporary_write_lock may
 
2297
                #       not be an atomic operation.
 
2298
                self._lock_token = new_lock
 
2299
                self._state_file = new_lock.f
 
2300
                if not grabbed_write_lock:
 
2301
                    # We couldn't grab a write lock, so we switch back to a read one
 
2302
                    return
 
2303
            try:
 
2304
                self._state_file.seek(0)
 
2305
                self._state_file.writelines(self.get_lines())
 
2306
                self._state_file.truncate()
 
2307
                self._state_file.flush()
 
2308
                self._header_state = DirState.IN_MEMORY_UNMODIFIED
 
2309
                self._dirblock_state = DirState.IN_MEMORY_UNMODIFIED
 
2310
            finally:
 
2311
                if grabbed_write_lock:
 
2312
                    self._lock_token = self._lock_token.restore_read_lock()
 
2313
                    self._state_file = self._lock_token.f
 
2314
                    # TODO: jam 20070315 We should validate the disk file has
 
2315
                    #       not changed contents. Since restore_read_lock may
 
2316
                    #       not be an atomic operation.
2537
2317
 
2538
2318
    def _set_data(self, parent_ids, dirblocks):
2539
2319
        """Set the full dirstate data in memory.
2548
2328
        """
2549
2329
        # our memory copy is now authoritative.
2550
2330
        self._dirblocks = dirblocks
2551
 
        self._mark_modified(header_modified=True)
 
2331
        self._header_state = DirState.IN_MEMORY_MODIFIED
 
2332
        self._dirblock_state = DirState.IN_MEMORY_MODIFIED
2552
2333
        self._parents = list(parent_ids)
2553
2334
        self._id_index = None
2554
2335
        self._packed_stat_index = None
2574
2355
        self._make_absent(entry)
2575
2356
        self.update_minimal(('', '', new_id), 'd',
2576
2357
            path_utf8='', packed_stat=entry[1][0][4])
2577
 
        self._mark_modified()
 
2358
        self._dirblock_state = DirState.IN_MEMORY_MODIFIED
2578
2359
 
2579
2360
    def set_parent_trees(self, trees, ghosts):
2580
2361
        """Set the parent trees for the dirstate.
2625
2406
        parent_trees = [tree for rev_id, tree in trees if rev_id not in ghosts]
2626
2407
        # how many trees do we end up with
2627
2408
        parent_count = len(parent_trees)
2628
 
        st = static_tuple.StaticTuple
2629
2409
 
2630
2410
        # one: the current tree
2631
2411
        for entry in self._iter_entries():
2634
2414
                continue
2635
2415
            by_path[entry[0]] = [entry[1][0]] + \
2636
2416
                [DirState.NULL_PARENT_DETAILS] * parent_count
2637
 
            # TODO: Possibly inline this, since we know it isn't present yet
2638
 
            #       id_index[entry[0][2]] = (entry[0],)
2639
 
            self._add_to_id_index(id_index, entry[0])
 
2417
            id_index[entry[0][2]] = set([entry[0]])
2640
2418
 
2641
2419
        # now the parent trees:
2642
2420
        for tree_index, tree in enumerate(parent_trees):
2648
2426
            # the suffix is from tree_index+1:parent_count+1.
2649
2427
            new_location_suffix = [DirState.NULL_PARENT_DETAILS] * (parent_count - tree_index)
2650
2428
            # now stitch in all the entries from this tree
2651
 
            last_dirname = None
2652
 
            for path, entry in tree.iter_entries_by_dir():
 
2429
            for path, entry in tree.inventory.iter_entries_by_dir():
2653
2430
                # here we process each trees details for each item in the tree.
2654
2431
                # we first update any existing entries for the id at other paths,
2655
2432
                # then we either create or update the entry for the id at the
2662
2439
                file_id = entry.file_id
2663
2440
                path_utf8 = path.encode('utf8')
2664
2441
                dirname, basename = osutils.split(path_utf8)
2665
 
                if dirname == last_dirname:
2666
 
                    # Try to re-use objects as much as possible
2667
 
                    dirname = last_dirname
2668
 
                else:
2669
 
                    last_dirname = dirname
2670
 
                new_entry_key = st(dirname, basename, file_id)
 
2442
                new_entry_key = (dirname, basename, file_id)
2671
2443
                # tree index consistency: All other paths for this id in this tree
2672
2444
                # index must point to the correct path.
2673
 
                entry_keys = id_index.get(file_id, ())
2674
 
                for entry_key in entry_keys:
 
2445
                for entry_key in id_index.setdefault(file_id, set()):
2675
2446
                    # TODO:PROFILING: It might be faster to just update
2676
2447
                    # rather than checking if we need to, and then overwrite
2677
2448
                    # the one we are located at.
2680
2451
                        # other trees, so put absent pointers there
2681
2452
                        # This is the vertical axis in the matrix, all pointing
2682
2453
                        # to the real path.
2683
 
                        by_path[entry_key][tree_index] = st('r', path_utf8, 0,
2684
 
                                                            False, '')
2685
 
                # by path consistency: Insert into an existing path record
2686
 
                # (trivial), or add a new one with relocation pointers for the
2687
 
                # other tree indexes.
2688
 
                if new_entry_key in entry_keys:
2689
 
                    # there is already an entry where this data belongs, just
2690
 
                    # insert it.
 
2454
                        by_path[entry_key][tree_index] = ('r', path_utf8, 0, False, '')
 
2455
                # by path consistency: Insert into an existing path record (trivial), or
 
2456
                # add a new one with relocation pointers for the other tree indexes.
 
2457
                if new_entry_key in id_index[file_id]:
 
2458
                    # there is already an entry where this data belongs, just insert it.
2691
2459
                    by_path[new_entry_key][tree_index] = \
2692
2460
                        self._inv_entry_to_details(entry)
2693
2461
                else:
2697
2465
                    new_details = []
2698
2466
                    for lookup_index in xrange(tree_index):
2699
2467
                        # boundary case: this is the first occurence of file_id
2700
 
                        # so there are no id_indexes, possibly take this out of
 
2468
                        # so there are no id_indexs, possibly take this out of
2701
2469
                        # the loop?
2702
 
                        if not len(entry_keys):
 
2470
                        if not len(id_index[file_id]):
2703
2471
                            new_details.append(DirState.NULL_PARENT_DETAILS)
2704
2472
                        else:
2705
2473
                            # grab any one entry, use it to find the right path.
2706
 
                            a_key = iter(entry_keys).next()
 
2474
                            # TODO: optimise this to reduce memory use in highly
 
2475
                            # fragmented situations by reusing the relocation
 
2476
                            # records.
 
2477
                            a_key = iter(id_index[file_id]).next()
2707
2478
                            if by_path[a_key][lookup_index][0] in ('r', 'a'):
2708
 
                                # its a pointer or missing statement, use it as
2709
 
                                # is.
 
2479
                                # its a pointer or missing statement, use it as is.
2710
2480
                                new_details.append(by_path[a_key][lookup_index])
2711
2481
                            else:
2712
2482
                                # we have the right key, make a pointer to it.
2713
2483
                                real_path = ('/'.join(a_key[0:2])).strip('/')
2714
 
                                new_details.append(st('r', real_path, 0, False,
2715
 
                                                      ''))
 
2484
                                new_details.append(('r', real_path, 0, False, ''))
2716
2485
                    new_details.append(self._inv_entry_to_details(entry))
2717
2486
                    new_details.extend(new_location_suffix)
2718
2487
                    by_path[new_entry_key] = new_details
2719
 
                    self._add_to_id_index(id_index, new_entry_key)
 
2488
                    id_index[file_id].add(new_entry_key)
2720
2489
        # --- end generation of full tree mappings
2721
2490
 
2722
2491
        # sort and output all the entries
2724
2493
        self._entries_to_current_state(new_entries)
2725
2494
        self._parents = [rev_id for rev_id, tree in trees]
2726
2495
        self._ghosts = list(ghosts)
2727
 
        self._mark_modified(header_modified=True)
 
2496
        self._header_state = DirState.IN_MEMORY_MODIFIED
 
2497
        self._dirblock_state = DirState.IN_MEMORY_MODIFIED
2728
2498
        self._id_index = id_index
2729
2499
 
2730
2500
    def _sort_entries(self, entry_list):
2734
2504
        try to keep everything in sorted blocks all the time, but sometimes
2735
2505
        it's easier to sort after the fact.
2736
2506
        """
2737
 
        # When sorting, we usually have 10x more entries than directories. (69k
2738
 
        # total entries, 4k directories). So cache the results of splitting.
2739
 
        # Saving time and objects. Also, use StaticTuple to avoid putting all
2740
 
        # of these object into python's garbage collector.
2741
 
        split_dirs = {}
2742
 
        def _key(entry, _split_dirs=split_dirs, _st=static_tuple.StaticTuple):
 
2507
        def _key(entry):
2743
2508
            # sort by: directory parts, file name, file id
2744
 
            dirpath, fname, file_id = entry[0]
2745
 
            try:
2746
 
                split = _split_dirs[dirpath]
2747
 
            except KeyError:
2748
 
                split = _st.from_sequence(dirpath.split('/'))
2749
 
                _split_dirs[dirpath] = split
2750
 
            return _st(split, fname, file_id)
 
2509
            return entry[0][0].split('/'), entry[0][1], entry[0][2]
2751
2510
        return sorted(entry_list, key=_key)
2752
2511
 
2753
2512
    def set_state_from_inventory(self, new_inv):
2878
2637
                        current_old[0][1].decode('utf8'))
2879
2638
                self._make_absent(current_old)
2880
2639
                current_old = advance(old_iterator)
2881
 
        self._mark_modified()
 
2640
        self._dirblock_state = DirState.IN_MEMORY_MODIFIED
2882
2641
        self._id_index = None
2883
2642
        self._packed_stat_index = None
2884
2643
        if tracing:
2885
2644
            trace.mutter("set_state_from_inventory complete.")
2886
2645
 
2887
 
    def set_state_from_scratch(self, working_inv, parent_trees, parent_ghosts):
2888
 
        """Wipe the currently stored state and set it to something new.
2889
 
 
2890
 
        This is a hard-reset for the data we are working with.
2891
 
        """
2892
 
        # Technically, we really want a write lock, but until we write, we
2893
 
        # don't really need it.
2894
 
        self._requires_lock()
2895
 
        # root dir and root dir contents with no children. We have to have a
2896
 
        # root for set_state_from_inventory to work correctly.
2897
 
        empty_root = (('', '', inventory.ROOT_ID),
2898
 
                      [('d', '', 0, False, DirState.NULLSTAT)])
2899
 
        empty_tree_dirblocks = [('', [empty_root]), ('', [])]
2900
 
        self._set_data([], empty_tree_dirblocks)
2901
 
        self.set_state_from_inventory(working_inv)
2902
 
        self.set_parent_trees(parent_trees, parent_ghosts)
2903
 
 
2904
2646
    def _make_absent(self, current_old):
2905
2647
        """Mark current_old - an entry - as absent for tree 0.
2906
2648
 
2931
2673
            block[1].pop(entry_index)
2932
2674
            # if we have an id_index in use, remove this key from it for this id.
2933
2675
            if self._id_index is not None:
2934
 
                self._remove_from_id_index(self._id_index, current_old[0])
 
2676
                self._id_index[current_old[0][2]].remove(current_old[0])
2935
2677
        # update all remaining keys for this id to record it as absent. The
2936
2678
        # existing details may either be the record we are marking as deleted
2937
2679
        # (if there were other trees with the id present at this path), or may
2950
2692
            if update_tree_details[0][0] == 'a': # absent
2951
2693
                raise AssertionError('bad row %r' % (update_tree_details,))
2952
2694
            update_tree_details[0] = DirState.NULL_PARENT_DETAILS
2953
 
        self._mark_modified()
 
2695
        self._dirblock_state = DirState.IN_MEMORY_MODIFIED
2954
2696
        return last_reference
2955
2697
 
2956
2698
    def update_minimal(self, key, minikind, executable=False, fingerprint='',
2998
2740
                            # This entry has the same path (but a different id) as
2999
2741
                            # the new entry we're adding, and is present in ths
3000
2742
                            # tree.
3001
 
                            self._raise_invalid(
 
2743
                            raise errors.InconsistentDelta(
3002
2744
                                ("%s/%s" % key[0:2]).decode('utf8'), key[2],
3003
2745
                                "Attempt to add item at path already occupied by "
3004
2746
                                "id %r" % entry[0][2])
3006
2748
                    else:
3007
2749
                        break
3008
2750
            # new entry, synthesis cross reference here,
3009
 
            existing_keys = id_index.get(key[2], ())
 
2751
            existing_keys = id_index.setdefault(key[2], set())
3010
2752
            if not existing_keys:
3011
2753
                # not currently in the state, simplest case
3012
2754
                new_entry = key, [new_details] + self._empty_parent_info()
3043
2785
                    # loop.
3044
2786
                    other_entry = other_block[other_entry_index]
3045
2787
                    other_entry[1][0] = ('r', path_utf8, 0, False, '')
3046
 
                    if self._maybe_remove_row(other_block, other_entry_index,
3047
 
                                              id_index):
3048
 
                        # If the row holding this was removed, we need to
3049
 
                        # recompute where this entry goes
3050
 
                        entry_index, _ = self._find_entry_index(key, block)
 
2788
                    self._maybe_remove_row(other_block, other_entry_index,
 
2789
                        id_index)
3051
2790
 
3052
2791
                # This loop:
3053
2792
                # adds a tuple to the new details for each column
3055
2794
                #  - or by creating a new pointer to the right row inside that column
3056
2795
                num_present_parents = self._num_present_parents()
3057
2796
                if num_present_parents:
3058
 
                    # TODO: This re-evaluates the existing_keys set, do we need
3059
 
                    #       to do that ourselves?
3060
2797
                    other_key = list(existing_keys)[0]
3061
2798
                for lookup_index in xrange(1, num_present_parents + 1):
3062
2799
                    # grab any one entry, use it to find the right path.
3081
2818
                        pointer_path = osutils.pathjoin(*other_key[0:2])
3082
2819
                        new_entry[1].append(('r', pointer_path, 0, False, ''))
3083
2820
            block.insert(entry_index, new_entry)
3084
 
            self._add_to_id_index(id_index, key)
 
2821
            existing_keys.add(key)
3085
2822
        else:
3086
2823
            # Does the new state matter?
3087
2824
            block[entry_index][1][0] = new_details
3096
2833
            # converted to relocated.
3097
2834
            if path_utf8 is None:
3098
2835
                raise AssertionError('no path')
3099
 
            existing_keys = id_index.get(key[2], ())
3100
 
            if key not in existing_keys:
3101
 
                raise AssertionError('We found the entry in the blocks, but'
3102
 
                    ' the key is not in the id_index.'
3103
 
                    ' key: %s, existing_keys: %s' % (key, existing_keys))
3104
 
            for entry_key in existing_keys:
 
2836
            for entry_key in id_index.setdefault(key[2], set()):
3105
2837
                # TODO:PROFILING: It might be faster to just update
3106
2838
                # rather than checking if we need to, and then overwrite
3107
2839
                # the one we are located at.
3125
2857
            if not present:
3126
2858
                self._dirblocks.insert(block_index, (subdir_key[0], []))
3127
2859
 
3128
 
        self._mark_modified()
 
2860
        self._dirblock_state = DirState.IN_MEMORY_MODIFIED
3129
2861
 
3130
2862
    def _maybe_remove_row(self, block, index, id_index):
3131
2863
        """Remove index if it is absent or relocated across the row.
3132
2864
        
3133
2865
        id_index is updated accordingly.
3134
 
        :return: True if we removed the row, False otherwise
3135
2866
        """
3136
2867
        present_in_row = False
3137
2868
        entry = block[index]
3141
2872
                break
3142
2873
        if not present_in_row:
3143
2874
            block.pop(index)
3144
 
            self._remove_from_id_index(id_index, entry[0])
3145
 
            return True
3146
 
        return False
 
2875
            id_index[entry[0][2]].remove(entry[0])
3147
2876
 
3148
2877
    def _validate(self):
3149
2878
        """Check that invariants on the dirblock are correct.
3287
3016
        if self._id_index is not None:
3288
3017
            for file_id, entry_keys in self._id_index.iteritems():
3289
3018
                for entry_key in entry_keys:
3290
 
                    # Check that the entry in the map is pointing to the same
3291
 
                    # file_id
3292
3019
                    if entry_key[2] != file_id:
3293
3020
                        raise AssertionError(
3294
3021
                            'file_id %r did not match entry key %s'
3295
3022
                            % (file_id, entry_key))
3296
 
                    # And that from this entry key, we can look up the original
3297
 
                    # record
3298
 
                    block_index, present = self._find_block_index_from_key(entry_key)
3299
 
                    if not present:
3300
 
                        raise AssertionError('missing block for entry key: %r', entry_key)
3301
 
                    entry_index, present = self._find_entry_index(entry_key, self._dirblocks[block_index][1])
3302
 
                    if not present:
3303
 
                        raise AssertionError('missing entry for key: %r', entry_key)
3304
 
                if len(entry_keys) != len(set(entry_keys)):
3305
 
                    raise AssertionError(
3306
 
                        'id_index contained non-unique data for %s'
3307
 
                        % (entry_keys,))
3308
3023
 
3309
3024
    def _wipe_state(self):
3310
3025
        """Forget all state information about the dirstate."""
3367
3082
 
3368
3083
 
3369
3084
def py_update_entry(state, entry, abspath, stat_value,
3370
 
                 _stat_to_minikind=DirState._stat_to_minikind):
 
3085
                 _stat_to_minikind=DirState._stat_to_minikind,
 
3086
                 _pack_stat=pack_stat):
3371
3087
    """Update the entry based on what is actually on disk.
3372
3088
 
3373
3089
    This function only calculates the sha if it needs to - if the entry is
3386
3102
    except KeyError:
3387
3103
        # Unhandled kind
3388
3104
        return None
3389
 
    packed_stat = pack_stat(stat_value)
 
3105
    packed_stat = _pack_stat(stat_value)
3390
3106
    (saved_minikind, saved_link_or_sha1, saved_file_size,
3391
3107
     saved_executable, saved_packed_stat) = entry[1][0]
3392
3108
 
3406
3122
    # If we have gotten this far, that means that we need to actually
3407
3123
    # process this entry.
3408
3124
    link_or_sha1 = None
3409
 
    worth_saving = True
3410
3125
    if minikind == 'f':
3411
3126
        executable = state._is_executable(stat_value.st_mode,
3412
3127
                                         saved_executable)
3428
3143
        else:
3429
3144
            entry[1][0] = ('f', '', stat_value.st_size,
3430
3145
                           executable, DirState.NULLSTAT)
3431
 
            worth_saving = False
3432
3146
    elif minikind == 'd':
3433
3147
        link_or_sha1 = None
3434
3148
        entry[1][0] = ('d', '', 0, False, packed_stat)
3440
3154
                state._get_block_entry_index(entry[0][0], entry[0][1], 0)
3441
3155
            state._ensure_block(block_index, entry_index,
3442
3156
                               osutils.pathjoin(entry[0][0], entry[0][1]))
3443
 
        else:
3444
 
            worth_saving = False
3445
3157
    elif minikind == 'l':
3446
 
        if saved_minikind == 'l':
3447
 
            worth_saving = False
3448
3158
        link_or_sha1 = state._read_link(abspath, saved_link_or_sha1)
3449
3159
        if state._cutoff_time is None:
3450
3160
            state._sha_cutoff_time()
3455
3165
        else:
3456
3166
            entry[1][0] = ('l', '', stat_value.st_size,
3457
3167
                           False, DirState.NULLSTAT)
3458
 
    if worth_saving:
3459
 
        state._mark_modified([entry])
 
3168
    state._dirblock_state = DirState.IN_MEMORY_MODIFIED
3460
3169
    return link_or_sha1
3461
3170
 
3462
3171
 
3771
3480
            raise AssertionError("don't know how to compare "
3772
3481
                "source_minikind=%r, target_minikind=%r"
3773
3482
                % (source_minikind, target_minikind))
 
3483
            ## import pdb;pdb.set_trace()
3774
3484
        return None, None
3775
3485
 
3776
3486
    def __iter__(self):
4265
3975
        _bisect_path_left,
4266
3976
        _bisect_path_right,
4267
3977
        cmp_by_dirs,
4268
 
        pack_stat,
4269
3978
        ProcessEntryC as _process_entry,
4270
3979
        update_entry as update_entry,
4271
3980
        )
4277
3986
        _bisect_path_left,
4278
3987
        _bisect_path_right,
4279
3988
        cmp_by_dirs,
4280
 
        pack_stat,
4281
3989
        )
4282
3990
    # FIXME: It would be nice to be able to track moved lines so that the
4283
3991
    # corresponding python code can be moved to the _dirstate_helpers_py