MarshallingAttributeAnalyzers throws InvalidOperationException #72622

Activity

2. ghost added

   untriagedNew issue has not been triaged by the area ownerNew issue has not been triaged by the area owner

   on Jul 21, 2022
3. jkoritzinsky commented on Jul 21, 2022

   

   jkoritzinsky

   on Jul 21, 2022

   Member

   More actions

   This will be fixed by #72620

   👍React with 👍1Reacted by Sven Boemer
4. jkotas

   added

   area-System.Runtime.InteropServices

   on Jul 21, 2022
6. AaronRobinsonMSFT

   added

   source-generatorIndicates an issue with a source generator featureIndicates an issue with a source generator feature

   and removed

   untriagedNew issue has not been triaged by the area ownerNew issue has not been triaged by the area owner

   on Jul 21, 2022
7. AaronRobinsonMSFT

   added this to the 7.0.0 milestone on Jul 21, 2022
8. ghost added

   in-prThere is an active PR which will close this issue when it is mergedThere is an active PR which will close this issue when it is merged

   on Jul 21, 2022
9. jkoritzinsky

   closed this as completedin #72620on Jul 21, 2022
10. ghost removed

    in-prThere is an active PR which will close this issue when it is mergedThere is an active PR which will close this issue when it is merged

    on Jul 21, 2022
11. sbomer commented on Jul 21, 2022

    

    sbomer

    on Jul 21, 2022

    MemberAuthor

    More actions

    I'm still seeing errors like this:

    CSC : error AD0001: Analyzer 'Microsoft.Interop.Analyzers.CustomMarshallerAttributeAnalyzer' threw an exception of type 'System.InvalidCastExceptio
    n' with message 'Unable to cast object of type 'Microsoft.CodeAnalysis.CSharp.Symbols.PublicModel.ArrayTypeSymbol' to type 'Microsoft.CodeAnalysis.
    INamedTypeSymbol'.'. [C:\Users\svbomer\src\runtime\src\coreclr\System.Private.CoreLib\System.Private.CoreLib.csproj]
    
12. sbomer

    reopened this on Jul 21, 2022
13. jkoritzinsky commented on Jul 21, 2022

    

    jkoritzinsky

    on Jul 21, 2022

    Member

    More actions

    :( Why did I decide to try to be a kinder citizen in the VS ecosystem and work off of syntax instead of symbols until I can use operations???

    I'll take a look

    😕React with 😕2Reacted by Sven Boemer and Elinor Fung
14. jkoritzinsky commented on Jul 21, 2022

    

    jkoritzinsky

    on Jul 21, 2022

    Member

    More actions

    Also I didn't realize we had AD0001 disabled locally. That will make this much easier to debug
15. jkoritzinsky commented on Jul 21, 2022

    

    jkoritzinsky

    on Jul 21, 2022

    Member

    More actions

    Actually, this looks like a different bug. I'll include this one in my PR that adds more support for other things to the analyzer, as I'm about to put out the PR for it and the work would conflict.

    👍React with 👍1Reacted by Sven Boemer
16. ghost added

    in-prThere is an active PR which will close this issue when it is mergedThere is an active PR which will close this issue when it is merged

    on Jul 21, 2022
17. jkoritzinsky

    closed this as completedin #72643on Jul 26, 2022
18. ghost removed

    in-prThere is an active PR which will close this issue when it is mergedThere is an active PR which will close this issue when it is merged

    on Jul 26, 2022
19. ghost locked as resolved and limited conversation to collaborators on Aug 25, 2022