• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

gap-system / gap / 11888 / 3
4%
master: 4%

Build:
DEFAULT BRANCH: master
Ran 03 May 2020 02:04PM UTC
Files 682
Run time 49s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

03 May 2020 11:49AM UTC coverage: 77.096% (-0.002%) from 77.098%
CFLAGS="--coverage -O2 -g" CXXFLAGS="--coverage -O2 -g" LDFLAGS="--coverage" CONFIGFLAGS="--enable-debug" COVERALLS_PARALLEL=true TEST_SUITES="docomp teststandard" ABI=64 HPCGAP=yes

push

travis-ci

fingolfin
kernel: introduce GAP_TRY/GAP_CATCH helper macros

These are meant to replace uses of TRY_IF_NO_ERROR/CATCH_ERROR outside
of read.c, as those macros really were meant for code in the GAP
scanner/reader; for other code, they are dangerous to use because of the
way they interact with `STATE(NrError)`.

The new macros have the added benefit that they automatically save
and restore the value of `STATE(ReadJmpError)`, so it is safe to use
them recursively.

For now we only switch to GAP_TRY/GAP_CATCH in places were it is clear
that doing so is safe (as in: not touch `STATE(NrError)` is OK). To be
explicit, here are arguments why each change ought to be safe:

- gap.c, reading lib/init.g: if this catches, then we call `Panic`, so
  the value `STATE(NrError)` is irrelevant
- `FuncSERIALIZE_TO_NATIVE_STRING`: did not change `STATE(NrError)`
- `FuncDESERIALIZE_NATIVE_STRING`: did not change `STATE(NrError)`
- `ThreadedInterpreter`: this is the outermost handler for a thread;
  once we catch an error here, we exit this function, which then leads
  to the termination of the thread (for that reason, we also get rid of
  the `ClearError` and `SWITCH_TO_OLD_LVARS` at the end -- they are
  pointless
- `FuncWITH_TARGET_REGION`: did not change `STATE(NrError)`
- `Call0ArgsInNewReader`, `Call1ArgsInNewReader`: these save & restore
  the value of `STATE(NrError)` anyway

243688 of 316082 relevant lines covered (77.1%)

9796767.12 hits per line

Source Files on job 11888.3 (CFLAGS="--coverage -O2 -g" CXXFLAGS="--coverage -O2 -g" LDFLAGS="--coverage" CONFIGFLAGS="--enable-debug" COVERALLS_PARALLEL=true TEST_SUITES="docomp teststandard" ABI=64 HPCGAP=yes)
  • Tree
  • List 0
  • Changed 92
  • Source Changed 4
  • Coverage Changed 92
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Build 3732
  • Travis Job 11888.3
  • edc5339c on github
  • Prev Job for CFLAGS="--coverage -O2 -g" CXXFLAGS="--coverage -O2 -g" LDFLAGS="--coverage" CONFIGFLAGS="--enable-debug" COVERALLS_PARALLEL=true TEST_SUITES="docomp teststandard" ABI=64 HPCGAP=yes on master (#11886.3)
  • Next Job for CFLAGS="--coverage -O2 -g" CXXFLAGS="--coverage -O2 -g" LDFLAGS="--coverage" CONFIGFLAGS="--enable-debug" COVERALLS_PARALLEL=true TEST_SUITES="docomp teststandard" ABI=64 HPCGAP=yes on master (#11900.3)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc