<div>is this checked in svn or do i need to apply this patch?</div>
<div>thanks<br><br> </div>
<div><span class="gmail_quote">On 4/5/07, <b class="gmail_sendername">Tim Yardley</b> <<a href="mailto:liquid@haveheart.com">liquid@haveheart.com</a>> wrote:</span>
<blockquote class="gmail_quote" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: #ccc 1px solid">All;<br><br>Attached is a patch against the multithreaded branch that cleans up spacing<br>issues bringing the codebase to a hopefully consistent style. It also
<br>addresses -Wall warnings, some null checks (a TODO item iirc), and adds<br>asserts at the top of each function for any use of conn *c without checking<br>to see if c is NULL first.<br><br>Hopefully those asserts are not necessary as we all write perfect code...
<br>Right? Anyway, that conn object is used and passed around enough that it<br>should be checked.<br><br>Note that there are a couple other areas that could be addressed on an<br>optimization level. I addressed a few like multiple assignments to a
<br>variable where not needed, elimination of unused variables, value assignment<br>at initiation where it made sense, #ifdef where a function wont be used,<br>etc.<br><br>Other things that I noticed that could be addressed but require consensus:
<br>- optimization of the switch statements to leverage jump tables at<br>compilation<br>- focusing on popular default cases at the top of the switch to reduce<br>comparisons<br>- restructing switch statements if necessary to support that under
<br>different compilers (nesting the switches)<br>- visiting the usage of char vs int (byte alignment, powers of two<br>padding, etc)<br>- inlining some of the smaller functions (candidates would include some of<br>the item funtions, etc)
<br><br><br>A couple of the malloc cases in there looked potentially troubling<br>(arbitrary values chosen, and some potential other misallocations) but I<br>didn't spend the time to fully analyze them. Does anyone have access to one
<br>of the commercial source audit applications that would like to run memcached<br>through it? If not, would anyone be interested in me persuing that avenue<br>with some of the vendors that have offered to run their commercial
<br>applications on opensource software?<br><br>Feedback welcome.<br><br>/tmy<br><br></blockquote></div><br>