LGTM, one docstring nit. https://codereview.appspot.com/282540043/diff/1/plaso/storage/reader.py File plaso/storage/reader.py (right): https://codereview.appspot.com/282540043/diff/1/plaso/storage/reader.py#newcode2 plaso/storage/reader.py:2: """The storage reader objects.""" This isn't actually where storage reader classes are defined, just the interface.
https://codereview.appspot.com/282540043/diff/1/plaso/storage/reader.py File plaso/storage/reader.py (right): https://codereview.appspot.com/282540043/diff/1/plaso/storage/reader.py#newcode2 plaso/storage/reader.py:2: """The storage reader objects.""" On 2016/02/15 20:16:01, onager wrote: > This isn't actually where storage reader classes are defined, just the > interface. Done.
Code updated.
Changes have been merged with master branch. To close the review and clean up the feature branch you can run: python ./utils/review.py close storage1