Skip to content

Commit

Permalink
Add missing includes to Extractor.hpp
Browse files Browse the repository at this point in the history
  • Loading branch information
@gadomski
gadomski committed Mar 5, 2015
1 parent 0c43010 commit 80558e8
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions include/pdal/util/Extractor.hpp
View file
Expand Up @@ -34,6 +34,8 @@

#pragma once

#include <pdal/Dimension.hpp>
#include <pdal/pdal_macros.hpp>

This comment has been minimized.

Sign in to view

Copy link
@abellgithub

abellgithub Mar 5, 2015

Contributor

Does this need to be here?

This comment has been minimized.

Sign in to view

Copy link
@gadomski

gadomski Mar 5, 2015

Author Member

My code broke without it, so yes?

This comment has been minimized.

Sign in to view

Copy link
@abellgithub

abellgithub Mar 5, 2015

Contributor

All that's in that file is stuff for plugins and lround. Didn't see lround used. Hmmm.

This comment has been minimized.

Sign in to view

Copy link
@chambbj

chambbj Mar 5, 2015

Member

I'm guessing you were missing the definition for PDAL_DLL, which is indirectly included by pdal_macros (which itself includes pdal_internal, which includes pdal_export). Seems like including Dimension and pdal_export should do the trick and be a little more succinct.
#include <pdal/portable_endian.hpp>

namespace pdal
Expand Down

0 comments on commit 80558e8

Please sign in to comment.