You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@avro.apache.org by "Oscar Westra van Holthe - Kind (Jira)" <ji...@apache.org> on 2022/01/10 13:45:00 UTC
[jira] [Commented] (AVRO-3294) IDL parsing allows doc comments in strange places
[ https://issues.apache.org/jira/browse/AVRO-3294?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17472004#comment-17472004 ]
Oscar Westra van Holthe - Kind commented on AVRO-3294:
------------------------------------------------------
The current implementation of the IDL uses special tokens for doc comments, and passes them via a special call to {{DocCommentHelper}} (this was a similar call in the {{Idl}} class in the past). The PR does not change this due to backwards compatibility.
Given what this does for the calls to {{DocCommentHelper}}, it demonstrates that this solution is not really tenable for the future. After users have had time to adjust, we should break parsing on misplaced documentation comments (regular single-line / multi-line comments will remain to be supported anywhere).
> IDL parsing allows doc comments in strange places
> -------------------------------------------------
>
> Key: AVRO-3294
> URL: https://issues.apache.org/jira/browse/AVRO-3294
> Project: Apache Avro
> Issue Type: Improvement
> Reporter: Oscar Westra van Holthe - Kind
> Priority: Minor
> Labels: pull-request-available
> Time Spent: 20m
> Remaining Estimate: 0h
>
> As demonstrated by the current test case {{TestIdl#testDocCommentsAndWarnings}} (and the files {{comments.avdl}} & {{comments.avpr}}), doc comments are not handled according to spec.
> Goal: use doc comments that are placed according to spec, and generate warnings for all misplaced doc comments.
--
This message was sent by Atlassian Jira
(v8.20.1#820001)