Hi Ajit, Kevin looked into it already yesterday. There was some problem at github at the time I submitted the /integrate comment, which merely needs to be reissued by me. Having been on the road I was not able to do it yesterday, will be first thing after arriving at the office today. Also, thank you very much for your review efforts!
Best regards —-rony Rony G. Flatscher (mobil/e) > Am 29.06.2020 um 07:56 schrieb Ajit Ghaisas <aghai...@openjdk.java.net>: > > On Sat, 27 Jun 2020 14:07:24 GMT, Rony G. Flatscher > <github.com+60214806+rony...@openjdk.org> wrote: > >>> This PR adds a "compile" process instruction to FXML files with the >>> optional PI data "true" (default) and "false". The >>> PI data is turned into a boolean value using "Boolean.parseBoolean(String)". >>> This makes it possible to inject the compile PI everywhere in a FXML file >>> and turn on and off compilation of scripts if >>> the scripting engine implements the javax.script.Compilable interface. The >>> PR adds the ability for a fallback in case >>> compilation of scripts fails, in which case a warning gets issued about >>> this fact and evaluation of the script will be >>> done without compilation. Because of the fallback scripts get compiled with >>> this version by default. >>> --------- >>> ### Progress >>> - [x] Change must not contain extraneous whitespace >>> - [x] Commit message must refer to an issue >>> - [ ] Change must be properly reviewed >>> >>> ### Issue >>> * [JDK-8238080](https://bugs.openjdk.java.net/browse/JDK-8238080): >>> FXMLLoader: if script engines implement >>> javax.script.Compilable compile scripts >>> >>> >>> ### Download >>> `$ git fetch https://git.openjdk.java.net/jfx pull/192/head:pull/192` >>> `$ git checkout pull/192` >> >> Rony G. Flatscher has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Incorporating Kevin's review comments (overlooked some places). > > Marked as reviewed by aghaisas (Reviewer). > > ------------- > > PR: https://git.openjdk.java.net/jfx/pull/192