duplicate macro definition in instantiate.hpp header

[PeterSchuebel]

This is not an actual build issue right now, but I noticed that the pre-processor macro PCL_INSTANTIATE_PRODUCT_IMPL is defined twice in the instantiate.hpp header, if PCL_NO_PRECOMPILE is defined for the build. As I said, in practice this is currently not causing build problems. At the moment that macro is not used anywhere, if PCL_NO_PRECOMPILE is defined. It just looks unclean.

Your Environment

• Operating System and version: Windows 8.1 64 bit
• Compiler: Microsoft Visual Studio 2013
• PCL Version: 1.8.0-dev

Expected Behavior

Macro is only defined once.

Current Behavior

Macro is identically defined in lines 57 and 60. Macro re-definition is usually a compiler warning, although doesn't seem to trigger (in MSVC 2013) if the definition is identical in both cases.

Possible Solution

Remove duplicate line 57