Alexander Belopolsky <alexander.belopol...@gmail.com> added the comment:
Not directly related to the issue at hand, but I've noticed that while readline.get_current_history_length() is tested in the unittests, readline.get_history_length() is not. Attached patch adds tests for reading and writing history files. Also, I find it confusing that readline module static variable _history_length is named almost the same as readline library exported variable history_length. Maybe _history_length could be renamed to max_history_file_length. ---------- Added file: http://bugs.python.org/file16463/issue8065-tests.diff _______________________________________ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue8065> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
