/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/xml8.py

  • Committer: Robert Collins
  • Date: 2010-05-06 11:08:10 UTC
  • mto: This revision was merged to the branch mainline in revision 5223.
  • Revision ID: robertc@robertcollins.net-20100506110810-h3j07fh5gmw54s25
Cleaner matcher matching revised unlocking protocol.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005, 2006 Canonical Ltd
 
1
# Copyright (C) 2005-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
12
12
#
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
16
16
 
17
17
import cStringIO
18
18
import re
19
19
 
20
20
from bzrlib import (
21
21
    cache_utf8,
 
22
    errors,
22
23
    inventory,
23
 
    )
24
 
from bzrlib.xml_serializer import SubElement, Element, Serializer
25
 
from bzrlib.inventory import ROOT_ID, Inventory, InventoryEntry
 
24
    revision as _mod_revision,
 
25
    trace,
 
26
    )
 
27
from bzrlib.xml_serializer import (
 
28
    Element,
 
29
    SubElement,
 
30
    XMLSerializer,
 
31
    escape_invalid_chars,
 
32
    )
 
33
from bzrlib.inventory import InventoryEntry
26
34
from bzrlib.revision import Revision
27
35
from bzrlib.errors import BzrError
28
36
 
29
37
 
30
38
_utf8_re = None
31
 
_utf8_escape_map = {
 
39
_unicode_re = None
 
40
_xml_escape_map = {
32
41
    "&":'&',
33
42
    "'":"'", # FIXME: overkill
34
43
    "\"":""",
38
47
 
39
48
 
40
49
def _ensure_utf8_re():
41
 
    """Make sure the _utf8_re regex has been compiled"""
42
 
    global _utf8_re
43
 
    if _utf8_re is not None:
44
 
        return
45
 
    _utf8_re = re.compile(u'[&<>\'\"\u0080-\uffff]')
46
 
 
47
 
 
48
 
def _utf8_escape_replace(match, _map=_utf8_escape_map):
 
50
    """Make sure the _utf8_re and _unicode_re regexes have been compiled."""
 
51
    global _utf8_re, _unicode_re
 
52
    if _utf8_re is None:
 
53
        _utf8_re = re.compile('[&<>\'\"]|[\x80-\xff]+')
 
54
    if _unicode_re is None:
 
55
        _unicode_re = re.compile(u'[&<>\'\"\u0080-\uffff]')
 
56
 
 
57
 
 
58
def _unicode_escape_replace(match, _map=_xml_escape_map):
49
59
    """Replace a string of non-ascii, non XML safe characters with their escape
50
60
 
51
61
    This will escape both Standard XML escapes, like <>"', etc.
64
74
        return "&#%d;" % ord(match.group())
65
75
 
66
76
 
67
 
_unicode_to_escaped_map = {}
68
 
 
69
 
def _encode_and_escape(unicode_str, _map=_unicode_to_escaped_map):
 
77
def _utf8_escape_replace(match, _map=_xml_escape_map):
 
78
    """Escape utf8 characters into XML safe ones.
 
79
 
 
80
    This uses 2 tricks. It is either escaping "standard" characters, like "&<>,
 
81
    or it is handling characters with the high-bit set. For ascii characters,
 
82
    we just lookup the replacement in the dictionary. For everything else, we
 
83
    decode back into Unicode, and then use the XML escape code.
 
84
    """
 
85
    try:
 
86
        return _map[match.group()]
 
87
    except KeyError:
 
88
        return ''.join('&#%d;' % ord(uni_chr)
 
89
                       for uni_chr in match.group().decode('utf8'))
 
90
 
 
91
 
 
92
_to_escaped_map = {}
 
93
 
 
94
def _encode_and_escape(unicode_or_utf8_str, _map=_to_escaped_map):
70
95
    """Encode the string into utf8, and escape invalid XML characters"""
71
96
    # We frequently get entities we have not seen before, so it is better
72
97
    # to check if None, rather than try/KeyError
73
 
    text = _map.get(unicode_str)
 
98
    text = _map.get(unicode_or_utf8_str)
74
99
    if text is None:
75
 
        # The alternative policy is to do a regular UTF8 encoding
76
 
        # and then escape only XML meta characters.
77
 
        # Performance is equivalent once you use cache_utf8. *However*
78
 
        # this makes the serialized texts incompatible with old versions
79
 
        # of bzr. So no net gain. (Perhaps the read code would handle utf8
80
 
        # better than entity escapes, but cElementTree seems to do just fine
81
 
        # either way)
82
 
        text = str(_utf8_re.sub(_utf8_escape_replace, unicode_str)) + '"'
83
 
        _map[unicode_str] = text
 
100
        if unicode_or_utf8_str.__class__ is unicode:
 
101
            # The alternative policy is to do a regular UTF8 encoding
 
102
            # and then escape only XML meta characters.
 
103
            # Performance is equivalent once you use cache_utf8. *However*
 
104
            # this makes the serialized texts incompatible with old versions
 
105
            # of bzr. So no net gain. (Perhaps the read code would handle utf8
 
106
            # better than entity escapes, but cElementTree seems to do just fine
 
107
            # either way)
 
108
            text = str(_unicode_re.sub(_unicode_escape_replace,
 
109
                                       unicode_or_utf8_str)) + '"'
 
110
        else:
 
111
            # Plain strings are considered to already be in utf-8 so we do a
 
112
            # slightly different method for escaping.
 
113
            text = _utf8_re.sub(_utf8_escape_replace,
 
114
                                unicode_or_utf8_str) + '"'
 
115
        _map[unicode_or_utf8_str] = text
84
116
    return text
85
117
 
86
118
 
 
119
def _get_utf8_or_ascii(a_str,
 
120
                       _encode_utf8=cache_utf8.encode,
 
121
                       _get_cached_ascii=cache_utf8.get_cached_ascii):
 
122
    """Return a cached version of the string.
 
123
 
 
124
    cElementTree will return a plain string if the XML is plain ascii. It only
 
125
    returns Unicode when it needs to. We want to work in utf-8 strings. So if
 
126
    cElementTree returns a plain string, we can just return the cached version.
 
127
    If it is Unicode, then we need to encode it.
 
128
 
 
129
    :param a_str: An 8-bit string or Unicode as returned by
 
130
                  cElementTree.Element.get()
 
131
    :return: A utf-8 encoded 8-bit string.
 
132
    """
 
133
    # This is fairly optimized because we know what cElementTree does, this is
 
134
    # not meant as a generic function for all cases. Because it is possible for
 
135
    # an 8-bit string to not be ascii or valid utf8.
 
136
    if a_str.__class__ is unicode:
 
137
        return _encode_utf8(a_str)
 
138
    else:
 
139
        return intern(a_str)
 
140
 
 
141
 
87
142
def _clear_cache():
88
143
    """Clean out the unicode => escaped map"""
89
 
    _unicode_to_escaped_map.clear()
90
 
 
91
 
 
92
 
class Serializer_v5(Serializer):
93
 
    """Version 5 serializer
94
 
 
95
 
    Packs objects into XML and vice versa.
 
144
    _to_escaped_map.clear()
 
145
 
 
146
 
 
147
class Serializer_v8(XMLSerializer):
 
148
    """This serialiser adds rich roots.
 
149
 
 
150
    Its revision format number matches its inventory number.
96
151
    """
97
 
    
 
152
 
98
153
    __slots__ = []
99
154
 
 
155
    root_id = None
100
156
    support_altered_by_hack = True
101
157
    # This format supports the altered-by hack that reads file ids directly out
102
158
    # of the versionedfile, without doing XML parsing.
103
159
 
104
 
    def write_inventory_to_string(self, inv):
105
 
        """Just call write_inventory with a StringIO and return the value"""
 
160
    supported_kinds = set(['file', 'directory', 'symlink'])
 
161
    format_num = '8'
 
162
    revision_format_num = None
 
163
 
 
164
    def _check_revisions(self, inv):
 
165
        """Extension point for subclasses to check during serialisation.
 
166
 
 
167
        :param inv: An inventory about to be serialised, to be checked.
 
168
        :raises: AssertionError if an error has occurred.
 
169
        """
 
170
        if inv.revision_id is None:
 
171
            raise AssertionError("inv.revision_id is None")
 
172
        if inv.root.revision is None:
 
173
            raise AssertionError("inv.root.revision is None")
 
174
 
 
175
    def _check_cache_size(self, inv_size, entry_cache):
 
176
        """Check that the entry_cache is large enough.
 
177
 
 
178
        We want the cache to be ~2x the size of an inventory. The reason is
 
179
        because we use a FIFO cache, and how Inventory records are likely to
 
180
        change. In general, you have a small number of records which change
 
181
        often, and a lot of records which do not change at all. So when the
 
182
        cache gets full, you actually flush out a lot of the records you are
 
183
        interested in, which means you need to recreate all of those records.
 
184
        An LRU Cache would be better, but the overhead negates the cache
 
185
        coherency benefit.
 
186
 
 
187
        One way to look at it, only the size of the cache > len(inv) is your
 
188
        'working' set. And in general, it shouldn't be a problem to hold 2
 
189
        inventories in memory anyway.
 
190
 
 
191
        :param inv_size: The number of entries in an inventory.
 
192
        """
 
193
        if entry_cache is None:
 
194
            return
 
195
        # 1.5 times might also be reasonable.
 
196
        recommended_min_cache_size = inv_size * 1.5
 
197
        if entry_cache.cache_size() < recommended_min_cache_size:
 
198
            recommended_cache_size = inv_size * 2
 
199
            trace.mutter('Resizing the inventory entry cache from %d to %d',
 
200
                         entry_cache.cache_size(), recommended_cache_size)
 
201
            entry_cache.resize(recommended_cache_size)
 
202
 
 
203
    def write_inventory_to_lines(self, inv):
 
204
        """Return a list of lines with the encoded inventory."""
 
205
        return self.write_inventory(inv, None)
 
206
 
 
207
    def write_inventory_to_string(self, inv, working=False):
 
208
        """Just call write_inventory with a StringIO and return the value.
 
209
 
 
210
        :param working: If True skip history data - text_sha1, text_size,
 
211
            reference_revision, symlink_target.
 
212
        """
106
213
        sio = cStringIO.StringIO()
107
 
        self.write_inventory(inv, sio)
 
214
        self.write_inventory(inv, sio, working)
108
215
        return sio.getvalue()
109
216
 
110
 
    def write_inventory(self, inv, f):
 
217
    def write_inventory(self, inv, f, working=False):
111
218
        """Write inventory to a file.
112
 
        
 
219
 
113
220
        :param inv: the inventory to write.
114
 
        :param f: the file to write.
 
221
        :param f: the file to write. (May be None if the lines are the desired
 
222
            output).
 
223
        :param working: If True skip history data - text_sha1, text_size,
 
224
            reference_revision, symlink_target.
 
225
        :return: The inventory as a list of lines.
115
226
        """
116
227
        _ensure_utf8_re()
 
228
        self._check_revisions(inv)
117
229
        output = []
118
230
        append = output.append
119
231
        self._append_inventory_root(append, inv)
121
233
        # Skip the root
122
234
        root_path, root_ie = entries.next()
123
235
        for path, ie in entries:
124
 
            self._append_entry(append, ie)
 
236
            if ie.parent_id != self.root_id:
 
237
                parent_str = ' parent_id="'
 
238
                parent_id  = _encode_and_escape(ie.parent_id)
 
239
            else:
 
240
                parent_str = ''
 
241
                parent_id  = ''
 
242
            if ie.kind == 'file':
 
243
                if ie.executable:
 
244
                    executable = ' executable="yes"'
 
245
                else:
 
246
                    executable = ''
 
247
                if not working:
 
248
                    append('<file%s file_id="%s name="%s%s%s revision="%s '
 
249
                        'text_sha1="%s" text_size="%d" />\n' % (
 
250
                        executable, _encode_and_escape(ie.file_id),
 
251
                        _encode_and_escape(ie.name), parent_str, parent_id,
 
252
                        _encode_and_escape(ie.revision), ie.text_sha1,
 
253
                        ie.text_size))
 
254
                else:
 
255
                    append('<file%s file_id="%s name="%s%s%s />\n' % (
 
256
                        executable, _encode_and_escape(ie.file_id),
 
257
                        _encode_and_escape(ie.name), parent_str, parent_id))
 
258
            elif ie.kind == 'directory':
 
259
                if not working:
 
260
                    append('<directory file_id="%s name="%s%s%s revision="%s '
 
261
                        '/>\n' % (
 
262
                        _encode_and_escape(ie.file_id),
 
263
                        _encode_and_escape(ie.name),
 
264
                        parent_str, parent_id,
 
265
                        _encode_and_escape(ie.revision)))
 
266
                else:
 
267
                    append('<directory file_id="%s name="%s%s%s />\n' % (
 
268
                        _encode_and_escape(ie.file_id),
 
269
                        _encode_and_escape(ie.name),
 
270
                        parent_str, parent_id))
 
271
            elif ie.kind == 'symlink':
 
272
                if not working:
 
273
                    append('<symlink file_id="%s name="%s%s%s revision="%s '
 
274
                        'symlink_target="%s />\n' % (
 
275
                        _encode_and_escape(ie.file_id),
 
276
                        _encode_and_escape(ie.name),
 
277
                        parent_str, parent_id,
 
278
                        _encode_and_escape(ie.revision),
 
279
                        _encode_and_escape(ie.symlink_target)))
 
280
                else:
 
281
                    append('<symlink file_id="%s name="%s%s%s />\n' % (
 
282
                        _encode_and_escape(ie.file_id),
 
283
                        _encode_and_escape(ie.name),
 
284
                        parent_str, parent_id))
 
285
            elif ie.kind == 'tree-reference':
 
286
                if ie.kind not in self.supported_kinds:
 
287
                    raise errors.UnsupportedInventoryKind(ie.kind)
 
288
                if not working:
 
289
                    append('<tree-reference file_id="%s name="%s%s%s '
 
290
                        'revision="%s reference_revision="%s />\n' % (
 
291
                        _encode_and_escape(ie.file_id),
 
292
                        _encode_and_escape(ie.name),
 
293
                        parent_str, parent_id,
 
294
                        _encode_and_escape(ie.revision),
 
295
                        _encode_and_escape(ie.reference_revision)))
 
296
                else:
 
297
                    append('<tree-reference file_id="%s name="%s%s%s />\n' % (
 
298
                        _encode_and_escape(ie.file_id),
 
299
                        _encode_and_escape(ie.name),
 
300
                        parent_str, parent_id))
 
301
            else:
 
302
                raise errors.UnsupportedInventoryKind(ie.kind)
125
303
        append('</inventory>\n')
126
 
        f.writelines(output)
 
304
        if f is not None:
 
305
            f.writelines(output)
127
306
        # Just to keep the cache from growing without bounds
128
307
        # but we may actually not want to do clear the cache
129
308
        #_clear_cache()
 
309
        return output
130
310
 
131
311
    def _append_inventory_root(self, append, inv):
132
312
        """Append the inventory root to output."""
133
 
        append('<inventory')
134
 
        if inv.root.file_id not in (None, ROOT_ID):
135
 
            append(' file_id="')
136
 
            append(_encode_and_escape(inv.root.file_id))
137
 
        append(' format="5"')
138
313
        if inv.revision_id is not None:
139
 
            append(' revision_id="')
140
 
            append(_encode_and_escape(inv.revision_id))
141
 
        append('>\n')
142
 
        
143
 
    def _append_entry(self, append, ie):
144
 
        """Convert InventoryEntry to XML element and append to output."""
145
 
        # TODO: should just be a plain assertion
146
 
        assert InventoryEntry.versionable_kind(ie.kind), \
147
 
            'unsupported entry kind %s' % ie.kind
148
 
 
149
 
        append("<")
150
 
        append(ie.kind)
151
 
        if ie.executable:
152
 
            append(' executable="yes"')
153
 
        append(' file_id="')
154
 
        append(_encode_and_escape(ie.file_id))
155
 
        append(' name="')
156
 
        append(_encode_and_escape(ie.name))
157
 
        if self._parent_condition(ie):
158
 
            assert isinstance(ie.parent_id, basestring)
159
 
            append(' parent_id="')
160
 
            append(_encode_and_escape(ie.parent_id))
161
 
        if ie.revision is not None:
162
 
            append(' revision="')
163
 
            append(_encode_and_escape(ie.revision))
164
 
        if ie.symlink_target is not None:
165
 
            append(' symlink_target="')
166
 
            append(_encode_and_escape(ie.symlink_target))
167
 
        if ie.text_sha1 is not None:
168
 
            append(' text_sha1="')
169
 
            append(ie.text_sha1)
170
 
            append('"')
171
 
        if ie.text_size is not None:
172
 
            append(' text_size="%d"' % ie.text_size)
173
 
        append(" />\n")
174
 
        return
175
 
 
176
 
    def _parent_condition(self, ie):
177
 
        return ie.parent_id != ROOT_ID
 
314
            revid1 = ' revision_id="'
 
315
            revid2 = _encode_and_escape(inv.revision_id)
 
316
        else:
 
317
            revid1 = ""
 
318
            revid2 = ""
 
319
        append('<inventory format="%s"%s%s>\n' % (
 
320
            self.format_num, revid1, revid2))
 
321
        append('<directory file_id="%s name="%s revision="%s />\n' % (
 
322
            _encode_and_escape(inv.root.file_id),
 
323
            _encode_and_escape(inv.root.name),
 
324
            _encode_and_escape(inv.root.revision)))
178
325
 
179
326
    def _pack_revision(self, rev):
180
327
        """Revision object -> xml tree"""
 
328
        # For the XML format, we need to write them as Unicode rather than as
 
329
        # utf-8 strings. So that cElementTree can handle properly escaping
 
330
        # them.
 
331
        decode_utf8 = cache_utf8.decode
 
332
        revision_id = rev.revision_id
 
333
        if isinstance(revision_id, str):
 
334
            revision_id = decode_utf8(revision_id)
 
335
        format_num = self.format_num
 
336
        if self.revision_format_num is not None:
 
337
            format_num = self.revision_format_num
181
338
        root = Element('revision',
182
339
                       committer = rev.committer,
183
 
                       timestamp = '%.9f' % rev.timestamp,
184
 
                       revision_id = rev.revision_id,
 
340
                       timestamp = '%.3f' % rev.timestamp,
 
341
                       revision_id = revision_id,
185
342
                       inventory_sha1 = rev.inventory_sha1,
186
 
                       format='5',
 
343
                       format=format_num,
187
344
                       )
188
345
        if rev.timezone is not None:
189
346
            root.set('timezone', str(rev.timezone))
190
347
        root.text = '\n'
191
348
        msg = SubElement(root, 'message')
192
 
        msg.text = rev.message
 
349
        msg.text = escape_invalid_chars(rev.message)[0]
193
350
        msg.tail = '\n'
194
351
        if rev.parent_ids:
195
352
            pelts = SubElement(root, 'parents')
196
353
            pelts.tail = pelts.text = '\n'
197
354
            for parent_id in rev.parent_ids:
198
 
                assert isinstance(parent_id, basestring)
 
355
                _mod_revision.check_not_reserved_id(parent_id)
199
356
                p = SubElement(pelts, 'revision_ref')
200
357
                p.tail = '\n'
 
358
                if isinstance(parent_id, str):
 
359
                    parent_id = decode_utf8(parent_id)
201
360
                p.set('revision_id', parent_id)
202
361
        if rev.properties:
203
362
            self._pack_revision_properties(rev, root)
206
365
    def _pack_revision_properties(self, rev, under_element):
207
366
        top_elt = SubElement(under_element, 'properties')
208
367
        for prop_name, prop_value in sorted(rev.properties.items()):
209
 
            assert isinstance(prop_name, basestring) 
210
 
            assert isinstance(prop_value, basestring) 
211
368
            prop_elt = SubElement(top_elt, 'property')
212
369
            prop_elt.set('name', prop_name)
213
370
            prop_elt.text = prop_value
214
371
            prop_elt.tail = '\n'
215
372
        top_elt.tail = '\n'
216
373
 
217
 
    def _unpack_inventory(self, elt):
218
 
        """Construct from XML Element
219
 
        """
220
 
        assert elt.tag == 'inventory'
221
 
        root_id = elt.get('file_id') or ROOT_ID
 
374
    def _unpack_inventory(self, elt, revision_id=None, entry_cache=None,
 
375
                          return_from_cache=False):
 
376
        """Construct from XML Element"""
 
377
        if elt.tag != 'inventory':
 
378
            raise errors.UnexpectedInventoryFormat('Root tag is %r' % elt.tag)
222
379
        format = elt.get('format')
223
 
        if format is not None:
224
 
            if format != '5':
225
 
                raise BzrError("invalid format version %r on inventory"
226
 
                                % format)
 
380
        if format != self.format_num:
 
381
            raise errors.UnexpectedInventoryFormat('Invalid format version %r'
 
382
                                                   % format)
227
383
        revision_id = elt.get('revision_id')
228
384
        if revision_id is not None:
229
 
            revision_id = cache_utf8.get_cached_unicode(revision_id)
230
 
        inv = Inventory(root_id, revision_id=revision_id)
 
385
            revision_id = cache_utf8.encode(revision_id)
 
386
        inv = inventory.Inventory(root_id=None, revision_id=revision_id)
231
387
        for e in elt:
232
 
            ie = self._unpack_entry(e)
233
 
            if ie.parent_id == ROOT_ID:
234
 
                ie.parent_id = root_id
 
388
            ie = self._unpack_entry(e, entry_cache=entry_cache,
 
389
                                    return_from_cache=return_from_cache)
235
390
            inv.add(ie)
 
391
        self._check_cache_size(len(inv), entry_cache)
236
392
        return inv
237
393
 
238
 
    def _unpack_entry(self, elt, none_parents=False):
 
394
    def _unpack_entry(self, elt, entry_cache=None, return_from_cache=False):
 
395
        elt_get = elt.get
 
396
        file_id = elt_get('file_id')
 
397
        revision = elt_get('revision')
 
398
        # Check and see if we have already unpacked this exact entry
 
399
        # Some timings for "repo.revision_trees(last_100_revs)"
 
400
        #               bzr     mysql
 
401
        #   unmodified  4.1s    40.8s
 
402
        #   using lru   3.5s
 
403
        #   using fifo  2.83s   29.1s
 
404
        #   lru._cache  2.8s
 
405
        #   dict        2.75s   26.8s
 
406
        #   inv.add     2.5s    26.0s
 
407
        #   no_copy     2.00s   20.5s
 
408
        #   no_c,dict   1.95s   18.0s
 
409
        # Note that a cache of 10k nodes is more than sufficient to hold all of
 
410
        # the inventory for the last 100 revs for bzr, but not for mysql (20k
 
411
        # is enough for mysql, which saves the same 2s as using a dict)
 
412
 
 
413
        # Breakdown of mysql using time.clock()
 
414
        #   4.1s    2 calls to element.get for file_id, revision_id
 
415
        #   4.5s    cache_hit lookup
 
416
        #   7.1s    InventoryFile.copy()
 
417
        #   2.4s    InventoryDirectory.copy()
 
418
        #   0.4s    decoding unique entries
 
419
        #   1.6s    decoding entries after FIFO fills up
 
420
        #   0.8s    Adding nodes to FIFO (including flushes)
 
421
        #   0.1s    cache miss lookups
 
422
        # Using an LRU cache
 
423
        #   4.1s    2 calls to element.get for file_id, revision_id
 
424
        #   9.9s    cache_hit lookup
 
425
        #   10.8s   InventoryEntry.copy()
 
426
        #   0.3s    cache miss lookus
 
427
        #   1.2s    decoding entries
 
428
        #   1.0s    adding nodes to LRU
 
429
        if entry_cache is not None and revision is not None:
 
430
            key = (file_id, revision)
 
431
            try:
 
432
                # We copy it, because some operations may mutate it
 
433
                cached_ie = entry_cache[key]
 
434
            except KeyError:
 
435
                pass
 
436
            else:
 
437
                # Only copying directory entries drops us 2.85s => 2.35s
 
438
                if return_from_cache:
 
439
                    if cached_ie.kind == 'directory':
 
440
                        return cached_ie.copy()
 
441
                    return cached_ie
 
442
                return cached_ie.copy()
 
443
 
239
444
        kind = elt.tag
240
445
        if not InventoryEntry.versionable_kind(kind):
241
446
            raise AssertionError('unsupported entry kind %s' % kind)
242
447
 
243
 
        get_cached = cache_utf8.get_cached_unicode
 
448
        get_cached = _get_utf8_or_ascii
244
449
 
245
 
        parent_id = elt.get('parent_id')
246
 
        if parent_id is None and not none_parents:
247
 
            parent_id = ROOT_ID
248
 
        # TODO: jam 20060817 At present, caching file ids costs us too 
249
 
        #       much time. It slows down overall read performances from
250
 
        #       approx 500ms to 700ms. And doesn't improve future reads.
251
 
        #       it might be because revision ids and file ids are mixing.
252
 
        #       Consider caching *just* the file ids, for a limited period
253
 
        #       of time.
254
 
        #parent_id = get_cached(parent_id)
255
 
        #file_id = get_cached(elt.get('file_id'))
256
 
        file_id = elt.get('file_id')
 
450
        file_id = get_cached(file_id)
 
451
        if revision is not None:
 
452
            revision = get_cached(revision)
 
453
        parent_id = elt_get('parent_id')
 
454
        if parent_id is not None:
 
455
            parent_id = get_cached(parent_id)
257
456
 
258
457
        if kind == 'directory':
259
458
            ie = inventory.InventoryDirectory(file_id,
260
 
                                              elt.get('name'),
 
459
                                              elt_get('name'),
261
460
                                              parent_id)
262
461
        elif kind == 'file':
263
462
            ie = inventory.InventoryFile(file_id,
264
 
                                         elt.get('name'),
 
463
                                         elt_get('name'),
265
464
                                         parent_id)
266
 
            ie.text_sha1 = elt.get('text_sha1')
267
 
            if elt.get('executable') == 'yes':
 
465
            ie.text_sha1 = elt_get('text_sha1')
 
466
            if elt_get('executable') == 'yes':
268
467
                ie.executable = True
269
 
            v = elt.get('text_size')
 
468
            v = elt_get('text_size')
270
469
            ie.text_size = v and int(v)
271
470
        elif kind == 'symlink':
272
471
            ie = inventory.InventoryLink(file_id,
273
 
                                         elt.get('name'),
 
472
                                         elt_get('name'),
274
473
                                         parent_id)
275
 
            ie.symlink_target = elt.get('symlink_target')
 
474
            ie.symlink_target = elt_get('symlink_target')
276
475
        else:
277
 
            raise BzrError("unknown kind %r" % kind)
278
 
        revision = elt.get('revision')
279
 
        if revision is not None:
280
 
            revision = get_cached(revision)
 
476
            raise errors.UnsupportedInventoryKind(kind)
281
477
        ie.revision = revision
 
478
        if revision is not None and entry_cache is not None:
 
479
            # We cache a copy() because callers like to mutate objects, and
 
480
            # that would cause the item in cache to mutate as well.
 
481
            # This has a small effect on many-inventory performance, because
 
482
            # the majority fraction is spent in cache hits, not misses.
 
483
            entry_cache[key] = ie.copy()
282
484
 
283
485
        return ie
284
486
 
285
487
    def _unpack_revision(self, elt):
286
488
        """XML Element -> Revision object"""
287
 
        assert elt.tag == 'revision'
288
489
        format = elt.get('format')
 
490
        format_num = self.format_num
 
491
        if self.revision_format_num is not None:
 
492
            format_num = self.revision_format_num
289
493
        if format is not None:
290
 
            if format != '5':
291
 
                raise BzrError("invalid format version %r on inventory"
 
494
            if format != format_num:
 
495
                raise BzrError("invalid format version %r on revision"
292
496
                                % format)
293
 
        get_cached = cache_utf8.get_cached_unicode
 
497
        get_cached = _get_utf8_or_ascii
294
498
        rev = Revision(committer = elt.get('committer'),
295
499
                       timestamp = float(elt.get('timestamp')),
296
500
                       revision_id = get_cached(elt.get('revision_id')),
298
502
                       )
299
503
        parents = elt.find('parents') or []
300
504
        for p in parents:
301
 
            assert p.tag == 'revision_ref', \
302
 
                   "bad parent node tag %r" % p.tag
303
505
            rev.parent_ids.append(get_cached(p.get('revision_id')))
304
506
        self._unpack_revision_properties(elt, rev)
305
507
        v = elt.get('timezone')
313
515
    def _unpack_revision_properties(self, elt, rev):
314
516
        """Unpack properties onto a revision."""
315
517
        props_elt = elt.find('properties')
316
 
        assert len(rev.properties) == 0
317
518
        if not props_elt:
318
519
            return
319
520
        for prop_elt in props_elt:
320
 
            assert prop_elt.tag == 'property', \
321
 
                "bad tag under properties list: %r" % prop_elt.tag
 
521
            if prop_elt.tag != 'property':
 
522
                raise AssertionError(
 
523
                    "bad tag under properties list: %r" % prop_elt.tag)
322
524
            name = prop_elt.get('name')
323
525
            value = prop_elt.text
324
526
            # If a property had an empty value ('') cElementTree reads
326
528
            # properties have string values
327
529
            if value is None:
328
530
                value = ''
329
 
            assert name not in rev.properties, \
330
 
                "repeated property %r" % name
 
531
            if name in rev.properties:
 
532
                raise AssertionError("repeated property %r" % name)
331
533
            rev.properties[name] = value
332
534
 
333
535
 
334
 
serializer_v5 = Serializer_v5()
 
536
serializer_v8 = Serializer_v8()