Issue Details (XML | Word | Printable)

Key: BOO-1129
Type: Improvement Improvement
Status: Resolved Resolved
Resolution: Fixed
Priority: Minor Minor
Assignee: Cedric Vivier
Reporter: Cedric Vivier
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Boo

WARNING: Likely typo in type member name 'X'. Did you mean 'X2'?

Created: 22/Jan/09 02:34 AM   Updated: 25/Jan/09 03:16 PM   Resolved: 25/Jan/09 03:16 PM
Return to search
Component/s: Compiler
Affects Version/s: 0.8.2
Fix Version/s: 0.9

Time Tracking:
Not Specified


 Description  « Hide

Following the short time thinking there was a regression somewhere wrt to attribute constructors (working on BOO-1078), I think we should get a typo warning for some boo keywords that are used in a precise context.
After all C# and other languages do not have such potential typo problem with constructors/destructors (at least) since the syntax requires repeating the type name.

Typo would be checked in a method context only (not property etc..) case-insensitively (ie. "def Constructor" would warn) and with +/- 1-letter distance (such as "def IsConstructor" for instance, do not warn).

'Keywords' to be checked:
constructor
destructor

And also (if static):
op_Addition
op_Subtraction
op_Multiply
op_Division
op_Modulus
op_Exponentiation
op_Equality
op_LessThan
op_LessThanOrEqual
op_GreaterThan
op_GreaterThanOrEqual
op_Match
op_NotMatch
op_Member
op_NotMember
op_BitwiseOr
op_BitwiseAnd



Cedric Vivier added a comment - 25/Jan/09 03:16 PM

Landed in rev. 3191