RFR: 8272234: Pass originating elements from Filer to JavaFileManager [v7]

Jonathan Gibbons jjg at openjdk.java.net
Wed Dec 1 15:45:33 UTC 2021


On Fri, 26 Nov 2021 14:35:27 GMT, Jan Lahoda <jlahoda at openjdk.org> wrote:

>> This is a first prototype of a patch that propagates originating elements from `Filer` (`createSourceFile`/`createClassFile`/`createResource`) to the corresponding methods in `JavaFileManager`. As file managers generally don't know about `Element`s, the `Element`s are first converted to their corresponding `FileObject`s (if any). As the currently existing methods only take one `FileObject` as a sibling of the newly created file, a new set of methods is proposed that take multiple originating files.
>> 
>> Any feedback on this prototype would be welcome.
>
> Jan Lahoda has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 28 commits:
> 
>  - Merge branch 'master' into ap-keep-originating-elements
>  - Applying changes as suggested on the review.
>  - Improving javadoc as suggested in the PR.
>  - Test cleanup.
>  - Merge branch 'ap-keep-originating-elements' into ap-keep-originating-elements-forwarding-tweak
>  - Cleanup
>  - Merge branch 'master' into ap-keep-originating-elements
>  - Fixing test (need to exclude a private method from the test).
>  - Fixing typo, as per review comments.
>  - Fixing javadoc.
>  - ... and 18 more: https://git.openjdk.java.net/jdk/compare/040b2c52...e191d936

The recent commits all look good.

-------------

Marked as reviewed by jjg (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/5076


More information about the compiler-dev mailing list