OpenLP

Code review comment for lp://qastaging/~trb143/openlp/media_state

  1. media_state
  2. Merge into trunk
Revision history for this message
Raoul Snyman (raoul-snyman) wrote :

Overall this looks good. There is some commented code that I think either need to be fixed or removed? Also, I noticed there's some code that deals with streaming on Windows and Linux, which I presume you'll need the macOS equivalent for. Is there any other code that you need the macOS equivalent for?

review: Needs Information

« Back to merge proposal