-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10363/
-----------------------------------------------------------
Review request for qpid, Andrew Stitcher and Rafael Schloming.
Bugs: proton-212
https://issues.apache.org/jira/browse/proton-212
Repository: qpid
Description
-------
Provides wrapper functions for sprintf and vsprintf to do the right thing on
overflow and argument checking.
Despite the same names the Windows version only provides the same result in the
sunny day scenario.
Also includes cmake enforcement via a post build step to verify the qpid-proton
library doesn't inadvertently use the non-wrapped versions.
The first try checked for the symbols in the dll, but (some versions of) Visual
Studio can insert vsnprintf callouts in the dll support runtime. This version
checks each foo.obj.
Sorry about the stray tariling whitespace. I'll fix that.
Diffs
-----
http://svn.apache.org/repos/asf/qpid/proton/trunk/proton-c/CMakeLists.txt
1502397
http://svn.apache.org/repos/asf/qpid/proton/trunk/proton-c/src/codec/codec.c
1502397
http://svn.apache.org/repos/asf/qpid/proton/trunk/proton-c/src/engine/engine.c
1502397
http://svn.apache.org/repos/asf/qpid/proton/trunk/proton-c/src/error.c
1502397
http://svn.apache.org/repos/asf/qpid/proton/trunk/proton-c/src/object/object.c
1502397
http://svn.apache.org/repos/asf/qpid/proton/trunk/proton-c/src/platform.h
1502397
http://svn.apache.org/repos/asf/qpid/proton/trunk/proton-c/src/platform.c
1502397
http://svn.apache.org/repos/asf/qpid/proton/trunk/tools/cmake/Modules/WindowsC99CheckDef.cmake
PRE-CREATION
http://svn.apache.org/repos/asf/qpid/proton/trunk/tools/cmake/Modules/WindowsC99SymbolCheck.py
PRE-CREATION
Diff: https://reviews.apache.org/r/10363/diff/
Testing
-------
windows and rhel 6
Thanks,
Cliff Jansen