[Mono-devel-list] Patch for Parameter(s).Location (Re: Do we really need "related location" for CS0136?)

[Atsushi Eno]

Oops, just ignore the title of the previous post. I originally
wanted to talk mainly about that, before writing the patch ;-)

Atsushi Eno

Atsushi Eno wrote:
> Hi,
> 
> I created a patch for mcs that 1) adds Location property to
> ParameterBase, and 2) removes Location property from Parameters.
> On Parameter class there was a comment:
> 
> //TODO: Add location member to this or base class for better error
> location and all methods simplification.
> 
> For Parameters.Location it is only used for CS0136 where IMHO
> we don't need "related location" information (since it must be
> always the exact declaration part). So I would suggest to just
> remove that reporting but for now I just replaced
> Parameters.Location with the exact Location of related Parameter.
> 
> Sounds good?
> 
> (I wanted to solve this matter before improving my precise location
> patch which I created once upon a time.)
> 
> Atsushi Eno