ERROR new is not provided with the correct number of arguments

[lendle]

I'm calling new() without args in an inner constructor to avoid allocating/defining some fields until later. This causes Lint to announce an error. As far as I know this is the only way to do not define all fields when constructing a type.

I'm not really sure what Lint should do here, but I think it is not necessarily wrong to use new() without args. Thoughts?

lintstr("type A  x::Vector ; A() = new() end")
1-element Array{LintMessage,1}:
                 none [A                   ]    3 ERROR  new is not provided with the correct number of arguments

[tonyhffong]

Yes, I missed that. How about this?

• nargs = 0. No warning
• 0 < nargs < nfields. Lint Warning (I do want to catch fat finger cases)
• nargs > nfields. Lint Error (as it is now)

[tonyhffong]

Also, if you do something like this:

type MyType
    a::Int
    b::Int
    function MyType( x )
        @lintpragma( "Ignore short new argument" )
        new(x)
    end
end

It would suppress warnings as well.

[lendle]

I was thinking something along the lines of nargs=0 -> no warning.

[tonyhffong]

Yes, that's exactly the change I've made. The test case:

s = """
type MyType
    a::Int
    b::Int
    MyType()= new()
end
"""
msgs = lintstr(s)
@assert( isempty( msgs ) ) # ok