Replies: 3 comments 2 replies
-
|
Sorry, I have no memory of this. Commented out many years ago a792a7e#diff-dececaf95c2886d45c2a14e906e3f9ec5e985ac726dd64c8a8c3d851969f8ebc Feel free to do as you please with it |
Beta Was this translation helpful? Give feedback.
-
|
Thank - I can’t remember code changes I made in 2018 either. |
Beta Was this translation helpful? Give feedback.
-
|
Ok, there was an issue with this, just not in MSVC. The C++ spec actually doesn't allow this kind of in-class template specialization. But luckily with if constexpr it's even easier to bake it in. |
Beta Was this translation helpful? Give feedback.
-
|
Sorry, did I mess something up? It compiles fine with MSVC. |
Beta Was this translation helpful? Give feedback.
-
|
Yeah, no worries, it's working again, I also didn't realize this upfront. I think it's one of the last areas where msvc diverges from the spec. |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
In IfcHierarchyHelper.h there is
addRelatedObject<typename Schema::IfcRelDefinesByType>function that is commented out.I'd like to use this function but I'm wondering why it is commented out. I don't want to introduce a problem that I'm not seeing.
I moved the function into the IfcHierarchyHelper class declaration, and tweaked the code so it builds. It seems to work fine.
Thanks
Beta Was this translation helpful? Give feedback.
All reactions