Dmitry Shachnev

Ambiguous license in test/test_utils/test_math/test__init__.py

input_event_sec/input_event_usec is always available. On 64-bit platforms and old-school 32-bit platforms it expands to: #define input_event_sec time.tv_sec #define input_event_usec time.tv_usec So for these platforms my change is no-op. And on 32-bit platforms with 64-bit time_t it expands to: #define input_event_sec __sec #define input_event_usec __usec See https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/include/uapi/linux/input.h.

Fix build with 64-bit time_t on 32-bit architectures

Fix build with 64-bit time_t on 32-bit architectures

Remove top-level .dirhash before running testUpdateNoChanges

Remove top-level .dirhash before running testUpdateNoChanges

Hi Engelbert! I think your fix is not fully correct. It adds this fragment: self._has_a_table = True if self._has_a_table: # the comment to hint that preprocessor tbl should be called self.head.insert(0, "'\\\" t\n") Here, the condition in if statement will be always true. So every time we encounter a table, we will add a new line to self.head. Maybe you meant something like this instead? if not self._has_a_table: self._has_a_table = True # the comment to hint that preprocessor tbl should be called...

Hi @branden! You could just include the '\" t unconditionally, since tbl(1) will not do any damage to a well-formed man page that doesn't use tables, and the time tbl(1) takes to run has not been noticeable for decades. I would just include it unconditionally, because that would not increase the complexity level of docutils codebase. Of course, Engelbert or Günter may have other opinions, but I don't mind going the other way too. Thank you! -- Dmitry Shachnev