Skip to content
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

Handle {} as a default value in IDL #53

Open
julianpeeters opened this issue Oct 26, 2016 · 0 comments
Open

Handle {} as a default value in IDL #53

julianpeeters opened this issue Oct 26, 2016 · 0 comments

Comments

@julianpeeters
Copy link
Owner

julianpeeters commented Oct 26, 2016

The {} in ME last_meta_event = {}; results in new ME(, , ), which is incorrect. Check the behavior of Apache Avro Java compilation and conform to that as the defacto standard, wether it throws an error or treats it as a 0-arg ctor to generate new ME("asd", "sdf", None).

Example Idl:

@namespace("com.google.analytics")
protocol MEProtocol {
record ME {
    string type = "asd";
    string service = "sdf";
    union { null, string } service_timezone = null;
}
record Visit {
    ME first_meta_event = {"type": "asd", "service": "sdf", "service_timezone": null };
    ME last_meta_event = {};
  }
}
@julianpeeters julianpeeters changed the title Handle {} as a default value Handle {} as a default value in IDL Oct 26, 2016
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

No branches or pull requests

1 participant