[SciPy-dev] patch: allow arrayobject.h to compile in c++ compiler
David M. Cooke
cookedm at physics.mcmaster.ca
Tue Nov 22 15:04:48 EST 2005
On Nov 22, 2005, at 13:47 , Fernando Perez wrote:
> Andrew Straw wrote:
>> Sorry, my mistake! It turns out that OpenProducer (part of
>> OpenSceneGraph) must do "typedef int Bool", and so this, combined
>> with
>> arrayobject.h's "typedef undefined char Bool" results in the
>> "redeclaration of int" compiler error. So, you're right, Bool is not
>> defined in C++, and its probaby safest to remove my patch. But
>> would it
>> make sense to rename Bool to ScipyBool or something less likely to
>> defined in other source?
>
> OK, my bad again. I'll stop trying to be useful with low-level
> parts of the
> code I'm not truly intimately familiar with. Two tries, two
> mistakes, one too
> many.
Heh, don't worry about it. I'm not a fan of using "generic" names for
types, and I think we should look at putting them under a namespace
(using a prefix). I added the PY_ARRAY_TYPES_PREFIX to make it easy
for a user to work around what I perceive as our deficiencies.
--
|>|\/|<
/------------------------------------------------------------------\
|David M. Cooke http://arbutus.physics.mcmaster.ca/dmc/
|cookedm at physics.mcmaster.ca
More information about the SciPy-Dev
mailing list