-
Notifications
You must be signed in to change notification settings - Fork 668
examples/xmlrpc: Fix calls buffers size. #3027
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
examples/xmlrpc: Fix calls buffers size. #3027
Conversation
|
Formatting issues, sorry, will fix after a short sleep mode (6AM here) :-) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@acassis: Please fix coding style issues
Done :-)
Mixed case identifiers were here before and seem to be part of XMLRPC implementation? See https://xmlrpc-c.sourceforge.io/doc/libxmlrpc_client++.html. These are internal variables so we can change them with no impact I guess :-)
9b15b51 to
439a654
Compare
|
@cederom suggestion: lastCommand -> lastcmd; curState -> curstate |
* examples/xmlrpc/calls.c used 80 bytes call buffers. * update buffers to CONFIG_XMLRPC_STRINGSIZE+1 that is build time configurable. * this keeps buffers size coherent with configuration. * updated internal variable names to pass lint checks. Signed-off-by: Tomasz 'CeDeROM' CEDRO <tomek@cedro.info>
439a654 to
3b03e73
Compare
|
Variables updated: |
Summary
Impact
Testing
Please help in runtime testing :-)