Compiler allows a global variable name to be the same as a function name

[yzhang90]

What's your issue about?

The compiler should rejects the following program:

foo: decimal[3] 
@public def foo(): 
    self.foo = [1, 2.1, 3]

because the global variable and the function have the same name and it is very confusing to write such program.

How can it be fixed?

Checking name duplication in parser.py

Cute Animal Picture

[DavidKnott]

@yzhang90 On it 👍

[DavidKnott]

@fubuloubu I'm working on this. Do you think the following should be allowed?

def foo():
    foo: num

I'm leaning towards yes, as I'm thinking that uniforming naming can increase readability.

[fubuloubu]

I'm not a big fan of that personally, but I think it would work because foo() is in the self "namespace" so that's fine

[jakerockland]

@yzhang90 @DavidKnott was poking around at bugs that could be addressed and ended up here, but it looks like this issue can be closed with #659 merged?

[jacqueswww]

@jakerockland that is correct ;) sorry for the later reply.