Skip to content

Commit

Permalink
Merge pull request #3145 from garth-wells/fix-cpp-build-errors
Browse files Browse the repository at this point in the history
Syntax fix in declarations for C++20 compliance
  • Loading branch information
Chuck Atkins authored Apr 3, 2022
2 parents 6891c60 + 51561aa commit 9641e4c
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
6 changes: 3 additions & 3 deletions bindings/CXX11/adios2/cxx11/Attribute.h
Original file line number Diff line number Diff line change
Expand Up @@ -43,8 +43,8 @@ class Attribute
* attributes from IO:DefineAttribute<T> or IO:InquireAttribute<T>.
* Can be used with STL containers.
*/
Attribute<T>() = default;
~Attribute<T>() = default;
Attribute() = default;
~Attribute() = default;

/** Checks if object is valid, e.g. if( attribute ) { //..valid } */
explicit operator bool() const noexcept;
Expand Down Expand Up @@ -74,7 +74,7 @@ class Attribute
bool IsValue() const;

private:
Attribute<T>(core::Attribute<IOType> *attribute);
Attribute(core::Attribute<IOType> *attribute);
core::Attribute<IOType> *m_Attribute = nullptr;
};

Expand Down
6 changes: 3 additions & 3 deletions bindings/CXX11/adios2/cxx11/Variable.h
Original file line number Diff line number Diff line change
Expand Up @@ -141,10 +141,10 @@ class Variable
* variables from IO:DefineVariable<T> or IO:InquireVariable<T>.
* Can be used with STL containers.
*/
Variable<T>() = default;
Variable() = default;

/** Default, using RAII STL containers */
~Variable<T>() = default;
~Variable() = default;

/** Checks if object is valid, e.g. if( variable ) { //..valid } */
explicit operator bool() const noexcept;
Expand Down Expand Up @@ -389,7 +389,7 @@ class Variable
using Span = adios2::detail::Span<T>;

private:
Variable<T>(core::Variable<IOType> *variable);
Variable(core::Variable<IOType> *variable);
core::Variable<IOType> *m_Variable = nullptr;

std::vector<std::vector<typename Variable<T>::Info>> DoAllStepsBlocksInfo();
Expand Down

0 comments on commit 9641e4c

Please sign in to comment.