[foreign] RFR 8223185: Consolidate jextract output logic in Writer
Jorn Vernee
jbvernee at xs4all.nl
Thu May 2 11:21:25 UTC 2019
Thanks, I had my doubts about that myself.
Updated webrev:
http://cr.openjdk.java.net/~jvernee/panama/webrevs/8223185/webrev.01/
FWIW; I thought the compilation was a pretty big side-effect to have, so
it should rather be triggered explicitly. But, I didn't really have a
strong opinion either way.
Jorn
Maurizio Cimadamore schreef op 2019-05-02 12:34:
> Overall I like it. The only thing I don't like much is that Main seems
> to be in control of whether classes are being compiled (see call to
> Writer::compileClasses). I think it would be better if that was hidden
> inside Writer itself - e.g. in order to write classfiles, jars or
> jmod, first the internals will have to call 'compileClasses'. That
> way, Main would be free from any ordering issue.
>
> Maurizio
>
> On 01/05/2019 14:06, Jorn Vernee wrote:
>> Hi,
>>
>> I have been experimenting with a plugin based on jextract's new source
>> code generation, for my earlier idea to generate LayoutType constants
>> in various generated classes [1].
>>
>> I ran into a problem using --src-dump-dir. Since the dumping is done
>> inline in the JavaSourceFactory* classes this meant the the changes
>> from the plugin were not being shown in the output.
>>
>> It seems good to move the source output logic to Writer, since then we
>> can do the source output once and for all in Main, after the jextract
>> run. I've also moved the JModWriter and JarWriter calls into Writer,
>> so that we have 1 main interface for doing output (Writer).
>>
>> Please review the following:
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8223185
>> Webrev:
>> http://cr.openjdk.java.net/~jvernee/panama/webrevs/8223185/webrev.00/
>>
>> Thanks,
>> Jorn
>>
>> [1] : https://bugs.openjdk.java.net/browse/JDK-8220063
More information about the panama-dev
mailing list