Merge lp://qastaging/~jelmer/bzr-search/lazy-xml into lp://qastaging/bzr-search

Proposed by Jelmer Vernooij
Status: Merged
Approved by: Robert Collins
Approved revision: 86
Merged at revision: 86
Proposed branch: lp://qastaging/~jelmer/bzr-search/lazy-xml
Merge into: lp://qastaging/bzr-search
Diff against target: 79 lines (+35/-27)
1 file modified
index.py (+35/-27)
To merge this branch: bzr merge lp://qastaging/~jelmer/bzr-search/lazy-xml
Reviewer Review Type Date Requested Status
Robert Collins (community) Approve
Review via email: mp+53394@code.qastaging.launchpad.net

This proposal supersedes a proposal from 2011-03-14.

Description of the change

Add lazy function for determining XML serializers.

This prevents them from being imported every time bzr-search is loaded.

To post a comment you must log in.
Revision history for this message
Robert Collins (lifeless) wrote : Posted in a previous version of this proposal

This seems to be much more than that - have you included your rejected
other branch perhaps?

Revision history for this message
Jelmer Vernooij (jelmer) wrote : Posted in a previous version of this proposal

Argh. Cherrypicked the right changes now. I should not call things I've modified "trunk".

Revision history for this message
Jelmer Vernooij (jelmer) wrote :

ping?

Revision history for this message
Robert Collins (lifeless) :
review: Approve
Revision history for this message
Jelmer Vernooij (jelmer) wrote :

Thanks! Can you merge?

Revision history for this message
Robert Collins (lifeless) wrote :

Future request, can you please include NEWS entries? Thanks.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: