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

Hpcgap lib merge 1 #1327

Merged
merged 6 commits into from
May 16, 2017
Merged

Conversation

ChrisJefferson
Copy link
Contributor

This patch merges some files in hpcgap/lib. It also makes a wide selection of strings which GAP creates immutable -- in many cases if these strings were edited bad things would happen, so making them immutable shouldn't break anything.

@codecov
Copy link

codecov bot commented May 10, 2017

Codecov Report

Merging #1327 into master will decrease coverage by 0.02%.
The diff coverage is 89.65%.

@@            Coverage Diff             @@
##           master    #1327      +/-   ##
==========================================
- Coverage   61.54%   61.52%   -0.03%     
==========================================
  Files        1038     1035       -3     
  Lines      358268   356356    -1912     
  Branches    14291    14293       +2     
==========================================
- Hits       220492   219234    -1258     
+ Misses     134120   133461     -659     
- Partials     3656     3661       +5
Impacted Files Coverage Δ
lib/stbc.gi 87.83% <ø> (ø) ⬆️
lib/meatauto.gi 94.43% <ø> (ø) ⬆️
hpcgap/lib/meataxe.gi 74.6% <ø> (ø) ⬆️
hpcgap/lib/meatauto.gi 94.49% <ø> (ø) ⬆️
lib/overload.g 31.81% <ø> (ø) ⬆️
lib/meataxe.gi 82.29% <ø> (ø) ⬆️
src/opers.c 60.16% <100%> (ø) ⬆️
src/scanner.c 70.06% <100%> (ø) ⬆️
src/code.c 86.97% <100%> (ø) ⬆️
src/records.c 66.99% <100%> (ø) ⬆️
... and 8 more

@ChrisJefferson
Copy link
Contributor Author

This has been rebased on top of #1333 (MakeImm and Imm), and that is used to do some more simple merging.

@ChrisJefferson
Copy link
Contributor Author

It also requires #1334 as well (sorry)

@ChrisJefferson ChrisJefferson force-pushed the hpcgap-lib-merge-1 branch 2 times, most recently from 7b633bc to 15e84dd Compare May 14, 2017 06:21
@fingolfin
Copy link
Member

OK, #1334 is in -- please rebase this PR so that we can merge it.

@fingolfin
Copy link
Member

Or wait, I'll just do it

@fingolfin
Copy link
Member

Ah well, seems we both did it... also fine :-)

@fingolfin fingolfin merged commit 9d46d0c into gap-system:master May 16, 2017
@ChrisJefferson ChrisJefferson deleted the hpcgap-lib-merge-1 branch May 16, 2017 20:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants