Collin Funk wrote:
> Yep, you are right. Fixed with the attached patch.

Thanks.

> You could skip the strlen call and check namep[1] == '\0', but I like
> using the tests as a reference when trying to use these modules. So
> lets keep them readable.

Absolutely. Unit tests need to be prepared for
  - commenting out test cases,
  - refining existing test cases,
  - adding new test cases.
Micro-optimizations that don't help readability are inappropriate here.

Bruno




Reply via email to