2464
93%

Ran 31 Jan 2020 12:08AM UTC

Jobs 1

Files 20

Run time 3s

Badge

Embed ▾

pending completion

Build # 2464

Build Type

push

travis-ci-com

Committed by

BurntSushi

Commit Message

syntax: fix flag scoping issue

This fixes a rather nasty bug where flags set inside a group were being
applies to expressions outside the group. e.g., In the simplest case,
`((?i)a)b)` would match `aB`, even though the case insensitive flag
_shouldn't_ be applied to `b`.

The issue here was that we were actually going out of our way to reset
the flags when a group is popped only _some_ of the time. Namely, when
flags were set via `(?i:a)b` syntax. Instead, flags should be reset to
their previous state _every_ time a group is popped in the translator.

The fix here is pretty simple. When we open a group, if the group itself
does not have any flags, then we simply record the current state of the
flags instead of trying to replace the current flags. Then, when we pop
the group, we are guaranteed to obtain the old flags, at which point, we
reset them.

Fixes #640

Run Details

3690 of 3979 relevant lines covered (92.74%)

0.93 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
4	2464.4	31 Jan 2020 12:08AM UTC	0	92.74	Travis Job 2464.4

Source Files on build 2464

Detailed source file information is not available for this build.

rust-lang / regex / 2464
93%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 2464

rust-lang / regex / 2464 93%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 2464

rust-lang / regex / 2464
93%

README BADGES
x