Hello,
Just wondering if this change is going to be placed under version control or it is something that should be i.e. tested beforehand / postponed for some reason? Jb Evain-2 wrote: > > > Hi, > > On 7/30/09, sergiy.sakharov <[email protected]> wrote: >> It seems that part of logic that is responsible for >> this(PropertyDefinition >> lookup by token) functionality has been forgotten to be implemented: >> The only method that uses ReflectionReader.m_properties field is >> ReflectionReader.GetPropertyDefAt; and there is no >> ReflectionReader.ReadProperties() method defined like in >> AggressiveReflectionReader... > > Nope, it's because the aggressive reflection reader, once it's done > reading the properties and the events, set m_properties and m_events > to null. Just comment out the lines 62 and 63 of the > AggressiveReflectionReader. > > Thanks, > > -- > Jb Evain <[email protected]> > > > > > Best regards, Sergiy -- View this message in context: http://www.nabble.com/Property-lookup-by-token-issue-tp24730645p25097729.html Sent from the Mono - Cecil mailing list archive at Nabble.com. --~--~---------~--~----~------------~-------~--~----~ -- mono-cecil -~----------~----~----~----~------~----~------~--~---
