timvideos / edid.tv

EDID Database Website
GNU General Public License v2.0
26 stars 7 forks source link

Relax xrandr parser to allow cut down pastes? #47

Open imirkin opened 4 years ago

imirkin commented 4 years ago

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?