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

Relax xrandr parser to allow cut down pastes? #47

Open
imirkin opened this issue Nov 23, 2019 · 0 comments
Open

Relax xrandr parser to allow cut down pastes? #47

imirkin opened this issue Nov 23, 2019 · 0 comments

Comments

@imirkin
Copy link

imirkin commented Nov 23, 2019

If I just paste a section like

EDID:
asdf
asdf

and that's it, nothing is processed. It wants at least one non-EDID line before yield'ing the data. Is this by design, or would it be OK to allow these incomplete pastes. I suspect (some) people will feel weird pasting the whole xrandr (/xorg log with my PR), and would just want to include the relevant part. And the "hex" paste doesn't accept this sort of thing either, since it wants bytes separated by spaces.

The danger is someone is a line short. However I think there's validation in the parsing which would prevent that sort of issue?

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