NightOwl888 commented on issue #648: URL: https://github.com/apache/lucenenet/issues/648#issuecomment-1285347387
I guess it depends. What happens if you create a sample app that has an API that returns an instance (declared in a static readonly variable) of a readonly collection from `System.Collections.ObjectModel` and scan it? If it doesn't even consider those to be "immutable", I doubt there is much chance we can make it identify the J2N collections. But I agree that getting it to recognize those collections as being immutable for the general public would be ideal, since J2N is a general library that others might use outside of Lucene.NET. That being said, the code analyzer infrastructure is flexible in that it allows you to define your own IDs to suppress with the same old `[SuppressMessage]` attribute, so if we can work out what the ID is for that issue, we can deal with this conditionally (assuming it is built on top of the .NET code analysis infrastructure). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
