Commit 2f02fa58 authored by Chris Harris's avatar Chris Harris
Browse files

Change __APPLE__ to __clang__ for Clang visibilty fix

The visibility issue fixed by 6d8a5f6e is also seen with Clang
on other platforms. Also moved the exports to existing forward
declarations.

The Clang bug in question is:

http://llvm.org/bugs/show_bug.cgi?id=10113

Change-Id: I32592d8e8c4169925006185aaf7f027b588697f0
parent 7b21caed
......@@ -34,7 +34,12 @@ class vtkIndent;
class vtkObjectBase;
class vtkLargeInteger;
class vtkSmartPointerBase;
// workaround clang bug, needs export on forward declaration
#ifdef __clang__
class VTKCOMMONCORE_EXPORT vtkStdString;
#else
class vtkStdString;
#endif
class VTKCOMMONCORE_EXPORT vtkOStreamWrapper
{
......
......@@ -23,12 +23,6 @@
# pragma warning (disable: 4661)
#endif
#ifdef __APPLE__
#include "vtkCommonCoreModule.h"
// Make visibility known prior to any symbols referencing this.
class VTKCOMMONCORE_EXPORT vtkStdString;
#endif
#include "vtkStdString.h"
#include "vtkArrayIteratorTemplate.txx"
......
......@@ -56,7 +56,12 @@ class vtkStdString;
class vtkUnicodeString;
class vtkObjectBase;
class vtkAbstractArray;
// workaround clang bug, needs export on forward declaration
#ifdef __clang__
class VTKCOMMONCORE_EXPORT vtkVariant;
#else
class vtkVariant;
#endif
struct vtkVariantLessThan;
//BTX
......
......@@ -24,12 +24,6 @@
# pragma warning (disable: 4661)
#endif
#ifdef __APPLE__
#include "vtkCommonCoreModule.h"
// Make visibility known prior to any symbols referencing this.
class VTKCOMMONCORE_EXPORT vtkVariant;
#endif
#include "vtkVariantArray.h"
#include "vtkDataArray.h"
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment