8
99%
master: 99%

Ran 20 Sep 2016 12:17PM UTC

Files 5

Run time 0s

Badge

Embed ▾

Committed 20 Sep 2016 12:07PM UTC coverage: 99.538% (+0.01%) from 99.526%

Job # 210.8

Build Type

push

travis-ci

Committed by

marshallward

Commit Message

Patch bugfix for vector of different size

Previously, the `patch` function only worked if the namelist and its
patch used vectors of the same size.  If the patch vector was smaller,
then the trailing values would still be retained in the patched namelist
file.

This patch fixes this issue by making the following changes:

1. Check if we are beyond the range of the patched vector; if so, then
   enable a new flag (`patch_skip`) in `update_tokens`.

2. Store patched whitespace and comments into a saved variable, rather
   than directly write to the patched file during token sweeping

3. When the new flag is on, check if the next value is either an old
   value or a new variable name.  This is done by checking if the next
   token is '='.

We also had to wrap one of the `next` calls around a StopIteration
exception check, and force an early write to patch if this occurs.  This
is because we rely on StopIteration to terminate the token sweep.
(Probably dumb but whatever...)

This was a mess to code, and probably indicates that f90nml has a lot of
crap that needs to be refactored ASAP.

Run Details

646 of 649 relevant lines covered (99.54%)

1.0 hits per line

marshallward / f90nml / 210 / 8
99%
master: 99%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 210.8

marshallward / f90nml / 210 / 8 99% master: 99%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 210.8

marshallward / f90nml / 210 / 8
99%
master: 99%

README BADGES
x