Skip to content

Commit 23782b7

Browse files
authored
fix: update jsonc-parser to 0.26 (#39)
1 parent 05dd3e3 commit 23782b7

File tree

4 files changed

+12
-11
lines changed

4 files changed

+12
-11
lines changed

Cargo.lock

+2-2
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Cargo.toml

+1-1
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ tracing = ["dprint-core/tracing"]
2828
anyhow = "1.0.64"
2929
dprint-core = { version = "0.66.2", features = ["formatting"] }
3030
dprint-core-macros = "0.1.0"
31-
jsonc-parser = { version = "0.23.0" }
31+
jsonc-parser = { version = "0.26.0" }
3232
serde = { version = "1.0.144", features = ["derive"] }
3333
serde_json = { version = "1.0", optional = true }
3434
text_lines = "0.6.0"

src/format_text.rs

+4-3
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ use dprint_core::configuration::resolve_new_line_kind;
66
use dprint_core::formatting::PrintOptions;
77
use jsonc_parser::parse_to_ast;
88
use jsonc_parser::CollectOptions;
9+
use jsonc_parser::CommentCollectionStrategy;
910
use jsonc_parser::ParseResult;
1011

1112
use super::configuration::Configuration;
@@ -48,16 +49,16 @@ fn parse(text: &str) -> Result<ParseResult<'_>> {
4849
let parse_result = parse_to_ast(
4950
text,
5051
&CollectOptions {
51-
comments: true,
52+
comments: CommentCollectionStrategy::Separate,
5253
tokens: true,
5354
},
5455
&Default::default(),
5556
);
5657
match parse_result {
5758
Ok(result) => Ok(result),
5859
Err(err) => bail!(dprint_core::formatting::utils::string_utils::format_diagnostic(
59-
Some((err.range.start, err.range.end)),
60-
&err.message,
60+
Some((err.range().start, err.range().end)),
61+
&err.kind().to_string(),
6162
text,
6263
)),
6364
}

src/generation/generate.rs

+5-5
Original file line numberDiff line numberDiff line change
@@ -190,7 +190,7 @@ fn gen_object<'a>(obj: &'a Object, context: &mut Context<'a, '_>) -> PrintItems
190190
open_token: sc!("{"),
191191
close_token: sc!("}"),
192192
range: obj.range,
193-
first_member: obj.properties.first().map(|f| &f.range),
193+
first_member: obj.properties.first().map(|f| f.range()),
194194
prefer_single_line_when_empty: false,
195195
},
196196
context,
@@ -276,7 +276,7 @@ fn gen_comma_separated_values<'a>(
276276
|| match context.config.trailing_commas {
277277
TrailingCommaKind::Always => true,
278278
TrailingCommaKind::Maintain => match &value {
279-
Some(value) => context.token_finder.get_next_token_if_comma(value.range()).is_some(),
279+
Some(value) => context.token_finder.get_next_token_if_comma(&value.range()).is_some(),
280280
None => false,
281281
},
282282
TrailingCommaKind::Jsonc => context.is_jsonc,
@@ -356,17 +356,17 @@ fn gen_comma_separated_value<'a>(
356356
}
357357
}
358358

359-
struct GenSurroundedByTokensOptions<'a> {
359+
struct GenSurroundedByTokensOptions {
360360
open_token: &'static StringContainer,
361361
close_token: &'static StringContainer,
362362
range: Range,
363-
first_member: Option<&'a Range>,
363+
first_member: Option<Range>,
364364
prefer_single_line_when_empty: bool,
365365
}
366366

367367
fn gen_surrounded_by_tokens<'a, 'b>(
368368
gen_inner: impl FnOnce(&mut Context<'a, 'b>) -> PrintItems,
369-
opts: GenSurroundedByTokensOptions<'a>,
369+
opts: GenSurroundedByTokensOptions,
370370
context: &mut Context<'a, 'b>,
371371
) -> PrintItems {
372372
let open_token_end = opts.range.start + opts.open_token.text.len();

0 commit comments

Comments
 (0)