Skip to content

Commit

Permalink
Merge pull request #316 from jwillemsen/jwi-cpp14
Browse files Browse the repository at this point in the history
Removed check for C++14, that is our mandatory required C++ level
  • Loading branch information
jwillemsen authored Aug 17, 2023
2 parents 43fb8aa + a80f583 commit 8481afb
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
5 changes: 2 additions & 3 deletions ridlbe/c++11/templates/cli/hdr/union_post.erb
Original file line number Diff line number Diff line change
Expand Up @@ -66,8 +66,7 @@ private:
<%= switch_cxxtype %> disc_ {<% if has_default?() || has_implicit_default?() %><%= default_label %><% else %><%= members.first.nondefault_labels.first %><% end %>};
union u_type_
{
// Compilation with c++11 standard gives errors on the ' = default' declaration
#if !defined (ACE_HAS_CPP14) || (defined (_MSC_VER) && (_MSC_VER < 1930))
#if defined (_MSC_VER) && (_MSC_VER < 1930)
u_type_ ();
#else
u_type_ () = default;
Expand All @@ -78,7 +77,7 @@ private:
% # Visual Studio 2019 doesn't seem to like using default, this leads to
% # an internal compiler error, see SW331
% if (has_default?() && _m.is_default?) || (!has_default?() && _m == members.first)
#if !defined (ACE_HAS_CPP14) || (defined (_MSC_VER) && (_MSC_VER < 1930))
#if defined (_MSC_VER) && (_MSC_VER < 1930)
<%= _m.cxx_member_type %> <%= _m.cxxname %>_;
#else
<%= _m.cxx_member_type %> <%= _m.cxxname %>_<% if (has_default?() && _m.is_default?) || (!has_default?() && _m == members.first) %> <%= _m.zero_initializer %><% end %>;
Expand Down
2 changes: 1 addition & 1 deletion ridlbe/c++11/templates/cli/inl/union_inl.erb
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
// generated from <%= ridl_template_path %>
#if (!defined(ACE_HAS_CPP14) || (defined (_MSC_VER) && (_MSC_VER < 1930)))
#if defined (_MSC_VER) && (_MSC_VER < 1930)
inline <%= scoped_cxxname %>::u_type_::u_type_ ()
% if has_default?()
: <%= default_member.cxxname %>_<%= default_member.zero_initializer %>
Expand Down

0 comments on commit 8481afb

Please sign in to comment.