Skip to content

Add ParmParse::Add for AMREX_NUM#4765

Merged
ax3l merged 1 commit intoAMReX-Codes:developmentfrom
WeiqunZhang:enum_addquery
Nov 7, 2025
Merged

Add ParmParse::Add for AMREX_NUM#4765
ax3l merged 1 commit intoAMReX-Codes:developmentfrom
WeiqunZhang:enum_addquery

Conversation

@WeiqunZhang
Copy link
Copy Markdown
Member

This will also enable ParmParse::queryAdd.

This will also enable `ParmParse::queryAdd`.
Copy link
Copy Markdown
Member

@ax3l ax3l left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

This adds add, but didn't you mean to also add queryAdd?

@ax3l ax3l self-assigned this Nov 7, 2025
@ax3l ax3l merged commit 43a3256 into AMReX-Codes:development Nov 7, 2025
72 of 73 checks passed
@WeiqunZhang
Copy link
Copy Markdown
Member Author

We already have a templated queryAdd. But it needs to call add. That's why by adding add for AMREX_NUM, we also enable queryAdd for AMREX_ENUM.

@WeiqunZhang WeiqunZhang deleted the enum_addquery branch November 7, 2025 03:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants