Skip to content

[Java.Interop.Tools.JavaSource] Handle common parsing exceptions #1060

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Nov 29, 2022

Conversation

pjcollins
Copy link
Member

@pjcollins pjcollins commented Nov 22, 2022

Fixes the 91 instances of exceptions being thrown when generating API
docs for the API 33 version of Mono.Android. The two most common
instances were:

System.IndexOutOfRangeException: Index was outside the bounds of the array.
  at Java.Interop.Tools.JavaSource.UnknownHtmlElementStartTerminal.TryMatch (Irony.Parsing.ParsingContext context, Irony.Parsing.ISourceStream source) [0x0003e] in /Users/builder/azdo/_work/4/s/external/Java.Interop/src/Java.Interop.Tools.JavaSource/Java.Interop.Tools.JavaSource/SourceJavadocToXmldocGrammar.HtmlBnfTerms.cs:390

System.ArgumentOutOfRangeException: Index was out of range. Must be non-negative and less than the size of the collection.
Parameter name: index
  at System.Collections.Generic.List`1[T].get_Item (System.Int32 index) [0x00009] in <08f46039e5064c628bf7795f9b970b7b>:0
  at Java.Interop.Tools.JavaSource.SourceJavadocToXmldocGrammar+BlockTagsBnfTerms+<>c__DisplayClass1_0.<CreateRules>b__8 (Irony.Ast.AstContext context, Irony.Parsing.ParseTreeNode parseNode) [0x00025] in /Users/builder/azdo/_work/4/s/external/Java.Interop/src/Java.Interop.Tools.JavaSource/Java.Interop.Tools.JavaSource/SourceJavadocToXmldocGrammar.BlockTagsBnfTerms.cs:151

35 of the 118 href element parsing errors have also been fixed.

@pjcollins pjcollins force-pushed the doc-parse-exceptions branch from 7377d37 to 43faa47 Compare November 23, 2022 22:24
Fixes the 91 instances of exceptions being thrown when generating API
docs for the API 33 version of Mono.Android.  The two most common
instances were:

    System.IndexOutOfRangeException: Index was outside the bounds of the array.
      at Java.Interop.Tools.JavaSource.UnknownHtmlElementStartTerminal.TryMatch (Irony.Parsing.ParsingContext context, Irony.Parsing.ISourceStream source) [0x0003e] in /Users/builder/azdo/_work/4/s/external/Java.Interop/src/Java.Interop.Tools.JavaSource/Java.Interop.Tools.JavaSource/SourceJavadocToXmldocGrammar.HtmlBnfTerms.cs:390

    System.ArgumentOutOfRangeException: Index was out of range. Must be non-negative and less than the size of the collection.
    Parameter name: index
      at System.Collections.Generic.List`1[T].get_Item (System.Int32 index) [0x00009] in <08f46039e5064c628bf7795f9b970b7b>:0
      at Java.Interop.Tools.JavaSource.SourceJavadocToXmldocGrammar+BlockTagsBnfTerms+<>c__DisplayClass1_0.<CreateRules>b__8 (Irony.Ast.AstContext context, Irony.Parsing.ParseTreeNode parseNode) [0x00025] in /Users/builder/azdo/_work/4/s/external/Java.Interop/src/Java.Interop.Tools.JavaSource/Java.Interop.Tools.JavaSource/SourceJavadocToXmldocGrammar.BlockTagsBnfTerms.cs:151

35 of the 118 href element parsing errors have also been fixed.
@pjcollins pjcollins force-pushed the doc-parse-exceptions branch from 43faa47 to 802ae2d Compare November 23, 2022 23:46
@pjcollins
Copy link
Member Author

Corresponding docs update is in xamarin/android-api-docs#45

@pjcollins pjcollins marked this pull request as ready for review November 28, 2022 20:47
@pjcollins pjcollins requested a review from jonpryor November 28, 2022 20:47
@jonpryor jonpryor merged commit 3a9f770 into dotnet:main Nov 29, 2022
@pjcollins pjcollins deleted the doc-parse-exceptions branch November 29, 2022 20:13
@github-actions github-actions bot locked and limited conversation to collaborators Apr 12, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants