_load_string: Save $@ so we don't generate misleading error messages if ... #31

Merged
merged 1 commit into from Jun 11, 2014

Conversation

Projects
None yet
2 participants
@hillu
Contributor

hillu commented Jun 11, 2014

...$@ is clobbered. This fixes #30.

@karenetheridge

This comment has been minimized.

Show comment Hide comment
@karenetheridge

karenetheridge Jun 11, 2014

Member

Unfortunately, travis shows test failures, e.g.:

t/01_api.t .............. Global symbol "$err" requires explicit package name at /home/travis/build/Perl-Toolchain-Gang/YAML-Tiny/lib/YAML/Tiny.pm line 297.
Member

karenetheridge commented Jun 11, 2014

Unfortunately, travis shows test failures, e.g.:

t/01_api.t .............. Global symbol "$err" requires explicit package name at /home/travis/build/Perl-Toolchain-Gang/YAML-Tiny/lib/YAML/Tiny.pm line 297.
@hillu

This comment has been minimized.

Show comment Hide comment
@hillu

hillu Jun 11, 2014

Contributor
  • Karen Etheridge:

Unfortunately, travis shows test failures, e.g.: [...]

How embarrassing. :-)

I have updated my commit.

Cheers,
-Hilko

Contributor

hillu commented Jun 11, 2014

  • Karen Etheridge:

Unfortunately, travis shows test failures, e.g.: [...]

How embarrassing. :-)

I have updated my commit.

Cheers,
-Hilko

karenetheridge added a commit that referenced this pull request Jun 11, 2014

Merge pull request #31 from hillu/errors
_load_string: Save $@ so we don't generate misleading error messages if ...

@karenetheridge karenetheridge merged commit 551ad71 into Perl-Toolchain-Gang:devel Jun 11, 2014

1 check passed

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

This comment has been minimized.

Show comment Hide comment
@karenetheridge

karenetheridge Jun 11, 2014

Member

released in 1.63.
and thank you!

Member

karenetheridge commented Jun 11, 2014

released in 1.63.
and thank you!

@hillu hillu deleted the hillu:errors branch Jun 12, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment