Skip to content

bugfix: *ALTIVEC_H -> *ZVECTOR_H

Shiva Ghoserequested to merge
shivaghose/eigen:master into master

Hello, I noticed that some some header guards were repeated between the AltiVec package and the ZVector packages. This could cause a problem if (for whatever reason) someone attempts to include headers for both architectures.

I'm not sure how to write a test for this, but I'm open to suggestions.

Reference issue

I couldn't find any existing issues (happy to file an issue if the bug tracking makes your lives easier).

What does this implement/fix?

This MR replaces the duplicated header guards found in ZVector/Complex.h and MathFunctions.h with unique header guards.

Additional information

Ideally we would use something like pragma once but I understand that not all compilers and operating systems could easily support it.

Merge request reports