Escape :: to avoid weird bug #30
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
I think this is the same bug from PR#16. Basically, using the latest
YAML release (1.16), I can run this test script and trigger the bug:
use strict;
use warnings;
use Module::Runtime 'use_module';
use Safe;
my $safe = Safe->new;
use_module('YAML');
print YAML::Dump( { yarr => 1 } );
Resulting int:
mo-pm git:bugfix-issue_24 ❯ perl test2.pl
Undefined subroutine &main::main:: called at /home/ad/nigelg/.plenv/versions/new5203/lib/perl5/site_perl/5.20.3/YAML/Mo.pm line 5.
After rebuilding Mo and running mo-inline on YAML/Mo.pm, I get:
mo-pm git:bugfix-issue_24 ❯ perl test2.pl
---
yarr: 1
veryrusty
added a commit
to veryrusty/mo-pm
that referenced
this pull request
Jul 5, 2016
Issue ingydotnet#24 noted an Undefined subroutine &main::main:: error that needed tests and a patch. ingydotnet#30 has a patch; and here's a simple test to tickle it.
perlpunk
pushed a commit
that referenced
this pull request
Jul 5, 2016
|
Applied and pushed to master. Will release tomorrow. |
|
FWIW, the new Mo code was injected into YAML::Mo and YAML-0.17 was released. |
|
Yay! Thank you. I'm glad to have helped. :D |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I think this is the same bug from PR #16. Basically, using the latest
YAML release (1.16), I can run this test script and trigger the bug:
Resulting in:
After rebuilding Mo and running mo-inline on YAML/Mo.pm, I get: