/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/weavefile.py

  • Committer: Vincent Ladeuil
  • Date: 2012-01-05 14:26:58 UTC
  • mto: This revision was merged to the branch mainline in revision 6468.
  • Revision ID: v.ladeuil+lp@free.fr-20120105142658-vek3v6pzlxb751s2
Tests passing for a first rough version of a cached branch config store. The changes here are too invasive and several parallel proposals have been made. 

@only_raises is evil and gave a hard time since any exception during
save_changes() was swallowed.

Possible improvements: 

- add some needs_write_lock decorators to crucial
  methods (_set_config_location ?) but keep locking the branch at higher levels

- decorate branch.unlock to call stack.save if last_lock() it True
  outside of @only_raises scope (evil decorator)

- add @needs_write_lock to stack.set and stack.remove (will probably get
  rid of most testing issues) we probably need a specialized decorator
  that can relay to the store and from there to the branch or whatever is
  needed. This will also helps bzr config to get it right. The
  get_mutable_section trick should not be needed anymore either.

- decorate branch.unlock to call stack.save if last_lock() it True outside
  of @only_raises scope (evil decorator)

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/weavefile.py
1
 
# Copyright (C) 2005 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
16
16
#
17
17
# Author: Martin Pool <mbp@canonical.com>
18
18
 
19
 
 
20
 
 
21
 
 
22
19
"""Store and retrieve weaves in files.
23
20
 
24
21
There is one format marker followed by a blank line, followed by a
37
34
line contains a newline, or ',' if not.
38
35
"""
39
36
 
 
37
from __future__ import absolute_import
 
38
 
40
39
# TODO: When extracting a single version it'd be enough to just pass
41
40
# an iterator returning the weave lines...  We don't really need to
42
41
# deserialize it into memory.
90
89
 
91
90
def read_weave(f):
92
91
    # FIXME: detect the weave type and dispatch
93
 
    from bzrlib.trace import mutter
94
 
    from weave import Weave
 
92
    from bzrlib.weave import Weave
95
93
    w = Weave(getattr(f, 'name', None))
96
94
    _read_weave_v5(f, w)
97
95
    return w
116
114
    # +59363 0    311.8780    311.8780   +<method 'append' of 'list' objects>
117
115
    # +200   0     30.2500     30.2500   +<method 'readlines' of 'file' objects>
118
116
 
119
 
    from weave import WeaveFormatError
 
117
    from bzrlib.weave import WeaveFormatError
120
118
 
121
 
    lines = iter(f.readlines())
 
119
    try:
 
120
        lines = iter(f.readlines())
 
121
    finally:
 
122
        f.close()
122
123
 
123
124
    try:
124
125
        l = lines.next()