Remove pointless super() calls.
Remove redundant interface decs.
Remove unnecessary enum annotations.
Change .length == 0 to .isEmpty.

Change-Id: Ic89c70772063157e970f046b1cd04980a60bc1a8
Reviewed-on: http://gerrit.dmdirc.com/3844
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Change-Id: I22300a4ec7a09ee30a1fbc4d6fb91ff049ca3e39
Reviewed-on: http://gerrit.dmdirc.com/3839
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Change-Id: I8e7e36bec9e6231f7596cd0a788649a27bdc8b1f
Reviewed-on: http://gerrit.dmdirc.com/3809
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Change-Id: If94166e179b88d1700ae7eb89c03d71e35df7f51
Reviewed-on: http://gerrit.dmdirc.com/3801
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Change-Id: Ic2a1b97aa44e9168bae4a1e28941208e7711bb46
Reviewed-on: http://gerrit.dmdirc.com/3741
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I4d9c708ae950ae09cc7eb0bf14c8f424f791f5fe
Reviewed-on: http://gerrit.dmdirc.com/3687
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: Ib561477b4fbb5eeef0fa459a9bf20f7f37720f7b
Reviewed-on: http://gerrit.dmdirc.com/3618
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Chris Smith <chris@dmdirc.com>

Change-Id: I3c7a6b5c9139928687422b0044e37c6ee975c00d
Reviewed-on: http://gerrit.dmdirc.com/3575
Reviewed-by: Chris Smith <chris@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Change-Id: I686dd4b6cd64853eed4d407e3d10471ea1a0b7fd
Fixes-Issue: CLIENT-477
Reviewed-on: http://gerrit.dmdirc.com/3237
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Chris Smith <chris@dmdirc.com>

Change-Id: Iad5e74e45bb052f65b761ce8e334b571838edc10
Reviewed-on: http://gerrit.dmdirc.com/2978
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: Iaf88955e20a8797c0f20924ef343626a435af91f
Reviewed-on: http://gerrit.dmdirc.com/2970
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: If346fc78ffb10e4302d129fd16386370398bec1d
Reviewed-on: http://gerrit.dmdirc.com/2971
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I7ed80c3e353e8f68f97928555e4f6e114a03b755
Reviewed-on: http://gerrit.dmdirc.com/2901
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: Id834157281c02302bfe6f71201042712144b22ff
Reviewed-on: http://gerrit.dmdirc.com/2894
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Chris Smith <chris@dmdirc.com>

Change-Id: If8c377f64b3b336203ab4d01aaaa0a731f0545a1
Reviewed-on: http://gerrit.dmdirc.com/2885
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Chris Smith <chris@dmdirc.com>

Change-Id: I8e74b3d9a247c289c96cd22736bcc4636f222996
Reviewed-on: http://gerrit.dmdirc.com/2839
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Chris Smith <chris@dmdirc.com>

Get rid of weird round-the-houses method of processing, and avoid
duplicating the value in a map and a field.

This fixes our handling of servers doing really stupid, theoretical
things like resetting the CHANTYPES they support. (And by "fixes"
I mean the parser will keep state, while everything else burns
around it.)

Change-Id: Ie4936f21e68e4bb1134e9be3f16932eef4ab086f
Reviewed-on: http://gerrit.dmdirc.com/2831
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Shane Mc Cormack <shane@dmdirc.com>

Change-Id: Ib9222a1a0ddf182fe9c092fa2340c3954c251848
Reviewed-on: http://gerrit.dmdirc.com/2815
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I4ee609a42962a025e022315e64d199d1a8ecf3ba
Reviewed-on: http://gerrit.dmdirc.com/2814
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I907f185d82fb4f7273eb5341acd100815688c4f2
Fixes-issue: CLIENT-406
Reviewed-on: http://gerrit.dmdirc.com/2813
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Introduce methods for handling individual bits, instead of having
a pair of giant if statements of doom.

Tidy some comments and formatting up at the same time.

This is functionally the same as it was before.

Change-Id: I83f09cb6c36be203d102c44188f90de2b0587e58
Reviewed-on: http://gerrit.dmdirc.com/2812
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I69e3dde890fef13c31cf519523ce447a27ba29b9
Reviewed-on: http://gerrit.dmdirc.com/2701
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Chris Smith <chris@dmdirc.com>

Change-Id: Ia13810eaecc9b387247c17948190ed5ccb692e14
Reviewed-on: http://gerrit.dmdirc.com/2667
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Change-Id: Idcb2ad40b09e56a43a4931e337fe097db8996aad
Reviewed-on: http://gerrit.dmdirc.com/2649
Reviewed-by: Chris Smith <chris@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Change-Id: I1c711ca0525af9a78cbd7c07876eaf6cc91c1434
Reviewed-on: http://gerrit.dmdirc.com/2604
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I6e48c0092c26fc5faec05f6400b9b35d12f91902
Reviewed-on: http://gerrit.dmdirc.com/2577
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I15144999a27f9f022bd0f27ba0967048e58da182
Reviewed-on: http://gerrit.dmdirc.com/2570
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Of the top 20 networks, it looks like only quakenet support it right now,
but it makes sense and allows for multiple servers to exist on the same IP
with different ports without needing to know the ports.

Eg for a test network you could just connect to irc://inspircd.testnetwork/
or irc://ircu.testnetwork/ without needing to remember the ports.

Change-Id: Ia366292e64b2ca46cb8658b34fdac598fa155722
Reviewed-on: http://gerrit.dmdirc.com/2567
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I00f141e3fd4090131d116f103cae34a051bf89c1
Reviewed-on: http://gerrit.dmdirc.com/2566
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Chris Smith <chris@dmdirc.com>

Change-Id: I701d0dca3e03fc8f5f03356a641bb5985c3c661e
Depends-On: I4395b5b15edf196390244da0658c47ca9c0dcda0
Reviewed-on: http://gerrit.dmdirc.com/2547
Reviewed-by: Chris Smith <chris@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Add support for "away-notify" capability (http://ircv3.atheme.org/extensions/away-notify-3.1)
Add support for "account-notify" capability (http://ircv3.atheme.org/extensions/account-notify-3.1)
Add support for "extended-join" capability (http://ircv3.atheme.org/extensions/extended-join-3.1)
Don't bother negotiating UHNAMES or NAMESX using PROTOCTL if we have already enabled the userhost-in-names and multi-prefix capabilities with CAP
Don't bother asking for TSIRC using the TIMESTAMPEDIRC command if we have negotiated it using CAP

Rename "tsirc" capability to "dfbnc.com/tsirc" as a non-standard capability

From a DMDirc point of view, capabilities probably need exposing through the general interface somehow so that if (for example) away-notify is enabled, the client doesn't still send lots of WHOs.

Change-Id: Ice7a8ceae5595a364881e5a1fbe2d7fbdef9a639
Reviewed-on: http://gerrit.dmdirc.com/2544
Reviewed-by: Chris Smith <chris@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

(See: http://www.leeh.co.uk/draft-mitchell-irc-capabilities-02.html)

Change-Id: I9aaf2877770f86a93c9f8c525bea6315a7a8802b
Reviewed-on: http://gerrit.dmdirc.com/2541
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Change-Id: I88eee521770db9ec756a9c18c28f935cdbdc19d7
Reviewed-on: http://gerrit.dmdirc.com/2542
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Change-Id: Id43956b2fad628754196aea3782d4892f3b768a5
Reviewed-on: http://gerrit.dmdirc.com/2537
Reviewed-by: Shane Mc Cormack <shane@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Change-Id: I0f247f1fd7ecfd434777845d194812a219246319
Reviewed-on: http://gerrit.dmdirc.com/2507
Reviewed-by: Shane Mc Cormack <shane@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Change-Id: I7f7aae8a819f464a99ee3f0169ca2258bf046997
Reviewed-on: http://gerrit.dmdirc.com/2526
Reviewed-by: Chris Smith <chris@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Change-Id: Ib1038c0d46b0c1f2de15050bb772fe519417d7bd
Reviewed-on: http://gerrit.dmdirc.com/2534
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Shane Mc Cormack <shane@dmdirc.com>

Previously it added an object as a callback target even if it didn't implement the callback handler, this was pointless and stupid, now it checks.

Change-Id: I24ce9f063a0f1f95e84ad2ae28a67e335cd63afe
Reviewed-on: http://gerrit.dmdirc.com/2519
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I399f70e589c94570c2e9e96c8d2a92a66448e627
Reviewed-on: http://gerrit.dmdirc.com/2518
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Change-Id: I56e7ab284abb91e22bf5c35b73035302a85b45b4
Reviewed-on: http://gerrit.dmdirc.com/2513
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I3ac4dcb46143d29ce2f3304f5cf81d3e4e3943c3
Reviewed-on: http://gerrit.dmdirc.com/2510
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: Id91f50347ce9a63ebbfa0be1792c0feba7348d87
Reviewed-on: http://gerrit.dmdirc.com/2503
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Chris Smith <chris@dmdirc.com>

Change-Id: I460645e88ba7b5614dc8b2ac83870941cfb7dc52
Reviewed-on: http://gerrit.dmdirc.com/2497
Reviewed-by: Chris Smith <chris@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Change-Id: I85a6a95916a03ff4766a4319653a1fb929747566
Reviewed-on: http://gerrit.dmdirc.com/2494
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Chris Smith <chris@dmdirc.com>

Change-Id: Ib29494e43a7271f8511bc7f733534aae47aea8b2
Fixes-Issue: CLIENT-367
Reviewed-on: http://gerrit.dmdirc.com/2481
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

We no longer truncate lines if there's a character which
can't be coded for in unicode, which stops lots of other
breakage.

Change-Id: I1886703fc7f6e4d2076da4e4bd3b9aa60e8bc051
Fixes-Issue: CLIENT-353
Reviewed-on: http://gerrit.dmdirc.com/2414
Reviewed-by: Shane Mc Cormack <shane@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Change-Id: I92f7184110dec669bf9f4aa09dab0a07245f164e
Reviewed-on: http://gerrit.dmdirc.com/2384
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I75ef7125aeefbddbf612629db37743a06ef72834
Reviewed-on: http://gerrit.dmdirc.com/2366
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I3959753c4342b3fd1c774134ec35b6129add7503
Reviewed-on: http://gerrit.dmdirc.com/2364
Reviewed-by: Chris Smith <chris@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Change-Id: I04b90827b4ea4bcffd8f1a917d9e83fd87cf1505
Reviewed-on: http://gerrit.dmdirc.com/2355
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>