New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix regex roundtrip (#69) #70

Merged
merged 2 commits into from Dec 16, 2017

Conversation

Projects
None yet
1 participant
@perlpunk
Collaborator

perlpunk commented Dec 8, 2017

This is a quick fix for issue #69

@perlpunk perlpunk referenced this pull request Dec 8, 2017

Closed

round trip grows regexp #69

@perlpunk perlpunk requested a review from ingydotnet Dec 8, 2017

@perlpunk

This comment has been minimized.

Show comment
Hide comment
@perlpunk

perlpunk Dec 16, 2017

Collaborator

As requested by @ingydotnet I added a test for this.
There wasn't a test for regex roundtrips at all before, so hopefully this will help us to see changes in future perl versions.

Collaborator

perlpunk commented Dec 16, 2017

As requested by @ingydotnet I added a test for this.
There wasn't a test for regex roundtrips at all before, so hopefully this will help us to see changes in future perl versions.

@perlpunk perlpunk merged commit c1e1e43 into master Dec 16, 2017

1 check passed

continuous-integration/travis-ci/push The Travis CI build passed
Details
@perlpunk

This comment has been minimized.

Show comment
Hide comment
@perlpunk

perlpunk Dec 16, 2017

Collaborator

Applied

Collaborator

perlpunk commented Dec 16, 2017

Applied

@perlpunk perlpunk deleted the Issue69-RegexRoundtrip branch Dec 20, 2017

gentoo-bot pushed a commit to gentoo/gentoo that referenced this pull request Apr 3, 2018

dev-perl/YAML-LibYAML: Bump to version 0.690.0
Upstream:
- (Security) Add $LoadBlessed option[1] to run on/off loading objects
- Fix regex roundtrip [2]
- Fix loading of many regex classes [3]
- Support tags !!str, !!map and !!seq [4]
- Support JSON::PP::Boolean via $YAML::XS::Boolean [5]
- Fix Dump modifying inspected values [6]

[1]
Bug: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=861958
Bug: ingydotnet/yaml-libyaml-pm#45
Bug: ingydotnet/yaml-libyaml-pm#73
Bug: ingydotnet/yaml-libyaml-pm#74
[2]
Bug: ingydotnet/yaml-libyaml-pm#69
Bug: ingydotnet/yaml-libyaml-pm#70
[3]
Bug: ingydotnet/yaml-libyaml-pm#64
Bug: ingydotnet/yaml-libyaml-pm#71
[4]
Bug: ingydotnet/yaml-libyaml-pm#67
[5]
Bug: ingydotnet/yaml-libyaml-pm#66
[6]
Bug: ingydotnet/yaml-libyaml-pm#32
Bug: ingydotnet/yaml-libyaml-pm#55
Package-Manager: Portage-2.3.24, Repoman-2.3.6

madmartin added a commit to madmartin/gentoo that referenced this pull request Apr 4, 2018

dev-perl/YAML-LibYAML: Bump to version 0.690.0
Upstream:
- (Security) Add $LoadBlessed option[1] to run on/off loading objects
- Fix regex roundtrip [2]
- Fix loading of many regex classes [3]
- Support tags !!str, !!map and !!seq [4]
- Support JSON::PP::Boolean via $YAML::XS::Boolean [5]
- Fix Dump modifying inspected values [6]

[1]
Bug: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=861958
Bug: ingydotnet/yaml-libyaml-pm#45
Bug: ingydotnet/yaml-libyaml-pm#73
Bug: ingydotnet/yaml-libyaml-pm#74
[2]
Bug: ingydotnet/yaml-libyaml-pm#69
Bug: ingydotnet/yaml-libyaml-pm#70
[3]
Bug: ingydotnet/yaml-libyaml-pm#64
Bug: ingydotnet/yaml-libyaml-pm#71
[4]
Bug: ingydotnet/yaml-libyaml-pm#67
[5]
Bug: ingydotnet/yaml-libyaml-pm#66
[6]
Bug: ingydotnet/yaml-libyaml-pm#32
Bug: ingydotnet/yaml-libyaml-pm#55
Package-Manager: Portage-2.3.24, Repoman-2.3.6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment