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

vteFeedChild #256

Closed
hanspr opened this issue Nov 1, 2019 · 6 comments
Closed

vteFeedChild #256

hanspr opened this issue Nov 1, 2019 · 6 comments

Comments

@hanspr
Copy link
Contributor

hanspr commented Nov 1, 2019

@egmontkob

I do not know how to reach you, but after a lot of testing I came to the conclusion that this sub should be rewritten like this, otherwise you get the array reference on the terminal instead of the text. I think that the perl library handles the passing of arguments inside.

imagen

I have tested and it works fine like this, the other way around does not work.

But I would like to check with you.

@hanspr
Copy link
Contributor Author

hanspr commented Nov 1, 2019

Will close, already got my self the answer, and created the PR to fix.

@hanspr hanspr closed this as completed Nov 1, 2019
@egmontkob
Copy link

This is Perl magic, and I'm not that familiar with Perl to comment on this. However, I'm curious:

  • What's the bug that it fixes?
  • Have you tested with accented letters?
  • Shouldn't _vteFeed() and _vteFeedChildBinary also receive this change?

@hanspr
Copy link
Contributor Author

hanspr commented Nov 1, 2019

You can read the original bug report on #205

The bug is this one

imagen

You type an "a" on one terminal, and using vteFeedChild should be passed to the second terminal as an "a", but as you see in the image, it arrives as ARRAY(0x......), which is the reference and not the letter.

After changing it like that, you get this : type "ll" and is copyed correctly to the second terminal.

imagen

With respect to accented letters, huuh, it has issues in did.

imagen

This complicates things a little bit more, then, maybe using vteFeedChildBinary?

Now, I'll have to do more testing.

@hanspr
Copy link
Contributor Author

hanspr commented Nov 1, 2019

fiiiixed !

imagen

I will have to send another PR, this guys are going to hallucinate me.

imagen

@KlaasT
Copy link
Contributor

KlaasT commented Nov 1, 2019

I was a bit shocked by the number of PRs when I came online today. Really great job. Very much appreciated :-)

@egmontkob
Copy link

Cool, thanks!

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

No branches or pull requests

3 participants