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

terminal support for the accepting unicode input #162

Merged
merged 11 commits into from
Jun 14, 2017

Conversation

bagajjal
Copy link

@bagajjal bagajjal commented Jun 9, 2017

  1. Support the unicode characters as input to the ssh client.
    Entering character ö prints two characters Win32-OpenSSH#711

  2. Remove the bAnsi related code changes to the ssh-shellhost.c as that code never executes..

  3. Fixed "long pressing key issue for the slow ssh connections"
    Long-pressing arrow-keys prints control sequences instead of advancing cursor (Linux client) Win32-OpenSSH#701

  4. fixed warning message in the latestw_all recent commits.

  5. Fixed the issue with ssh failures in the latestw_all recent commits.


p = p + strlen(p);
const char *permissions = "********* ";
for(int i = 0; i < strlen(permissions); i++)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why replace strncpy?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

strncpy is giving a warning to use strncpy_s..
To use strncpy_s we need to know the size of p.. since p is a char*, the caller can pass pointer to a memory (or) an array so we can't decide the size here.. so to avoid the warning.. changed it to for loop

return;
wchar_t *buf = utf8_to_utf16(ansikey);

if (!buf) {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we call fatal("...") here?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fatal is not available in shell-host.c and also printf() output is displayed on client side..

@manojampalam manojampalam merged commit 98eca17 into latestw_all Jun 14, 2017
@manojampalam manojampalam deleted the terminal_unicode_#711 branch June 14, 2017 04:25
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

Successfully merging this pull request may close these issues.

4 participants