git.delta.rocks / jrsonnet / refs/commits / 7b5cb9f1353f

--- a/crates/jrsonnet-evaluator/src/builtin/mod.rs
+++ b/crates/jrsonnet-evaluator/src/builtin/mod.rs
@@ -144,12 +144,7 @@
 		0, obj: ty!(object) => Val::Obj;
 		1, inc_hidden: ty!(boolean) => Val::Bool;
 	], {
-		let mut out = obj.fields_visibility()
-			.into_iter()
-			.filter(|(_k, v)| *v || inc_hidden)
-			.map(|(k, _v)|k)
-			.collect::<Vec<_>>();
-		out.sort();
+		let out = obj.fields_ex(inc_hidden);
 		Ok(Val::Arr(out.into_iter().map(Val::Str).collect::<Vec<_>>().into()))
 	})
 }
@@ -164,12 +159,7 @@
 		1, f: ty!(string) => Val::Str;
 		2, inc_hidden: ty!(boolean) => Val::Bool;
 	], {
-		Ok(Val::Bool(
-			obj.fields_visibility()
-				.into_iter()
-				.filter(|(_k, v)| *v || inc_hidden)
-				.any(|(k, _v)| *k == *f),
-		))
+		Ok(Val::Bool(obj.has_field_ex(f, inc_hidden)))
 	})
 }
 

--- a/crates/jrsonnet-evaluator/src/evaluate.rs
+++ b/crates/jrsonnet-evaluator/src/evaluate.rs
@@ -245,7 +245,7 @@
 		new_bindings.fill(bindings);
 	}
 
-	let mut new_members = HashMap::new();
+	let mut new_members = FxHashMap::default();
 	for member in members.iter() {
 		match member {
 			Member::Field(FieldMember {
@@ -320,7 +320,7 @@
 		ObjBody::MemberList(members) => evaluate_member_list_object(context, members)?,
 		ObjBody::ObjComp(obj) => {
 			let future_this = FutureObjValue::new();
-			let mut new_members = HashMap::new();
+			let mut new_members = FxHashMap::default();
 			for (k, v) in evaluate_comp(
 				context.clone(),
 				&|ctx| {

3	throw, LazyBinding, LazyVal, ObjMember, ObjValue, Val,3	throw, LazyBinding, LazyVal, ObjMember, ObjValue, Val,
4};4};
5use jrsonnet_parser::Visibility;5use jrsonnet_parser::Visibility;
6use rustc_hash::FxHasher;
6use serde_json::{Map, Number, Value};7use serde_json::{Map, Number, Value};
7use std::{8use std::{
8	collections::HashMap,9	collections::HashMap,
9	convert::{TryFrom, TryInto},10	convert::{TryFrom, TryInto},
11	hash::BuildHasherDefault,
10	rc::Rc,12	rc::Rc,
11};13};
1214
31			}33			}
32			Val::Obj(o) => {34			Val::Obj(o) => {
33				let mut out = Map::new();35				let mut out = Map::new();
34				for key in o.visible_fields() {36				for key in o.fields() {
35					out.insert(37					out.insert(
36						(&key as &str).into(),38						(&key as &str).into(),
37						(&o.get(key)?.expect("field exists")).try_into()?,39						(&o.get(key)?.expect("field exists")).try_into()?,
59				Self::Arr(out.into())61				Self::Arr(out.into())
60			}62			}
61			Value::Object(o) => {63			Value::Object(o) => {
62				let mut entries = HashMap::with_capacity(o.len());64				let mut entries = HashMap::with_capacity_and_hasher(
65					o.len(),
66					BuildHasherDefault::<FxHasher>::default(),
67				);
63				for (k, v) in o {68				for (k, v) in o {
64					entries.insert(69					entries.insert(

--- a/crates/jrsonnet-evaluator/src/val.rs
+++ b/crates/jrsonnet-evaluator/src/val.rs
@@ -395,7 +395,7 @@
 			Self::Obj(obj) => obj,
 			_ => throw!(MultiManifestOutputIsNotAObject),
 		};
-		let keys = obj.visible_fields();
+		let keys = obj.fields();
 		let mut out = Vec::with_capacity(keys.len());
 		for key in keys {
 			let value = obj
@@ -588,8 +588,8 @@
 			if ObjValue::ptr_eq(a, b) {
 				return Ok(true);
 			}
-			let fields = a.visible_fields();
-			if fields != b.visible_fields() {
+			let fields = a.fields();
+			if fields != b.fields() {
 				return Ok(false);
 			}
 			for field in fields {