Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Detect and suppress an additional gmtime() warning in test_util.c #915

Merged
merged 2 commits into from Apr 10, 2019

Conversation

Labels
None yet
Projects
None yet
3 participants
@nmathewson
Copy link
Contributor

@nmathewson nmathewson commented Apr 8, 2019

Fixes bug 29922; bugfix on 0.2.9.3-alpha when we tried to capture
all these warnings. No need to backport any farther than 0.3.5,
though -- these warnings don't cause test failures before then.

This one was tricky to find because apparently it only happened on
some windows builds.

Fixes bug 29922; bugfix on 0.2.9.3-alpha when we tried to capture
all these warnings.  No need to backport any farther than 0.3.5,
though -- these warnings don't cause test failures before then.

This one was tricky to find because apparently it only happened on
_some_ windows builds.
@coveralls
Copy link

@coveralls coveralls commented Apr 8, 2019

Pull Request Test Coverage Report for Build 4653

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 13 unchanged lines in 3 files lost coverage.
  • Overall coverage decreased (-0.02%) to 60.303%

Files with Coverage Reduction New Missed Lines %
src/feature/hs/hs_common.c 1 83.28%
src/feature/dirauth/shared_random.c 3 85.64%
src/feature/dirauth/dirvote.c 9 64.79%
Totals Coverage Status
Change from base Build 4618: -0.02%
Covered Lines: 42857
Relevant Lines: 71070

💛 - Coveralls

@torproject-pusher torproject-pusher merged commit 9f3f999 into torproject:maint-0.3.5 Apr 10, 2019
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment