For some reason we're checking hostname (e.g. dmdirc.com)
against full hosts (e.g. chris!ident@dmdirc.com) and expecting
them to match.

Queries now check on nicknames.

This keeps happening, and I don't know why we try to unwrap twice.
Probably only makes sense for certain exception types...

Stop ChannelListModesRetrieved being displayable - not sure
why it ever was, we don't have a formatter for it.

Instead of passing them in to the FrameContainer ctor, create
them afterwards.

This means the reference in FrameContainer can't be final but
removes the horrible setOwner() kludge that was previously in
CommandParser, and cuts down the deps needed to pass in to the
FrameContainer ctor.

Start to move FrameContainers behind an iface so they can be
pulled out.

For now, just cast back when needed.

Start to move FrameContainers behind an iface so they can be
pulled out.

For now, just cast back when needed.

Only one place in the core needs the silly IRC format, so just
inline it there.

This is fairly trivial to implement with streams/lambdas,
no need to have our own hand-rolled list implementation
for it.

Fixes #521
Fixes #522