From a99852084ff6f0491734d508a4dcd2e7d8411814 Mon Sep 17 00:00:00 2001 From: _ov4 Date: Sat, 17 Oct 2026 06:04:11 +0000 Subject: feat(classes): warn when method declarations shadow fields Retain parameter source lines for warnings and preserve field access prefix validation when distinguishing fields from methods. Co-Authored-By: Claude Fable 5.1 --- lib/Parser/ParserFunction.cc | 1 + lib/Parser/ParserType.cc | 4 +++- 2 files changed, 4 insertions(+), 1 deletion(-) (limited to 'lib/Parser') diff --git a/lib/Parser/ParserFunction.cc b/lib/Parser/ParserFunction.cc index 1aef756..ad187c8 100644 --- a/lib/Parser/ParserFunction.cc +++ b/lib/Parser/ParserFunction.cc @@ -51,6 +51,7 @@ std::optional> Parser::ParsePrototypeArgs(bool AllowByRef return std::nullopt; } ParamDecl P; + P.Line = Lex.getLine(); P.Name = Lex.IdentifierStr; P.Mode = Current; getNextToken(); diff --git a/lib/Parser/ParserType.cc b/lib/Parser/ParserType.cc index 0d664d0..dab71a1 100644 --- a/lib/Parser/ParserType.cc +++ b/lib/Parser/ParserType.cc @@ -124,7 +124,9 @@ std::optional> Parser::ParseRecordBody( if (AtField()) { // Class fields default to public, like record fields. bool IsPublic = CurTok != tok_private; - if (IsClass && (CurTok == tok_public || CurTok == tok_private)) getNextToken(); + if (IsClass && (CurTok == tok_public || CurTok == tok_private) && + (Lex.peekToken() == tok_function || Lex.peekToken() == tok_procedure)) + getNextToken(); if (IsClass && (CurTok == tok_function || CurTok == tok_procedure)) { auto Method = CurTok == tok_function ? ParseFunction(Name, IsPublic) : ParseProcedure(Name, IsPublic); -- cgit v1.3.1-4-g156e