Derive Eq for ProtocolType

[VAWVAW]

I want to use ProtocolType in an iamb type that needs it to be Eq. I don't see how deriving this could lead to any problems.

[github-actions]

Merge of fa58c74 to master

Screenshots from all terminal emulators:

alacritty

Image diff vs master:

blackbox

Image diff vs master:

contour

Image diff vs master:

foot

Image diff vs master:

ghostty

Image diff vs master:

kitty

Image diff vs master:

konsole

Image diff vs master:

mlterm

Image diff vs master:

rio

Image diff vs master:

wezterm

Image diff vs master:

xfce4-terminal

Image diff vs master:

xterm

Image diff vs master:

xterm-vt340

Image diff vs master:

[benjajaja]

/merge

[benjajaja]

@VAWVAW may I ask why? I feel like everytime a user needs to deal with ProtocolType, the library has failed in some way.

[VAWVAW]

Thanks for merging this.

iamb exposes a ProtocolType and FontSize setting, because afaict Picker::from_query_stdio() doesn't work on windows (see ulyssa/iamb#653). I am implementing ulyssa/iamb#578 which needs all values that are changed to be Eq.

The only actual usecase I can think of is having a tmux session open in multiple terminals that support different protocols.

[benjajaja]

Ah right. I always thought that it would be best if we could remove the ability to configure the protocol_type etc in iamb. Otherwise some users might have some old configuration that isn't needed anymore (has been fixed upstream).