limscoder / amfast

An Adobe AMF serialization and RPC implementation for Python, written as a C extension for speed.
MIT License
5 stars 6 forks source link

Add Visual Studio 2008 support #24

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Currently compiling amfast with VS dies horribly. The basic reason is that
distutils sees '.c' files and then forces compiler into the brain-dead C
mode (i.e. uses switch /TcSOMEFILE.c). In this mode VS doesn't like a
couple .h file entries of the form "PyObject_HEAD;" (just remove ;) and the
various .c file cases where vars are not pre-declared in the function
header. Solution is to fix those couple issues.

Alternate solution: compile in CPP mode by renaming the .c files to .cpp
and replacing an unfortunate naming choice where "class" was used as local
variable, and a one instance where a string of constant length is stack
allocated.

Original issue reported on code.google.com by jmkin...@gmail.com on 9 Jul 2009 at 1:46

GoogleCodeExporter commented 9 years ago
Fixed in Trunk, waiting for review/confirmation.

Original comment by dthomp...@gmail.com on 9 Jul 2009 at 6:00

GoogleCodeExporter commented 9 years ago

Original comment by dthomp...@gmail.com on 9 Jul 2009 at 6:02

GoogleCodeExporter commented 9 years ago
Update Windows compile Faq before closing issue!

Original comment by dthomp...@gmail.com on 9 Jul 2009 at 6:15

GoogleCodeExporter commented 9 years ago

Original comment by dthomp...@gmail.com on 10 Jul 2009 at 11:54