This is more of a documentation issue than a code issue.  To be mathematically precise, the text and error message should read "a non-negative value".  Alternatively the text and error could be changed to report that timeout may not be negative, which would probably be clearer.  (Note that timeout is a float value, and thus the proposed code change would only break code, it would not change the functionality in any significant way.)