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

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

Build:
DEFAULT BRANCH: master
Ran 03 May 2020 02:22PM UTC
Files 656
Run time 59s
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: 72.182% (-0.001%) from 72.183%
CFLAGS="--coverage -O2 -g" CXXFLAGS="--coverage -O2 -g" LDFLAGS="--coverage" CONFIGFLAGS="--enable-debug" COVERALLS_PARALLEL=true TEST_SUITES="testinstall" JULIA=yes CONFIGFLAGS="--disable-Werror"

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

222161 of 307781 relevant lines covered (72.18%)

773007.34 hits per line

Source Files on job 11888.13 (CFLAGS="--coverage -O2 -g" CXXFLAGS="--coverage -O2 -g" LDFLAGS="--coverage" CONFIGFLAGS="--enable-debug" COVERALLS_PARALLEL=true TEST_SUITES="testinstall" JULIA=yes CONFIGFLAGS="--disable-Werror")
  • Tree
  • List 0
  • Changed 58
  • Source Changed 2
  • Coverage Changed 58
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Build 3732
  • Travis Job 11888.13
  • edc5339c on github
  • Prev Job for CFLAGS="--coverage -O2 -g" CXXFLAGS="--coverage -O2 -g" LDFLAGS="--coverage" CONFIGFLAGS="--enable-debug" COVERALLS_PARALLEL=true TEST_SUITES="testinstall" JULIA=yes CONFIGFLAGS="--disable-Werror" on master (#11886.13)
  • Next Job for CFLAGS="--coverage -O2 -g" CXXFLAGS="--coverage -O2 -g" LDFLAGS="--coverage" CONFIGFLAGS="--enable-debug" COVERALLS_PARALLEL=true TEST_SUITES="testinstall" JULIA=yes CONFIGFLAGS="--disable-Werror" on master (#11900.13)
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