Some key functions (e.g. from_document) consume their argument when I suspect there's no concrete benefit to them doing so. For 3.0 we should audit the API and bias towards accepting data (especially potentially large data like bson) by reference rather than by value.