Skip to content

Commit

Permalink
Remove #include<boost/serialization/vector.hpp>. (#903)
Browse files Browse the repository at this point in the history
The comments state the at some point boost serialization and
boost numeric had an ordering requirement of certain headers.

It claimed:

     // starting Boost 1.64, serialization header must come before ublas
     #include <boost/serialization/vector.hpp>
     #include <boost/numeric/ublas/matrix.hpp>

If this is still true then users must include the affected boost headers
before including `highfive/boost.hpp`. However, we're no longer forcing
the dependency onto all HighFive + Boost users.
  • Loading branch information
1uc authored Feb 8, 2024
1 parent 0b47bad commit c22ac21
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions include/highfive/boost.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,6 @@
#include "H5Exception.hpp"

#include <boost/multi_array.hpp>
// starting Boost 1.64, serialization header must come before ublas
#include <boost/serialization/vector.hpp>
#include <boost/numeric/ublas/matrix.hpp>

namespace HighFive {
Expand Down

0 comments on commit c22ac21

Please sign in to comment.