address comments #185

Job	Run time
downstream	26m 23s
clang-sanitizers (,thread)	1m 9s
linux-compiler-compat (clang-3)	1m 6s
linux-compat (manylinux1-x64)	42s
clang-sanitizers (,address,undefined)	1m 21s
linux-compiler-compat (clang-6)	1m 8s
linux-compat (manylinux1-x86)	51s
linux-compiler-compat (clang-8)	1m 40s
linux-compat (manylinux2014-x64)	1m 24s
linux-shared-libs	48s
linux-compiler-compat (clang-9)	1m 22s
linux-compat (manylinux2014-x86)	47s
linux-compiler-compat (clang-10)	1m 10s
osx	1m 26s
linux-compat (fedora-34-x64)	45s
linux-compiler-compat (clang-11)	1m 14s
linux-compat (opensuse-leap)	50s
linux-compiler-compat (gcc-4.8)	45s
windows	1m 43s
linux-compat (rhel8-x64)	37s
linux-compiler-compat (gcc-5)	53s
linux-compat (al2-x64)	42s
windows-app-verifier	2m 3s
linux-compiler-compat (gcc-6)	1m 0s
linux-compiler-compat (gcc-7)	51s
windows-shared-libs	2m 6s
linux-compiler-compat (gcc-8)	56s
windows-vc14 (x86)	1m 40s
windows-vc14 (x64)	1m 35s
	58m 57s

Provide feedback