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

uncolor fix #5

Open
wants to merge 3 commits into
base: master
from

Conversation

Projects
None yet
1 participant
@Varadinsky

Varadinsky commented Sep 28, 2015

uncolor stores all passed arguments in its lexical @DaTa array.

If the first element of @DaTa is arraref, script will dereference it and push its data to the end of @DaTa array, but will keep the arrayref as the first argument, which leads to bad output.

This fix makes sure that this arrayref is not kept in the @DaTa.

Varadinsky added some commits Sep 28, 2015

Added more tests for uncolor()
Testing the case when arrayref is passed as the first argument to uncolor,
in list and scalar context.
uncolor() fix - when arrayref is passed as the first argument.
uncolor stores all passed arguments in its lexical @DaTa array.

If the first element of @DaTa is arraref, script will dereference it
and push its data to the end of @DaTa array, but will keep the arrayref
as the first argument, which leads to bad output.

This fix makes sure that this arrayref is not kept in the @DaTa.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment