berkus
/
topola
mirror of https://codeberg.org/topola/topola.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

refactor(specctra-core/read): reuse String::read_dsn where possible

This commit is contained in:
Alain Emilia Anna Zscheile 2024-12-06 16:25:54 +01:00 committed by mikolaj
parent 5e54453433
commit 50ef279286
1 changed files with 3 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
crates/specctra-core/src/read.rs
View File

@ -66,7 +66,7 @@ impl<R: std::io::BufRead> ReadDsn<R> for String {
impl<R: std::io::BufRead> ReadDsn<R> for char {
fn read_dsn(tokenizer: &mut ListTokenizer<R>) -> Result<Self, ParseErrorContext> {
let err = tokenizer.add_context(ParseError::Expected("a single character"));
let string = tokenizer.consume_token()?.expect_leaf()?;
let string = String::read_dsn(tokenizer)?;
let mut it = string.chars();
let first = match it.next() {
None => return Err(err),
@ -81,7 +81,7 @@ impl<R: std::io::BufRead> ReadDsn<R> for char {
impl<R: std::io::BufRead> ReadDsn<R> for bool {
fn read_dsn(tokenizer: &mut ListTokenizer<R>) -> Result<Self, ParseErrorContext> {
match tokenizer.consume_token()?.expect_leaf()?.as_str() {
match String::read_dsn(tokenizer)?.as_str() {
"on" => Ok(true),
"off" => Ok(false),
_ => Err(tokenizer.add_context(ParseError::Expected("boolean"))),
@ -94,9 +94,7 @@ macro_rules! impl_ReadDsn_via_FromStr {
($(($t:ty, $name:expr));* $(;)?) => {
$( impl<R: std::io::BufRead> ReadDsn<R> for $t {
fn read_dsn(tokenizer: &mut ListTokenizer<R>) -> Result<Self, ParseErrorContext> {
tokenizer
.consume_token()?
.expect_leaf()?
String::read_dsn(tokenizer)?
.parse()
.map_err(|_| tokenizer.add_context(ParseError::Expected($name)))
}