please review this change, I tested it and it works, and eliminates the unused variables (since assert compiles away in release build)... But the *sourceBuffer++ in the original code was making my head spin. I think I got this right, but want a second pair of eyes on it.