Media Engine Sample Feedback RRS feed

  • Question

  • I don't want to come off as too pedantic, but a couple items in the Media Engine Sample...

    In meplayer.cpp, CoCreateInstance is used.  This method's documentation says it's a "Desktop only" function.  Should this be CoCreateInstanceFromApp?  I'm mostly curious if CoCreateInstance is incorrectly documented, or if this was just overlooked in the sample.

    Also in meplayer.cpp, the MEPlayer::OnTimer() method has a EnterCriticalSection, code, LeaveCriticalSection.  If the code between throws an error, LeaveCriticalSection will never be called.  This exists in other functions in the code also. Replacing with an "AutoLock" RAII class would fix.


    Thursday, September 13, 2012 10:24 PM

All replies

  • Hello,

    I check the document in the sample codes, it said that "This sample demonstrates video playback using the MediaEngine API in a native C++ app"

    I think this sample code is based on native application, although it looks like Windows Store app.

    Best regards,

    Jesse Jiang [MSFT]
    MSDN Community Support | Feedback to us

    Monday, September 17, 2012 6:04 AM
  • Yes it is a "Windows Store App".  I was only providing some feedback on this sample in hopes to improve it.  I have seen others confused about using CoCreateInstance within a Metro application.
    Monday, September 17, 2012 11:17 PM