mirror of
https://github.com/roc-lang/roc.git
synced 2025-09-28 22:34:45 +00:00
Clean up more unused Nat stuff
This commit is contained in:
parent
76dcbee25f
commit
204cee7d60
5 changed files with 12 additions and 39 deletions
|
@ -10,8 +10,7 @@ use roc_error_macros::internal_error;
|
|||
use roc_module::symbol::{ModuleId, Symbol};
|
||||
use roc_region::all::{Loc, Region};
|
||||
use roc_solve_problem::{
|
||||
NotDerivableContext, NotDerivableDecode, NotDerivableEncode, NotDerivableEq, TypeError,
|
||||
UnderivableReason, Unfulfilled,
|
||||
NotDerivableContext, NotDerivableEq, TypeError, UnderivableReason, Unfulfilled,
|
||||
};
|
||||
use roc_solve_schema::UnificationMode;
|
||||
use roc_types::num::NumericRange;
|
||||
|
@ -508,7 +507,6 @@ fn is_builtin_dec_alias(symbol: Symbol) -> bool {
|
|||
#[inline(always)]
|
||||
fn is_builtin_number_alias(symbol: Symbol) -> bool {
|
||||
is_builtin_fixed_int_alias(symbol)
|
||||
|| is_builtin_nat_alias(symbol)
|
||||
|| is_builtin_float_alias(symbol)
|
||||
|| is_builtin_dec_alias(symbol)
|
||||
}
|
||||
|
@ -947,8 +945,7 @@ impl DerivableVisitor for DeriveEncoding {
|
|||
|
||||
#[inline(always)]
|
||||
fn is_derivable_builtin_opaque(symbol: Symbol) -> bool {
|
||||
(is_builtin_number_alias(symbol) && !is_builtin_nat_alias(symbol))
|
||||
|| is_builtin_bool_alias(symbol)
|
||||
is_builtin_number_alias(symbol) || is_builtin_bool_alias(symbol)
|
||||
}
|
||||
|
||||
#[inline(always)]
|
||||
|
@ -1015,7 +1012,7 @@ impl DerivableVisitor for DeriveEncoding {
|
|||
}
|
||||
|
||||
#[inline(always)]
|
||||
fn visit_alias(var: Variable, symbol: Symbol) -> Result<Descend, NotDerivable> {
|
||||
fn visit_alias(_var: Variable, symbol: Symbol) -> Result<Descend, NotDerivable> {
|
||||
Ok(Descend(!is_builtin_number_alias(symbol)))
|
||||
}
|
||||
|
||||
|
@ -1041,8 +1038,7 @@ impl DerivableVisitor for DeriveDecoding {
|
|||
|
||||
#[inline(always)]
|
||||
fn is_derivable_builtin_opaque(symbol: Symbol) -> bool {
|
||||
(is_builtin_number_alias(symbol) && !is_builtin_nat_alias(symbol))
|
||||
|| is_builtin_bool_alias(symbol)
|
||||
is_builtin_number_alias(symbol) || is_builtin_bool_alias(symbol)
|
||||
}
|
||||
|
||||
#[inline(always)]
|
||||
|
@ -1075,9 +1071,9 @@ impl DerivableVisitor for DeriveDecoding {
|
|||
if subs[field].is_optional() {
|
||||
return Err(NotDerivable {
|
||||
var,
|
||||
context: NotDerivableContext::Decode(NotDerivableDecode::OptionalRecordField(
|
||||
context: NotDerivableContext::DecodeOptionalRecordField(
|
||||
subs[field_name].clone(),
|
||||
)),
|
||||
),
|
||||
});
|
||||
}
|
||||
}
|
||||
|
@ -1120,7 +1116,7 @@ impl DerivableVisitor for DeriveDecoding {
|
|||
}
|
||||
|
||||
#[inline(always)]
|
||||
fn visit_alias(var: Variable, symbol: Symbol) -> Result<Descend, NotDerivable> {
|
||||
fn visit_alias(_var: Variable, symbol: Symbol) -> Result<Descend, NotDerivable> {
|
||||
Ok(Descend(!is_builtin_number_alias(symbol)))
|
||||
}
|
||||
|
||||
|
@ -1179,9 +1175,9 @@ impl DerivableVisitor for DeriveHash {
|
|||
if subs[field].is_optional() {
|
||||
return Err(NotDerivable {
|
||||
var,
|
||||
context: NotDerivableContext::Decode(NotDerivableDecode::OptionalRecordField(
|
||||
context: NotDerivableContext::DecodeOptionalRecordField(
|
||||
subs[field_name].clone(),
|
||||
)),
|
||||
),
|
||||
});
|
||||
}
|
||||
}
|
||||
|
@ -1255,7 +1251,6 @@ impl DerivableVisitor for DeriveEq {
|
|||
#[inline(always)]
|
||||
fn is_derivable_builtin_opaque(symbol: Symbol) -> bool {
|
||||
is_builtin_fixed_int_alias(symbol)
|
||||
|| is_builtin_nat_alias(symbol)
|
||||
|| is_builtin_dec_alias(symbol)
|
||||
|| is_builtin_bool_alias(symbol)
|
||||
}
|
||||
|
@ -1294,9 +1289,9 @@ impl DerivableVisitor for DeriveEq {
|
|||
if subs[field].is_optional() {
|
||||
return Err(NotDerivable {
|
||||
var,
|
||||
context: NotDerivableContext::Decode(NotDerivableDecode::OptionalRecordField(
|
||||
context: NotDerivableContext::DecodeOptionalRecordField(
|
||||
subs[field_name].clone(),
|
||||
)),
|
||||
),
|
||||
});
|
||||
}
|
||||
}
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue