/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/_simple_set_pyx.pyx

  • 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/_simple_set_pyx.pyx
115
115
            raise MemoryError()
116
116
        memset(self._table, 0, n_bytes)
117
117
 
 
118
    def __sizeof__(self):
 
119
        # Note: Pyrex doesn't allow sizeof(class) so we re-implement it here.
 
120
        # Bits are:
 
121
        #   1: PyObject
 
122
        #   2: vtable *
 
123
        #   3: 3 Py_ssize_t
 
124
        #   4: PyObject**
 
125
        # Note that we might get alignment, etc, wrong, but at least this is
 
126
        # better than no estimate at all
 
127
        # return sizeof(SimpleSet) + (self._mask + 1) * (sizeof(PyObject*))
 
128
        return (sizeof(PyObject) + sizeof(void*)
 
129
                + 3*sizeof(Py_ssize_t) + sizeof(PyObject**)
 
130
                + (self._mask + 1) * sizeof(PyObject*))
 
131
 
118
132
    def __dealloc__(self):
119
133
        if self._table != NULL:
120
134
            PyMem_Free(self._table)