luaur-analysis 0.1.3

Luau type checker and type inference (Rust).
Documentation 
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85

use crate::functions::emit_warning::emit_warning;
use crate::records::lint_for_range::LintForRange;
use luaur_ast::records::ast_expr_constant_number::AstExprConstantNumber;
use luaur_ast::records::ast_expr_unary::{AstExprUnary, AstExprUnaryOp};
use luaur_ast::records::ast_node::AstNode;
use luaur_ast::records::ast_stat_for::AstStatFor;
use luaur_ast::records::location::Location;
use luaur_ast::rtti::ast_node_as;
use luaur_config::enums::code::Code;

impl LintForRange {
    pub fn visit_ast_stat_for_linter(&mut self, node: *mut AstStatFor) -> bool {
        unsafe {
            if (*node).step.is_null() {
                let fc = ast_node_as::<AstExprConstantNumber>((*node).from as *mut AstNode);
                let fu = ast_node_as::<AstExprUnary>((*node).from as *mut AstNode);
                let tc = ast_node_as::<AstExprConstantNumber>((*node).to as *mut AstNode);
                let tu = ast_node_as::<AstExprUnary>((*node).to as *mut AstNode);

                let range_location = Location::new(
                    (*(*node).from).base.location.begin,
                    (*(*node).to).base.location.end,
                );

                if !fu.is_null()
                    && (*fu).op == AstExprUnaryOp::Len
                    && !tc.is_null()
                    && (*tc).value == 1.0
                {
                    emit_warning(
                        &mut *self.context,
                        Code::Code_ForRange,
                        range_location,
                        format_args!("For loop should iterate backwards; did you forget to specify -1 as step?"),
                    );
                } else if !fc.is_null() && !tc.is_null() && (*fc).value > (*tc).value {
                    emit_warning(
                        &mut *self.context,
                        Code::Code_ForRange,
                        range_location,
                        format_args!("For loop should iterate backwards; did you forget to specify -1 as step?"),
                    );
                } else if !fc.is_null()
                    && !tc.is_null()
                    && self.get_loop_end((*fc).value, (*tc).value) != (*tc).value
                {
                    emit_warning(
                        &mut *self.context,
                        Code::Code_ForRange,
                        range_location,
                        format_args!(
                            "For loop ends at {} instead of {}; did you forget to specify step?",
                            self.get_loop_end((*fc).value, (*tc).value),
                            (*tc).value
                        ),
                    );
                } else if !fc.is_null()
                    && !tu.is_null()
                    && (*fc).value == 0.0
                    && (*tu).op == AstExprUnaryOp::Len
                {
                    emit_warning(
                        &mut *self.context,
                        Code::Code_ForRange,
                        range_location,
                        format_args!("For loop starts at 0, but arrays start at 1"),
                    );
                } else if !fu.is_null()
                    && (*fu).op == AstExprUnaryOp::Len
                    && !tc.is_null()
                    && (*tc).value == 0.0
                {
                    emit_warning(
                        &mut *self.context,
                        Code::Code_ForRange,
                        range_location,
                        format_args!("For loop should iterate backwards; did you forget to specify -1 as step? Also consider changing 0 to 1 since arrays start at 1"),
                    );
                }
            }
        }

        true
    }
}