Hello Daniel,
On Thu, 2011-05-19 at 06:51 -0700, Daniel Jelinski wrote:
> Hello,
> I think AvoidUnnecessarySpecialization should not report methods
> decorated with [WebMethod] attribute, or at least should not suggest
> replacing types with interfaces.
> My example:
> [WebMethod]
> public void deleteRecords (List<int> listOfIdentifiers)
> {
> ...
> }
Good point. [WebMethod] decorated methods are contracts so suggesting
changes on them can lead to interop issues.
> The above rule suggests replacing List<int> with IEnumerable<int>.
> However, after such change, WSDL file generation fails with
> InvalidOperationException: method deleteRecords can not be reflected --
> > NotSupportedException: Cannot serialize interface.
>
> Should I file a report?
Yes! that's the only 100% safe way to ensure I won't forget about it :-)
Thanks
Sebastien
--
You received this message because you are subscribed to the Google Groups
"Gendarme" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/gendarme?hl=en.
