Patch: static_const nitpicky patch
Paul Lindner
lindner at inuus.com
Tue Mar 6 21:17:17 UTC 2007
On Mon, Mar 05, 2007 at 02:55:44PM -0800, Steven Grimm wrote:
> No objections here! Paul, if you have time to merge this into the
> multithreaded branch as well as checking it in to the single-threaded
> code base, that'd be great.
Okay, patches committed and merged into the multithreaded branch.
Also there are more changes since my last email. I found a 0-size
mallocs and cleaned up a few more things.
All tests passing. It will be interesting to see if any of these
changes help with performance by enabling better compiler
optimization.
Here's what I put in the ChangeLog:
2007-03-05 Paul Lindner <lindner at inuus.com>
* Fix a number of places where (s)printf calls were using unsigned
or signed formats that did not match their arguments.
* Add support for stdbool.h and stdint.h to use the bool and
uint8_t types.
* Major refactoring - move API calls for assoc/items/slabs to
their own individual header files. Add apropriate const and
static declarations as appropriate.
* Avoid type-punning. Do a more efficient realloc inside the
conn_shrink routine.
* Fix overflow bug where uninitialized access to slabclass caused
size-0 mallocs during slab preallocation.
* Use EXIT_SUCCESS/EXIT_FAILURE constants.
* Convert some sprintf calls to snprintf to protect against
buffer overflows.
* Explicitly compare against NULL or zero in many places.
--
Paul Lindner ||||| | | | | | | | | |
lindner at inuus.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://lists.danga.com/pipermail/memcached/attachments/20070306/38052d71/attachment.pgp
More information about the memcached
mailing list