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

crash while rendering youtube-viewer results (most likely to be emoji) #37

Closed
dlqqq opened this issue Jan 1, 2019 · 2 comments
Closed

Comments

@dlqqq
Copy link

dlqqq commented Jan 1, 2019

(this is my first github issue so pls tell me if i need to provide more information)

I use youtube-viewer to browse Luke's videos because I hate using bloated webkits to perform such a basic service. I'm using this fork of st bc i'm too much of a brainlet to get the patches working, and anyways youtube-viewer almost always causes st to crash. Running st in another terminal, I get the following output.

X Error of failed request: BadLength (poly request too large or internal Xlib length error)
Major opcode of failed request: 139 (RENDER)
Minor opcode of failed request: 20 (RenderAddGlyphs)
Serial number of failed request: 4621
Current serial number in output stream: 4841

Interestingly, it seems to be dependent on window size. When the terminal is less than roughly 1/3 of my screen, youtuve-viewer renders fine. I hypothesize this is because the terminal size "cuts off" incompatible emoji titles in Luke Smith's videos, and prevents st from crashing. More interestingly, it seems only some emoji are affected. The "lightning bolts" render fine.

20190101-114433

So I changed the font to Unifont and it did render Luke Smith's video titles without crashing. However... it's output is a little less than ideal.

20190101-115855

I want to keep my Liberation (FREE AS IN FREEDUMS) Sans font, and I don't want to have to be limited by fonts that must support emoji unicodes. Is there a way for st to handle like URxvt and just "default" to a block when it fails to render a character? And if there isn't, can Luke pls include a statement in the README listing some cool, hip fonts that renders unicode emojis well in terminals? Thanks.

@dlqqq dlqqq closed this as completed Jan 1, 2019
@dlqqq dlqqq reopened this Jan 1, 2019
@dlqqq
Copy link
Author

dlqqq commented Jan 1, 2019

I have no idea why, but it seems that changing the font from Unifont back to Liberation Mono in .Xresources removed the bug for me. I really have no idea why this happened, but since I can't reproduce this, I'm going to close this issue until I can encounter a reliable way to generate this error.

@dlqqq dlqqq closed this as completed Jan 1, 2019
@dlqqq
Copy link
Author

dlqqq commented Apr 15, 2019

Really hate to necrobump this, but I found the issue and I hope anybody experiencing a similar issue can get help here. I did not have my locale set. To see your current locale:
% localectl
I had my system locale set to C for some reason. To change this, just do
% localectl set-locale LANG=[insert locale].
I set my locale to en_US.utf8, and restarted. Then all unicode in my terminal rendered properly.

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

1 participant