hibtc / MAD-X

67%

master: 54%

LAST BUILD ON BRANCH cleanup_command_par

branch: cleanup_command_par

CHANGE BRANCH

x

Reset

• cleanup_command_par
• bugfix-select
• cd-command
• cleanup
• cleanup2
• cmake-cxx11
• cmake-nptsa
• cmake-static-archive
• coverage-ci
• doc-shared-drift
• find_table
• fix-amux-inconsistency
• fix-aperture-plot
• fix-cmake28
• fix-delete-sequence
• fix-grow-empty-array
• fix-latexuguide
• fix-line-parsing
• fix-mvar
• fix-parsing-logical-after-string
• fix-select
• fix-select-class-variable
• fixup
• fortran-coverage
• gc-8.0.2
• hotfix
• intel-cmake
• intel-cmake-rel
• manylinux
• master
• multipole-kick
• nmass
• opt_X11
• per-constraint-weights
• read_table
• sectorfile
• shared-drifts
• srotation_angle_survey
• tests-interpolate
• tmbend-interpolate-k0
• travis-maint
• travis-test-plot
• various-fixes

Build # 65

Build Type

push

travis-ci

Committed by coldfix

Commit Message

Cleanup uses of command_parameter::call_def

This commit is mostly an equivalence transformation locally and
therefore has to introduce another set of accessor functions.

However, on a longer time scale, these distinct accessors should be
unified into one single canonical function to extract values from
commands, that always uses `cp->string`. This can be implemented by
fixing the command parameter decoding stage (i.e. construction time of
command):

    string              ->  cp->string = user-def
    bare word or `true` ->  cp->string = cp->call_def->string or DEFAULT
    `false`             ->  cp->string = NULL
    (unspecified)       ->  cp->string = cp->default

Furthermore, move default values from individual modules to mad_dict.

Run Details

11739 of 17558 relevant lines covered (66.86%)

842234.05 hits per line