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

Qute Template - add convenient data() methods #12656

Merged
merged 1 commit into from
Oct 13, 2020

Conversation

mkouba
Copy link
Contributor

@mkouba mkouba commented Oct 12, 2020

No description provided.

@mkouba mkouba added this to the 1.10 - master milestone Oct 12, 2020
@mkouba mkouba added the area/qute The template engine label Oct 12, 2020
@fwgreen
Copy link

fwgreen commented Oct 13, 2020

Would support for java.util.Map carry a performance penalty?

@mkouba
Copy link
Contributor Author

mkouba commented Oct 13, 2020

Would support for java.util.Map carry a performance penalty?

I'm not sure I understand? We do support Maps, i.e. you can do template.data(myMap).

@fwgreen
Copy link

fwgreen commented Oct 13, 2020

@mkouba Yes, exactly. I was curious because template.data(Map.of(key1, data1, key2, data2,...) would seem to also do the trick. Unless there was an unseen cost (I noticed there are no examples using this approach in the docs).

@mkouba
Copy link
Contributor Author

mkouba commented Oct 13, 2020

@fwgreen No hidden cost ;-). But quarkus/qute is still JDK8-compatible.

@mkouba mkouba merged commit 770db10 into quarkusio:master Oct 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/qute The template engine
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants