aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorgingerBill <bill@gingerbill.org>2018-08-14 19:29:31 +0100
committergingerBill <bill@gingerbill.org>2018-08-14 19:29:31 +0100
commite1e4a916a5ffb81b4a9fbe6a2ca146db17c42bc8 (patch)
treeb4a89afda20547b51dfb3160dd45b5a18d24a3a8 /src
parent71f94bff76b419b8382ee6f622121e43124427a8 (diff)
Fix demo and improve type hinting
Diffstat (limited to 'src')
-rw-r--r--src/check_expr.cpp58
-rw-r--r--src/check_stmt.cpp2
2 files changed, 59 insertions, 1 deletions
diff --git a/src/check_expr.cpp b/src/check_expr.cpp
index 014249747..df3a5dcff 100644
--- a/src/check_expr.cpp
+++ b/src/check_expr.cpp
@@ -3711,6 +3711,64 @@ isize add_dependencies_from_unpacking(CheckerContext *c, Entity **lhs, isize lhs
}
+void check_assignment_arguments(CheckerContext *ctx, Array<Operand> const &lhs, Array<Operand> *operands, Array<Ast *> const &rhs, bool allow_ok, bool *optional_ok_ = nullptr) {
+ bool optional_ok = false;
+ isize tuple_index = 0;
+ for_array(i, rhs) {
+ CheckerContext c_ = *ctx;
+ CheckerContext *c = &c_;
+
+ Operand o = {};
+
+ Type *type_hint = nullptr;
+
+ if (tuple_index < lhs.count) {
+ type_hint = lhs[tuple_index].type;
+ }
+
+ check_expr_base(c, &o, rhs[i], type_hint);
+ if (o.mode == Addressing_NoValue) {
+ error_operand_no_value(&o);
+ o.mode = Addressing_Invalid;
+ }
+
+ if (o.type == nullptr || o.type->kind != Type_Tuple) {
+ if (allow_ok && lhs.count == 2 && rhs.count == 1 &&
+ (o.mode == Addressing_MapIndex || o.mode == Addressing_OptionalOk)) {
+ Type *tuple = make_optional_ok_type(o.type);
+ add_type_and_value(&c->checker->info, o.expr, o.mode, tuple, o.value);
+
+ Operand val = o;
+ Operand ok = o;
+ val.mode = Addressing_Value;
+ ok.mode = Addressing_Value;
+ ok.type = t_bool;
+ array_add(operands, val);
+ array_add(operands, ok);
+
+ optional_ok = true;
+ tuple_index += 2;
+ } else {
+ array_add(operands, o);
+ tuple_index += 1;
+ }
+ } else {
+ TypeTuple *tuple = &o.type->Tuple;
+ for_array(j, tuple->variables) {
+ o.type = tuple->variables[j]->type;
+ array_add(operands, o);
+ }
+
+ isize count = tuple->variables.count;
+ tuple_index += 2;
+ }
+ }
+
+ if (optional_ok_) *optional_ok_ = optional_ok;
+}
+
+
+
void check_unpack_arguments(CheckerContext *ctx, Entity **lhs, isize lhs_count, Array<Operand> *operands, Array<Ast *> const &rhs, bool allow_ok, bool *optional_ok_ = nullptr) {
bool optional_ok = false;
isize tuple_index = 0;
diff --git a/src/check_stmt.cpp b/src/check_stmt.cpp
index 86dd1bd6d..1475d7b03 100644
--- a/src/check_stmt.cpp
+++ b/src/check_stmt.cpp
@@ -1122,7 +1122,7 @@ void check_stmt_internal(CheckerContext *ctx, Ast *node, u32 flags) {
}
}
- check_unpack_arguments(ctx, nullptr, lhs_operands.count, &rhs_operands, as->rhs, true);
+ check_assignment_arguments(ctx, lhs_operands, &rhs_operands, as->rhs, true);
isize rhs_count = rhs_operands.count;
for_array(i, rhs_operands) {