qpid-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Cliff Jansen" <cliffjan...@gmail.com>
Subject Review Request 17418: va_arg processing on Visual Studio 64bit compiler
Date Mon, 27 Jan 2014 20:32:18 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for qpid, Chug Rolke and Rafael Schloming.

Bugs: PROTON-488

Repository: qpid


See the parent Jira comment description of the bug with the first patch.

This patch achieves the same thing but by forcing all 'z' encodings to be passed as a single
pn_bytes_t struct and "retrieved" as a single pn_bytes_t struct via va_arg().

The other patch is a single line but this one is fairly isolated too.  I slightly prefer this
one because a pn_bytes_t struct is used going the other way (in pn_data_vscan, but by a pointer
to the struct on the stack).  I don't see a performance difference either way.  We just have
to stick to one paradigm everywhere in the code, either "struct in and out", or "separate
size_t and char* in and out".


  http://svn.apache.org/repos/asf/qpid/proton/trunk/proton-c/src/codec/codec.c 1561193 
  http://svn.apache.org/repos/asf/qpid/proton/trunk/proton-c/src/dispatcher/dispatcher.c 1561193

  http://svn.apache.org/repos/asf/qpid/proton/trunk/proton-c/src/sasl/sasl.c 1561193 

Diff: https://reviews.apache.org/r/17418/diff/


64 bit, windows and linux


Cliff Jansen

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message