> Then it only works for fields and properties defined in the current
> module. Cecil tries hard to not load another module unless it
> explicitly has too. And that's the case here, we have a perfectly
> valid representation, a bit cumbersome for boxed values admittedly,
> but workable.

Actually, come to think of it, why not just add another field to
CustomAttributeArgument — perhaps a bool “IsBoxed”, or even a
“DeclaredType” that specifies the type of the parameter/field/
property? That would seem kind of cleaner than having this nesting? It
would also cause less breakage in existing code that relied on the
previous behaviour.

-- 
--
mono-cecil

Reply via email to