tree 0ca2aa8d32665e6cb1734f711e2d88ce08f54110
parent f1714c3fc43866f4f0dcdafb94beff9d1eef263c
author Nicholas Junge <nicho.junge@gmail.com> 1791207321 +0200
committer GitHub <noreply@github.com> 1791207321 +0100
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsFcBAABCAAQBQJqw6eZCRC1aQ7uu5UhlAAAo6UQAGsFH2NeZeq8vWZvf63Eg/Cw
 wTzwSmSEo0mbT8fyxrhWd/HUsJ0ItHjW3GjUD1UvPhFv0Lwc3r/X1nAdLG829IpO
 8gxg/ob8OCLW/Ram+dvSsDIBGxuqC2iU50HmwIIFGe249UPuaBbhPKAP/9LjilMX
 stHZTHdYRDIUuXOfBrnwgYRzfz5JXdDtwnmDfGwd/EgnbDwGqPIi8t86fVXfuBAK
 4eyuEIojEfjpwyZYxr8uZJVtYZa5OX49HMhM7IfhSgtihvV9AgSzGLRaPNU7Ob/d
 fpE3laOUOP4u9CDS1iGXnCcmJos+a79VlCz3W5F3n8zYSRinr2vO+tZ/KXq12gNm
 fN047ntZARu7ENDNnrIFTwRVvFWaPFl3PZVMc65QB6dNtE/aea5oo+o3NUQVyYN0
 wKa9LkJ4T86hkPaDUx6m/vpH8QZ1DJa2Z1de+M0fhphNF2ba/BWGRUiQc5rJB23W
 gqxQeQlhQk8oxsUdY9+eD72zHd0SoZ4X7tQ94ZrStBlZsqxSvVJNXkOKNlRldZYr
 UENEHO2kQHVE1+hJ15H89WnLykRJC/BP3ZYswlAA3QB95umAnVXJpuK7GYLQA1BR
 GI+O9tjNeMIOCw81r9LlhUgu29ljM68Bzz7c0TOQSPwy71y91dalBsjIwYopTr/s
 yuFslebN8oawrnbv5m2r
 =RhU+
 -----END PGP SIGNATURE-----
 

fix(timing): Do not resume timing in pause scope on skipped benchmarks (#2318)

Currently, skipping a benchmark with an error from a `ScopedPauseTiming` fails
with GB built in debug mode, since `ResumeTiming()` is called outside of a benchmark
loop.

This mirrors the logic in `State::FinishKeepRunning()`, which pauses only if the
benchmark was not skipped. Leaving the timer paused is preferable to resuming after
a skip, which could then also corrupt timings.