OSSP CVS Repository |
|
Date | Version | Description |
---|---|---|
1.7 | Check-in
[634]:
First cut of the ruthless style adjustments to OSSP XDS:
o adjust source tree to follow OSSP source tree style by heavily combining sources into smaller sets (needs more work when still missing parts are added later) o automatic re-indentation of sources with GNU indent (still needs manual review and adjustments; will follow) These two heavy steps were mostly done automatically with the help of two helper scripts written in Perl. So expect more manual adjustments to follow... By rse. | |
1.6 | Check-in [4293]: Fixed warnings that complained about feeding a "char" into the isxxx() routines, which expect an "int". By simons. (diff) | |
1.5 | Check-in
[591]:
Merged the whole contents of branch "callback-semantic-rewrite" into
the main branch. The changes there include:
- A change of the callback semantics; callbacks do now return XDS_ERR_(UNDER|OVER)FLOW in case the buffer size doesn't fit. Rather than passing the differing byte size through the return code, it is stored in the location used_buffer_size points to -- a new parameter provided to all callbacks. In order to implement this, the framework, all callbacks and most of the test suits had to be adapted. - internal.h now provides the macro xds_check_parameter(), which can be used to verify the contents of function parameters with assert() and if in one line. If assert()s are deactivated, the routine will still return XDS_ERR_INVALID_ARG. Because of this change, internal.h now includes the system header assert.h. This means that this include coulde be removed from almost all modules. - internal.h now provides the macros xds_init_(en|de)coding_engine(). These can be used to comfortably verify a callback's parameters and to verify the buffer size. All engines have been rewritten to use these. By simons. | |
1.4.2.4 | Check-in
[4267] on branch callback-semantic-rewrite:
- Make sure that used_buffer_size is initialized to zero before the
callback is executed.
- Use used_buffer_size to determine the new size of the reallocated buffer in case of an overflow. If used_buffer_size is 0 (default), use 1 instead. By simons. (diff) | |
1.4.2.3 | Check-in [4265] on branch callback-semantic-rewrite: Now that internal.h includes assert.h we don't need to do that anymore. By simons. (diff) | |
1.4.2.2 | Check-in [4252] on branch callback-semantic-rewrite: Rewrote framework for new callback semantics: Callbacks now return an XDS_* code all the time. The length of the buffer used by the callback is returned via used_buffer_size. If the buffer is too small, XDS_ERR_OVERFLOW is returned. By simons. (diff) | |
1.4.2.1 | Check-in [4250] on branch callback-semantic-rewrite: Added xds_check_parameter() macro to short-cut the assert() and if-statements at the begining of every routine. Also rewrote some modules to take advantage of that macro. By simons. (diff) | |
1.4 | Check-in
[4245]:
xds_encode() and xds_decode() had two problems:
- In case any of the callbacks returns with an error, the original xds->buffer_len value must be restored or the buffer is messed up with values from successful callback executions. The routines wouldn't be atomic in that case. - If an encoding callback uses up all buffer space but formatted the value successfully nonetheless, the xds->buffer_len wasn't incremented. Both problems have been fixed. By simons. (diff) | |
1.3 | Check-in [4198]: Passing va_list parameters by value is fine, unless the called routine modifies the parameter list and you expect the change to propagate back to the mother routine. What is what we want. Hence, I changed the prototype of the engine callback to expect a pointer to va_list. By simons. (diff) | |
1.2 | Check-in [4193]: Implemented xds_decode(). By simons. (diff) | |
1.1 | Check-in [4144]: Added function stubs that return errors when called. By simons. |