Merge lp://staging/~cjwatson/storm/reference-set-getitem into lp://staging/storm
Proposed by
Colin Watson
| Status: | Merged |
|---|---|
| Merged at revision: | 544 |
| Proposed branch: | lp://staging/~cjwatson/storm/reference-set-getitem |
| Merge into: | lp://staging/storm |
| Diff against target: |
103 lines (+64/-0) 3 files modified
NEWS (+1/-0) storm/references.py (+3/-0) storm/tests/store/base.py (+60/-0) |
| To merge this branch: | bzr merge lp://staging/~cjwatson/storm/reference-set-getitem |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Ioana Lasc (community) | Approve | ||
| Storm Developers | Pending | ||
|
Review via email:
|
|||
Commit message
Implement __getitem__ on bound ReferenceSets.
Description of the change
It's useful to be able to do [index] and [start:end] on bound ReferenceSets just as you can with ordinary ResultSets, and supporting this is just a matter of passing through the __getitem__ method.
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Nicely done!