Why these changes? The previous code had more self-explanatory arguments. Why is "a stream of BE shorts" suddenly a `const char *` now?