Remove unused variables that everyone agrees on.
Fix the scope of the child variable.
cherokee_version_add_simple should return ret, instead of ret_ok.
I think one byte might be a good estimate here. Please confirm.
p seems the location after the sum, it is not used. Please confirm.
Make explict retcode can only be used when ret_ok is returned.
At this moment in time we are not doing anything with id, but for future reference I would like to keep it.
We don't use version at this moment at all.
Solve a potential memory leak, and reuse one buffer for the task.
Based on the suggestions by @kinnison I have changed these.
Fix an unused return value in buffer.c as well, while our other branch probably needs a merge now
Change the scope of the variables, and initialise the item.
Just an extra safe-guard to check for NULL
Initialise xsendfile_len with 0;
Initialise len = 0;
Merge pull request #1084 from cherokee/unused_but_set_variables

This merge cleans up virtually all compiler warnings on unused but set variables.