2012-05-17, 15:46
@jpsdr - now we're talking. The code 2 changes look good - it would appear that the new nVidia drivers are checking the channel mask, which was not the case during earlier testing, nor the current case with ATI as you can see from my earlier pastebin. But it causes no harm or regression, and I will certainly add that asap - good work. The avgBytesPerSec is only used for PT formats for the client's buffer duration calculations, so it isn't a factor per MS documents, but again, no harm done there.
For the AAC test, yes, I added the check for it when initially writing the device enumeration code, even though it's not currently "supported". Check your PM's for the issue with the code 1 logging you added - those additional lines for logging will not be added.
Okay, so you've got the right device selected now (whether VIA or nVidia) and with the original code you are getting all formats - good to hear!
For the AAC test, yes, I added the check for it when initially writing the device enumeration code, even though it's not currently "supported". Check your PM's for the issue with the code 1 logging you added - those additional lines for logging will not be added.
Okay, so you've got the right device selected now (whether VIA or nVidia) and with the original code you are getting all formats - good to hear!