git.delta.rocks / jrsonnet / refs/commits / 9070b9ae357d

difftreelog

fix StateBuilder leftovers

Yaroslav Bolyukin2024-05-27parent: #bc5db17.patch.diff
in: master

3 files changed

modifiedCargo.lockdiffbeforeafterboth
before · Cargo.lock
150 packageslockfile v3
after · Cargo.lock
149 packageslockfile v3
modifiedCargo.tomldiffbeforeafterboth
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -56,10 +56,6 @@
 sha2 = "0.10.8"
 sha3 = "0.10.8"
 
-# Pre-parsed stdlib serialization.
-# TODO: Drop in favor of replacing std.jsonnet with full native implementation, version 2.0 of bincode is bad.
-bincode = "1.3"
-
 # Source code parsing.
 # Jrsonnet has two parsers for jsonnet - one is for execution, and another is for better parsing diagnostics/lints/LSP.
 # First (and fast one) is based on peg, second is based on rowan.
modifiedcmds/jrsonnet/src/main.rsdiffbeforeafterboth
--- a/cmds/jrsonnet/src/main.rs
+++ b/cmds/jrsonnet/src/main.rs
@@ -147,9 +147,8 @@
 }
 
 fn main_catch(opts: Opts) -> bool {
-	let s = State::default();
 	let trace = opts.trace.trace_format();
-	if let Err(e) = main_real(&s, opts) {
+	if let Err(e) = main_real(opts) {
 		if let Error::Evaluation(e) = e {
 			let mut out = String::new();
 			trace.write_trace(&mut out, &e).expect("format error");
@@ -162,18 +161,17 @@
 	true
 }
 
-fn main_real(s: &State, opts: Opts) -> Result<(), Error> {
+fn main_real(opts: Opts) -> Result<(), Error> {
 	let _gc_leak_guard = opts.gc.leak_on_exit();
 	let _gc_print_stats = opts.gc.stats_printer();
 	let _stack_depth_override = opts.misc.stack_size_override();
 
 	let import_resolver = opts.misc.import_resolver();
-	s.set_import_resolver(import_resolver);
+	let std = opts.std.context_initializer()?;
 
-	let std = opts.std.context_initializer()?;
-	if let Some(std) = std {
-		s.set_context_initializer(std);
-	}
+	let mut s = State::builder();
+	s.import_resolver(import_resolver).context_initializer(std);
+	let s = s.build();
 
 	let input = opts.input.input.ok_or(Error::MissingInputArgument)?;
 	let val = if opts.input.exec {
@@ -189,7 +187,7 @@
 
 	let tla = opts.tla.tla_opts()?;
 	#[allow(unused_mut)]
-	let mut val = apply_tla(s.clone(), &tla, val)?;
+	let mut val = apply_tla(s, &tla, val)?;
 
 	#[cfg(feature = "exp-apply")]
 	for apply in opts.input.exp_apply {