Hi all,
First of all, I noticed that throughout the codebase there is a mix of spaces and tabs. This makes it very difficult to produce out-of-tree patches, since most of the editors will replace tabs with spaces. Is there a reason behind it?
The second question is about the bounds check. In the current implementation it is common to first access the provided pointer, and only after that check if it is valid or not (e.g. the __bound_str* functions). Doesn't it defy the whole purpose of the functions? What is the intended usecase of this feature?
BW,
George