Errata

"To see the media types, click the link connecting two nodes to each other".
Actually, clicking onto the link does not change the attributes displayed in the gray area of TopoEdit. To see the media types, it is necessary to click on any of the nodes of the link.

Note from the Author or Editor:
You are right - it should say "To see the media types, click the small black square to which to the link connecting the two nodes is attached".

The Note says that MF uses free-threaded COM, but then states that COM should be started as STA (COINIT_
APARTMENTTHREADED). Shouldn't that be MTA, i.e., COINIT_MULTITHREADED? If STA is correct, there must be something missing to reach that conclusion.

Note from the Author or Editor:
Yes, you are right - this is a typo. Here is a discussion of threading in MF, and what you should pass to CoInitializeEx():

http://msdn.microsoft.com/en-us/library/windows/desktop/ee892371(v=vs.85).aspx

The pTypeAttributes argument in the pType->Compare() call refers to the same object as pType. That is, the given media type is compared to itself instead of to the configured output media type.

The same issue is present in the downloadable code sample.

Note from the Author or Editor:
Yes, you are right.

The result of the call to MFGetStrideForBitmapInfoHeader is never assigned to m_stride.

Note from the Author or Editor:
Yes, I think you are right. That's probably when I tested this, I was always able to get the stride from the media type, and therefore never hit this error recovery code path.

Missing while(false); statement at the end of do-while cycle.

Note from the Author or Editor:
Yes, this is a mistake. In the code sample on page 340 there should be an extra line with "while(false);" right after the curly bracket and before the "return hr;" line.

So instead of

" // do something with pResult and pResult1
}

return hr; "

the code sample should look like this:

"
// do something with pResult and pResult1
}
while(false);

return hr;"

CComCritSecLock doesn't have constructor that takes pointer as an argument. Operator '&' in lock(&m_critSec) should be omitted.

Note from the Author or Editor:
This is true - it's a copy-paste error that appears in the appendix, on page 344. Instead of "CComCritSecLock<CComAutoCriticalSection> lock(&m_critSec);" it should be "CComCritSecLock<CComAutoCriticalSection> lock(m_critSec);" - without the ampersand. There are two examples of this problem on this page.