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

Answer 1

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

Answer 2

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.