Hello Sebastien, Report filed :) Daniel
On 19 Maj, 17:16, Sebastien Pouliot <[email protected]> wrote: > 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.
