gap-packages / NautyTracesInterface

Interface from GAP to Nauty
https://gap-packages.github.io/NautyTracesInterface/
Other
6 stars 6 forks source link

Fix crash for empty graphs #43

Closed fingolfin closed 3 years ago

fingolfin commented 3 years ago

The EMPTYGRAPH writes out-of-bounds for empty graphs.

Fixes #17

codecov[bot] commented 3 years ago

Codecov Report

Merging #43 (11c0ec7) into master (6d81392) will increase coverage by 1.82%. The diff coverage is 60.00%.

@@            Coverage Diff             @@
##           master      #43      +/-   ##
==========================================
+ Coverage   27.90%   29.72%   +1.82%     
==========================================
  Files          14       14              
  Lines        2835     2836       +1     
==========================================
+ Hits          791      843      +52     
+ Misses       2044     1993      -51     
Impacted Files Coverage Δ
src/NautyTracesInterface.c 57.93% <60.00%> (+0.29%) :arrow_up:
nauty27r1/naugraph.c 56.61% <0.00%> (+0.73%) :arrow_up:
nauty27r1/nautinv.c 1.72% <0.00%> (+1.72%) :arrow_up:
nauty27r1/nauty.c 59.13% <0.00%> (+4.94%) :arrow_up:
nauty27r1/nautil.c 45.26% <0.00%> (+6.84%) :arrow_up:
fingolfin commented 3 years ago

This only fixes the crash on my Mac. Ah well, no problem to create alternates for the special case of an empty graph.