Skip to content

Remove call to addError, as it isn't defined anywhere#7

Open
cainux wants to merge 1 commit intochasdevs:masterfrom
cainux:master
Open

Remove call to addError, as it isn't defined anywhere#7
cainux wants to merge 1 commit intochasdevs:masterfrom
cainux:master

Conversation

@cainux
Copy link

@cainux cainux commented Mar 17, 2021

I'm coming across schema which look something like:

protocol x {
  @namespace("foo")
  fixed Code(3);

  record Currency {
    @java-class("java.math.BigDecimal") string amount;
    foo.Code code;
  }
}

So RecordConverter ends up calling addError before returning "any", which doesn't seem to have a body defined anywhere, resulting in an error.

This PR is a small change to sidestep the error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant