📝 Update todo comments

ast/ast.go

@@ -203,7 +203,6 @@ type FunctionType struct {
 
 // Param is not a type and not an expression, but we place it here since it is
 // closely related to types.
-// @TODO: Data location is missing
 type Param struct {
 	Name         *Identifier  // param name e.g. "x" or "recipient"
 	Type         Type         // e.g. ElementaryType, FunctionType etc.

parser/parser.go

@@ -171,6 +171,10 @@ func (p *Parser) parseFunctionDeclaration() *ast.FunctionDeclaration {
 		// ( typeName <<data location>> <<identifier>>, ... )
 		// The typeName is required while data location and identifier are optional.
 
+		// @TODO: Param list parsing could be extracted to a separate
+		// function since it is used in other places as well e.g. fallback,
+		// receive functions, modifiers and return values.
+
 		// @TODO: Params can have other types than elementary types. For example:
 		// - user defined like Contract names and structs
 		// - function types