-
Notifications
You must be signed in to change notification settings - Fork 144
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
Add ValueAlias for ValueType reconstruction #4825
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see a problem with this but I guess I don't understand why its needed.
I always have assumed that classes and functions would be templated on Value since that basically determines everything except what the full precisions is.
If a derived type is complex definite and you have an indeterminant type I always assumed its type would be determined via
std::complex<RealAlias<T>> var;
// or
std::complex<FullPrecReal> var_full_prec;
What am I not seeing here?
Value is not complex definite. QTBase and QTFull relies on ifdef. To have full template, I tries not ot rely on QTFull but promote precision but to derive FullPrecValue based on Value using ValueAlias. |
This is why I thought we would need This is why |
|
I get you now. I agree that |
Test this please |
Proposed changes
Add a helper class for Value type reconstruction.
What type(s) of changes does this code introduce?
Does this introduce a breaking change?
What systems has this change been tested on?
laptop
Checklist