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

Some more finite field tweaks from from PR #2997 #3137

Merged
merged 3 commits into from
Jan 7, 2019

Conversation

fingolfin
Copy link
Member

This contains some more bits from PR #2997, which again I hope are uncontroversial.

I will shortly update PR #2997 with a version rebased on this PR.

@fingolfin fingolfin added topic: kernel release notes: not needed PRs introducing changes that are wholly irrelevant to the release notes labels Dec 21, 2018
@codecov
Copy link

codecov bot commented Dec 21, 2018

Codecov Report

Merging #3137 into master will decrease coverage by <.01%.
The diff coverage is 92%.

@@            Coverage Diff             @@
##           master    #3137      +/-   ##
==========================================
- Coverage   83.67%   83.67%   -0.01%     
==========================================
  Files         687      687              
  Lines      336648   336643       -5     
==========================================
- Hits       281695   281687       -8     
- Misses      54953    54956       +3
Impacted Files Coverage Δ
src/finfield.h 98.3% <ø> (ø) ⬆️
src/finfield.c 97.01% <92%> (+0.15%) ⬆️
src/iostream.c 62.35% <0%> (-1.15%) ⬇️
lib/streams.gi 70.24% <0%> (-0.22%) ⬇️
src/streams.c 72.4% <0%> (+0.11%) ⬆️

@coveralls
Copy link

coveralls commented Dec 21, 2018

Coverage Status

Coverage remained the same at 82.829% when pulling 618bdd7 on fingolfin:mh/ffe-more into 6e079af on gap-system:master.

@gap-system gap-system deleted a comment from codecov bot Dec 27, 2018
@fingolfin fingolfin force-pushed the mh/ffe-more branch 2 times, most recently from 546d48c to 7f0b531 Compare January 6, 2019 22:54
@fingolfin fingolfin merged commit 823bd43 into gap-system:master Jan 7, 2019
@fingolfin fingolfin deleted the mh/ffe-more branch January 7, 2019 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release notes: not needed PRs introducing changes that are wholly irrelevant to the release notes topic: kernel
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants