Public/nushell
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

throw error on redefined record key

Browse Source
This commit is contained in:
Devyn Cairns 2024-07-06 02:57:07 -07:00
parent 2304d71941
commit 716d016a28
No known key found for this signature in database
2 changed files with 12 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
crates/nu-engine/src/eval_ir.rs
View File

@ -436,7 +436,16 @@ fn eval_instruction<D: DebugContext>(
let val = ctx.collect_reg(*val, *span)?; let val = ctx.collect_reg(*val, *span)?;
let record_span = record_value.span(); let record_span = record_value.span();
let mut record = record_value.into_record()?; let mut record = record_value.into_record()?;
record.insert(key.coerce_into_string()?, val);
let key = key.coerce_into_string()?;
if let Some(old_value) = record.insert(&key, val) {
return Err(ShellError::ColumnDefinedTwice {
col_name: key,
second_use: *span,
first_use: old_value.span(),
});
}
ctx.put_reg( ctx.put_reg(
*src_dst, *src_dst,
Value::record(record, record_span).into_pipeline_data(), Value::record(record, record_span).into_pipeline_data(),

4
crates/nu-protocol/src/ir/mod.rs
View File

@ -150,8 +150,8 @@ pub enum Instruction {
ListPush { src_dst: RegId, item: RegId }, ListPush { src_dst: RegId, item: RegId },
/// Spread a value onto the end of a list. Used to construct list literals. /// Spread a value onto the end of a list. Used to construct list literals.
ListSpread { src_dst: RegId, items: RegId }, ListSpread { src_dst: RegId, items: RegId },
/// Insert a key-value pair into a record. Used to construct record literals. Any existing value /// Insert a key-value pair into a record. Used to construct record literals. Raises an error if
/// for the key is overwritten. /// the key already existed in the record.
RecordInsert { RecordInsert {
src_dst: RegId, src_dst: RegId,
key: RegId, key: RegId,