Lars Schneider <larsxschneider@xxxxxxxxx> writes: > Although the line is unnecessary, I felt it is safer/easier to > understand and maintain. Since both of you tripped over it, I will > remove it though. I didn't actually trip over it. It made it look as if the coder didn't understand what the code is doing to have that extra line. > I don't think HT makes too much sense. However, isspace() is nice > and I will use it. Being more permissive on the inputs should hurt. You are being incoherent in these three sentences. If you want to be more strict and only allow SP, then isspace() is already too broad, as it does allow HT (and even CR and LF). I do think HT makes just as much sense as SP, though, so if you use isspace(), that would be perfectly fine.