Deprecated headers

Issue #187 resolved
Frank Dellaert created an issue

@cforster the LieMatrix.h file itself now generates a warning :-) Find a solution? Perhaps a deprecated directory and new forwarding headers that actually generate the warning?

Also, if you run make check, the warnings are spit out for a number of files. Could you remove the #includes? Things should still work, if not create a branch/pull request and I'll see how to fix it.

Comments (5)

  1. Chris Beall

    How about just surrounding the include line in those three files with a #pragma to ignore that specific warning? For example, this is how we ignore an unused variable warning from including boost/bind.hpp:

    #ifdef __GNUC__
    #pragma GCC diagnostic push
    #pragma GCC diagnostic ignored "-Wunused-variable"
    #endif
    #include <boost/bind.hpp>
    #ifdef __GNUC__
    #pragma GCC diagnostic pop
    #endif
    
  2. Frank Dellaert reporter

    pragmas for every conceivable compiler? I was playing with the idea of forwarding headers anyway (to point from slam to sfm or sam), and the deprecated subdir (with a tests subdir, that is executed), makes sense to me...

  3. Log in to comment