codefire/nfcollector — comment on commit d727382dae1ce89462fdb1d635ca237d84da14f2
File: lib/nfcollector/payload_processor.rb line 6
By seangeo:

This seems odd to store a stream as an instance variable, considering once you call process_payload, you can't call it again without resetting the stream, which isn't done. It might make more sense if the stream was a parameter to the process_payload method. Of course it sort of depends on how reusable the PayloadProcessor object will be, e.g can you call it more than once anyway.