Skip to content
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

Here-doc missing terminator message is not null-clean #15479

Open
p5pRT opened this issue Jul 26, 2016 · 3 comments
Open

Here-doc missing terminator message is not null-clean #15479

p5pRT opened this issue Jul 26, 2016 · 3 comments

Comments

@p5pRT
Copy link

p5pRT commented Jul 26, 2016

Migrated from rt.perl.org#128743 (status was 'open')

Searchable as RT128743$

@p5pRT
Copy link
Author

p5pRT commented Jul 26, 2016

From @cpansprout

$ perl -e 'print qq|print <<"e\0e"|'>foo
$ perl foo
Can't find string terminator "e" anywhere before EOF at foo line 1.

It should not be chopping off part of the terminator.

This is a bit pedantic, but passing fixing this, which would entail passing the length to toke.c​:S_missingterm, would allow us to eliminate a temporary SV and simplify S_missingterm significantly.

(I am holding off on fixing this, as it may conflict with other here-doc work going on right now. In fact, if someone wants to get to this before I do, go ahead. This falls in the ‘easy’ category.)

--

Father Chrysostomos

@p5pRT
Copy link
Author

p5pRT commented Jul 26, 2016

From @cpansprout

On Tue Jul 26 13​:51​:06 2016, sprout wrote​:

$ perl -e 'print qq|print <<"e\0e"|'>foo
$ perl foo
Can't find string terminator "e" anywhere before EOF at foo line 1.

It should not be chopping off part of the terminator.

This is a bit pedantic, but passing

s/passing //;

fixing this, which would entail
passing the length to toke.c​:S_missingterm, would allow us to
eliminate a temporary SV and simplify S_missingterm significantly.

(I am holding off on fixing this, as it may conflict with other here-
doc work going on right now. In fact, if someone wants to get to this
before I do, go ahead. This falls in the ‘easy’ category.)

--

Father Chrysostomos

@p5pRT
Copy link
Author

p5pRT commented Jul 26, 2016

The RT System itself - Status changed from 'new' to 'open'

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

No branches or pull requests

2 participants