Skip to content

Conversation

@zerothi
Copy link
Contributor

@zerothi zerothi commented Jan 24, 2018

Fixes a possible call to laruv routines when N < 1.

See #182.

Fixes a possible call to laruv routines when N < 1.

Signed-off-by: Nick Papior <[email protected]>
@codecov
Copy link

codecov bot commented Jan 24, 2018

Codecov Report

Merging #225 into master will decrease coverage by <.01%.
The diff coverage is 50%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #225      +/-   ##
==========================================
- Coverage   82.67%   82.67%   -0.01%     
==========================================
  Files        1841     1841              
  Lines      178570   178574       +4     
==========================================
+ Hits       147627   147629       +2     
- Misses      30943    30945       +2
Impacted Files Coverage Δ
SRC/slaruv.f 96.87% <50%> (-3.13%) ⬇️
SRC/dlaruv.f 81.25% <50%> (-2.09%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6fb44f1...645c505. Read the comment docs.

@zerothi zerothi closed this Sep 7, 2018
@zerothi zerothi deleted the laruv branch September 7, 2018 07:14
@langou
Copy link
Contributor

langou commented Sep 7, 2018

Hi Nick. Thanks for looking into this! No need for a commit, I guess then. Cheers, Julien.

@julielangou julielangou added this to the LAPACK 3.9.0 milestone Nov 16, 2019
@martin-frbg
Copy link
Collaborator

Anybody recall what happened here ? I see that a fix was prepared but closed rather than merged, but I see nothing in this conversation (or the original issue) that explains the "no need for a commit, I guess"

weslleyspereira added a commit to weslleyspereira/lapack that referenced this pull request May 24, 2023
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.

4 participants