[Mono-bugs] [Bug 373431] New: [PATCH] ReflectionReader using wrong Param table rows

[bugzilla_noreply at novell.com]

https://bugzilla.novell.com/show_bug.cgi?id=373431


           Summary: [PATCH] ReflectionReader using wrong Param table rows
           Product: Mono: Class Libraries
           Version: SVN
          Platform: Other
        OS/Version: Other
            Status: NEW
          Severity: Normal
          Priority: P5 - None
         Component: Cecil
        AssignedTo: jbevain at novell.com
        ReportedBy: davots at coverity.com
         QAContact: mono-bugs at lists.ximian.com
          Found By: ---


Created an attachment (id=203612)
 --> (https://bugzilla.novell.com/attachment.cgi?id=203612)
ParamTableRow.patch

Fixes ReflectionReader bug where Param table rows were applied to the wrong
method parameters.

Param table rows should be associated with method signature parameters using
the pRow's Sequence, not its relative index.  Sequence values are ordered, but
there can be gaps between values.

Example: The first pRow has Sequence value 2.  The first ParameterDefinition
should not use values from any pRow, and the second uses values from the first
pRow.  Without the fix, the 1st param def was using the pRow, and both param
defs were assigned a Sequence of 2.

I didn't find any similar problems with writing pRows, but
ReflectionWriter::VisitParameterDefinitionCollection could be rewritten to
leave out pRows for parameters that do not need them.


-- 
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.