Mirrors/nushell
2
0
Fork 0
mirror of https://github.com/nushell/nushell synced 2024-12-28 22:13:10 +00:00
Code Issues Projects Releases Packages Wiki Activity

Add inline attribute and address warning

Browse source
This commit is contained in:
Arthur Targaryen 2021-10-09 19:26:20 +02:00
parent d5fdfdb614
commit a0a63c966f
2 changed files with 5 additions and 34 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

37
crates/nu-protocol/src/value/mod.rs
View file

@ -492,40 +492,6 @@ impl PartialOrd for Value {
.. ..
}, },
) if lhs_headers == rhs_headers && lhs == rhs => Some(Ordering::Equal), ) if lhs_headers == rhs_headers && lhs == rhs => Some(Ordering::Equal),
// Note: This may look a bit strange, but a Stream is still just a List,
// it just happens to be in an iterator form instead of a concrete form. If the contained
// values are the same then it should be treated as equal
(
Value::Stream {
stream: stream_lhs, ..
},
Value::List {
vals: stream_rhs, ..
},
) => {
let vals_lhs: Vec<Value> = stream_lhs.clone().collect();
let vals_rhs: Vec<Value> =
stream_rhs.clone().into_iter().into_value_stream().collect();
vals_lhs.partial_cmp(&vals_rhs)
}
// Note: This may look a bit strange, but a Stream is still just a List,
// it just happens to be in an iterator form instead of a concrete form. If the contained
// values are the same then it should be treated as equal
(
Value::List {
vals: stream_lhs, ..
},
Value::Stream {
stream: stream_rhs, ..
},
) => {
let vals_lhs: Vec<Value> =
stream_lhs.clone().into_iter().into_value_stream().collect();
let vals_rhs: Vec<Value> = stream_rhs.clone().collect();
vals_lhs.partial_cmp(&vals_rhs)
}
(Value::Stream { stream: lhs, .. }, Value::Stream { stream: rhs, .. }) => { (Value::Stream { stream: lhs, .. }, Value::Stream { stream: rhs, .. }) => {
lhs.clone().partial_cmp(rhs.clone()) lhs.clone().partial_cmp(rhs.clone())
} }
@ -535,6 +501,9 @@ impl PartialOrd for Value {
(Value::String { val: lhs, .. }, Value::Stream { stream: rhs, .. }) => { (Value::String { val: lhs, .. }, Value::Stream { stream: rhs, .. }) => {
lhs.partial_cmp(&rhs.clone().collect_string()) lhs.partial_cmp(&rhs.clone().collect_string())
} }
// NOTE: This may look a bit strange, but a `Stream` is still just a `List`, it just
// happens to be in an iterator form instead of a concrete form. The contained values
// can be compared.
(Value::Stream { stream: lhs, .. }, Value::List { vals: rhs, .. }) => { (Value::Stream { stream: lhs, .. }, Value::List { vals: rhs, .. }) => {
lhs.clone().collect::<Vec<Value>>().partial_cmp(rhs) lhs.clone().collect::<Vec<Value>>().partial_cmp(rhs)
} }

2
crates/nu-protocol/src/value/range.rs
View file

@ -104,10 +104,12 @@ impl Range {
}) })
} }
#[inline]
fn moves_up(&self) -> bool { fn moves_up(&self) -> bool {
self.from <= self.to self.from <= self.to
} }
#[inline]
fn is_end_inclusive(&self) -> bool { fn is_end_inclusive(&self) -> bool {
matches!(self.inclusion, RangeInclusion::Inclusive) matches!(self.inclusion, RangeInclusion::Inclusive)
} }