diff options
author | Martin Fischer <martin@push-f.com> | 2023-08-12 11:06:02 +0200 |
---|---|---|
committer | Martin Fischer <martin@push-f.com> | 2023-08-19 06:41:55 +0200 |
commit | 9f1019afa7a8e9102d67356d85bd632044eb2d0c (patch) | |
tree | 4c6664aad5a11a942d6684a62e507de28193f5bb /src/reader.rs | |
parent | c3d60e88efa32329614178dfc9455ef33ea0a88d (diff) |
break!: merge Tokenizer::new_with_emitter into Tokenizer::new
The Tokenizer does not perform any state switching, since
proper state switching requires a feedback loop between
tokenization and DOM tree building. Using the Tokenizer
directly therefore is a bit of a pitfall, since you might
not expect it to e.g. tokenize `<script><b>` as:
StartTag(StartTag { name: "script", .. })
StartTag(StartTag { name: "b", .. })
Since we don't want to make walking into pitfalls
particularly easy, this commit changes the Tokenizer::new
method so that you have to specify the Emitter.
Since this makes new_with_emitter redundant it is removed.
Diffstat (limited to 'src/reader.rs')
-rw-r--r-- | src/reader.rs | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/reader.rs b/src/reader.rs index f756c65..19929d4 100644 --- a/src/reader.rs +++ b/src/reader.rs @@ -30,8 +30,8 @@ pub trait Reader { /// An object that can be converted into a [`crate::Reader`]. /// -/// For example, any utf8-string can be converted into a `StringReader`, such that -/// `Tokenizer::new("mystring")` and `Tokenizer::new(&String::new("foo"))` work. +/// For example, any utf8-string can be converted into a `StringReader`. +// TODO: , such that [give concrete examples of not-yet-implemented parser API] work. pub trait IntoReader<'a> { /// The reader type into which this type should be converted. type Reader: Reader + 'a; |