locked
Media Engine Sample Feedback

    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.

    -Jer

    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


    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