[PROTON-254] Swig generated c code for java bindings fails to compile on Windows Visual Studio 2010

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Won't Fix
• Affects Version/s: proton-0.4
• Component/s: proton-c
• Labels: patch windows
• Environment: Windows 7; Visual Studio 2010
• Assignee: Clifford Jansen
• Reporter: Keith Wall
• Created: 28/Feb/13 14:52
• Updated: 11/Jan/16 12:23
• Resolved: 11/Jan/16 12:23

Description

Swig generated C code for the Java bindings fails to compile on Visual Studio 2010 with error message:

C1083: Cannot open include file: 'stdbool.h': No such file or directory

After discussion on list [1], we tried changing bindings/java/CMakeLists.txt to use CPLUSPLUS ON directive to CMake in the case where BUILD_WITH_CXX has been enabled earlier in the build.

if (BUILD_WITH_CXX)
SET_SOURCE_FILES_PROPERTIES(java.i PROPERTIES CPLUSPLUS ON)
endif (BUILD_WITH_CXX)

Switching to CPP in this way exposed a number of errors when compiling the CPP code resulting from the java.i (casting issues, pointer arithmetic etc - see attached make-proton-jni-cxx-errors.txt). However, even after resolving the CPP compilation issues, we then encounter a problem with Swig's handling of nested unions when when CPP. The seeming inability of SWIG (when used with CPP) to represent the union means that SWIG does not produce pn_atom_t_u.java. JNIMessage.java, which references pn_atom_t_u, consequently fails to compile.

The warning from swig is:

/home/keith/src/proton/proton-c/include/proton/codec.h:91: Warning 312: Nested union not currently supported (ignored).

[1] http://mail-archives.apache.org/mod_mbox/qpid-proton/201302.mbox/[e-mail]%3E

Issue Links

• relates to: PROTON-249 Build fails on Win8 / VS 2012 with path error