Add `from_path` and `from_string` variants that don't buffer output #86

A `Serializer` now receives an instance of `std::io::Write`. This means that we can no longer write the UTF-8 BOM nor the `@charset "UTF-8" directive when serialization is done and instead do the ASCII-content check while visiting the AST. All `inspect_*` methods now create a temporary buffer and pass it to a `Serializer`. Then they (unsafely) interpret the buffer as a UTF-8 string. The goal is to avoid this extra set of allocations and always write directly to the writer passed by the API user. We also removed some extra allocations marked with todo comments.

This also adds a public API to write the output of a compiled set of SASS files into a `Write`. We should add a recommendation to the rustdoc about passing a buffered writer; otherwise performance will probably be horrible. The previous `String`-based APIs were also moved to this new set of APIs by creating a temporary `Vec<u8>` buffer, passing it to the serializer, and finally (unsafely) reinterpreting the buffer contents as a UTF-8 string.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add `from_path` and `from_string` variants that don't buffer output #86

Add `from_path` and `from_string` variants that don't buffer output #86

Commits on Oct 7, 2023

Add from_path and from_string variants that don't buffer output #86

Add from_path and from_string variants that don't buffer output #86

Commits on Oct 7, 2023

Add `from_path` and `from_string` variants that don't buffer output #86

Add `from_path` and `from_string` variants that don't buffer output #86