fmang/opustags

Let TagHandler know when all streams have been processed

rr- opened this issue · 3 comments

rr- commented

There are a few ways to accomplish this:

  1. Create an event ITagsHandler::start_of_file(const int stream_count)
  2. Create an event ITagsHandler::end_of_file()
  3. Both?

In any case, I'd like to avoid introducing any kind of state to ITagsHandler and let StreamTagsHandler deal with state as a part of its implementation.

fmang commented

Option 2 has been ready for a while, it's called end_of_stream and could benefit from a better name.

I think this is enough to implement all we'd ever need.

rr- commented

In that case, my proposition would be end_of_file or end_of_streams. Could you change the name?

fmang commented

Done. I added ITagsHandler::start_of_stream too so that the lister is aware of the unknown streams.