[ https://issues.apache.org/jira/browse/PROTON-2248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17312592#comment-17312592 ]
ASF subversion and git services commented on PROTON-2248: --------------------------------------------------------- Commit 5d3473dd484090590863c0d87dea4ea91bba87dc in qpid-proton's branch refs/heads/main from Andrew Stitcher [ https://gitbox.apache.org/repos/asf?p=qpid-proton.git;h=5d3473d ] PROTON-2248: [c] Some code doc fixes > [c] Codec pn_data_vfill() method has symbol and string types interchanged > ------------------------------------------------------------------------- > > Key: PROTON-2248 > URL: https://issues.apache.org/jira/browse/PROTON-2248 > Project: Qpid Proton > Issue Type: Bug > Components: proton-c > Reporter: Kim van der Riet > Priority: Major > > TheĀ *{{pn_data_vfill()}}* method in codec.c:571 has the symbol and string > types interchanged. The function documentation states: > {noformat} > /* Format codes: > code: AMQP-type (arguments) > ... > S: symbol (char*) > s: string (char*) > ... > */ > {noformat} > but uses these the other way around, ie {{s}} for symbol and {{S}} for string: > {noformat} > if (code == 'S') { > err = pn_data_put_string(data, pn_bytes(size, start)); > } else { > err = pn_data_put_symbol(data, pn_bytes(size, start)); > } > {noformat} > Guidance is needed on whether to fix the reversal and keep the stated codes, > or reverse the codes. -- This message was sent by Atlassian Jira (v8.3.4#803005) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org