mpusz/mp-units
1
0
Fork 1
mirror of https://github.com/mpusz/mp-units.git synced 2025-07-29 18:07:16 +02:00
Code Issues Packages Projects Releases Wiki Activity

refactor: MSVC bug is fixed so workaround is removed

Browse Source
This commit is contained in:
Mateusz Pusz
2025-02-15 10:39:36 +01:00
parent 014458864c
commit dc8843eeeb
2 changed files with 9 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
docs/getting_started/cpp_compiler_support.md
View File

@ -13,13 +13,13 @@
The table below provides the minimum compiler version required to compile the code using a specific The table below provides the minimum compiler version required to compile the code using a specific
C++ feature: C++ feature:
| C++ Feature | C++ version | gcc | clang | apple-clang | MSVC | | C++ Feature | C++ version | gcc | clang | apple-clang | MSVC |
|-----------------------------------------------------------|:-----------:|:----:|:-----:|:-----------:|:-----------------------------------------:| |-------------------------------|:-----------:|:----:|:-----:|:-----------:|:-----------------------------------------:|
| **Minimum support** | 20 | 12 | 16 | 15 | 194 :bug:{ title="BEWARE of MSVC Bugs!" } | | **Minimum support** | 20 | 12 | 16 | 15 | 194 :bug:{ title="BEWARE of MSVC Bugs!" } |
| **`std::format`** | 20 | 13 | 17 | 16 | 194 | | **`std::format`** | 20 | 13 | 17 | 16 | 194 |
| **C++ modules** | 20 | None | 17 | None | None | | **C++ modules** | 20 | None | 17 | None | None |
| **`import std;`** | 23 | None | 18 | None | None | | **`import std;`** | 23 | None | 18 | None | None |
| **Explicit `this` parameter** | 23 | 14 | 18 | None | None | | **Explicit `this` parameter** | 23 | 14 | 18 | None | None |
??? note "MSVC bugs" ??? note "MSVC bugs"
@ -30,7 +30,6 @@ C++ feature:
Here is a list of the most important MSVC bugs: Here is a list of the most important MSVC bugs:
- [Discrepancy in Behavior of operator*= and operator* for Multiplying int and double at compile time](https://developercommunity.visualstudio.com/t/Discrepancy-in-Behavior-of-operator-an/10732445)
- [Syntax error when using non-type template parameters in templated class member function](https://developercommunity.visualstudio.com/t/Syntax-error-when-using-non-type-templat/10729428) - [Syntax error when using non-type template parameters in templated class member function](https://developercommunity.visualstudio.com/t/Syntax-error-when-using-non-type-templat/10729428)
- [Type always preferred over value when using qualified identifiers](https://developercommunity.visualstudio.com/t/Type-always-prefered-over-value-when-usi/10729382) - [Type always preferred over value when using qualified identifiers](https://developercommunity.visualstudio.com/t/Type-always-prefered-over-value-when-usi/10729382)

8
src/core/include/mp-units/framework/quantity.h
View File

@ -442,9 +442,7 @@ public:
} }
friend constexpr decltype(auto) operator*=(Q&& lhs, const Value& val) friend constexpr decltype(auto) operator*=(Q&& lhs, const Value& val)
{ {
// TODO use *= when compiler bug is resolved: lhs.numerical_value_is_an_implementation_detail_ *= val;
// https://developercommunity.visualstudio.com/t/Discrepancy-in-Behavior-of-operator-an/10732445
lhs.numerical_value_is_an_implementation_detail_ = lhs.numerical_value_is_an_implementation_detail_ * val;
return std::forward<Q>(lhs); return std::forward<Q>(lhs);
} }
@ -465,9 +463,7 @@ public:
friend constexpr decltype(auto) operator/=(Q&& lhs, const Value& val) friend constexpr decltype(auto) operator/=(Q&& lhs, const Value& val)
{ {
MP_UNITS_EXPECTS_DEBUG(val != representation_values<Value>::zero()); MP_UNITS_EXPECTS_DEBUG(val != representation_values<Value>::zero());
// TODO use /= when compiler bug is resolved: lhs.numerical_value_is_an_implementation_detail_ /= val;
// https://developercommunity.visualstudio.com/t/Discrepancy-in-Behavior-of-operator-an/10732445
lhs.numerical_value_is_an_implementation_detail_ = lhs.numerical_value_is_an_implementation_detail_ / val;
return std::forward<Q>(lhs); return std::forward<Q>(lhs);
} }