dst->length is set again afterwards.
Typo.
TRI_CreateRecursiveDirectory returns an int, but the function prototype says it returns a bool (the return code is not checked in the source code, so it is not a problem).
In the function TRI_StringInt8InPlace, when attr == INT8_MIN, it was copying only 4 bytes instead of 5 and it was returning 3 instead of 4.
Merge branch 'experiment' of https://github.com/guidoreina/ArangoDB into experiment
In the function: RemoveAllLockedFiles(void), if we call TRI_RemoveVectorString() and TRI_RemoveVector() the next elements in the vector will be memmoved to the left. The next iteration of the loop will check the element at position 1 (but we are skipping the new element at position 0), so these calls have been removed.
The other alternative would be to have a loop: while (FileNames._length > 0) and take always the element at position 0. But this has the disadvantage that we are performing memmove() when later the vectors will be freed anyway.
In the function: TRI_LocateBinaryPath(), "binaryPath" was duplicated from "dir" and then "dir" freed. Better just to assign to "binaryPath" the result of TRI_Dirname().
Merge pull request #580 from guidoreina/experiment

Small improvements.