Remove all tests that depend on TestParser - they're rubbish and
need to be completely rewritten to be proper unit tests.

Seems a bit odd for each implementation to define a method that
*always* returns a static String[] when we can just pass that
up to the base class.

Just add a utility method to get the callback proxy instead.

Think this is a bit nicer, and is one less thing to have to change
if we switch to events.

All callbacks are now done through getCallback, which offers
type safety (gasp!) and is potentially extensible in the future
(wow!).

This means that nothing can depend on callbacks existing, so
there are two functional changes:

 - the parser will no longer disconnect automatically on failed
   pings; users have to listen for the event and handle it
   themselves;

 - the parser will no longer attempt to automatically use alt
   nicknames or create new nicknames on connection; users have to
   listen for the event and implement the logic themselves

Both of these were really weird anyway, as adding a listener (even
for debugging purposes) disabled the built-in behaviour.

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

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

Make all package-private fields and methods they hackily touched
into public, for now.

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

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

Change-Id: Iad5e74e45bb052f65b761ce8e334b571838edc10
Reviewed-on: http://gerrit.dmdirc.com/2978
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@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: 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: I1c711ca0525af9a78cbd7c07876eaf6cc91c1434
Reviewed-on: http://gerrit.dmdirc.com/2604
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

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: Ie7236e561d04eed00576a8ff6b43228106d3bff5
Reviewed-on: http://gerrit.dmdirc.com/2288
Reviewed-by: Chris Smith <chris@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

 - This is used for backbuffer support in DFBnc, we currently only care about timestamps in
   ProcessMessage, everything else will just use the current time in their callbacks as before.

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

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

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

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

Fixes CLIENT-135

Change-Id: I5f326523ff031c0996996f0efad6c8e5d7278c7f
Reviewed-on: http://gerrit.dmdirc.com/1859
Automatic-Compile: DMDirc Local Commits <dmdirc@googlemail.com>
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I4a6c51789e37e0f96967344c98010ee7ff760506
Reviewed-on: http://gerrit.dmdirc.com/1723
Reviewed-by: Chris Smith <chris@dmdirc.com>
Automatic-Compile: DMDirc Local Commits <dmdirc@googlemail.com>

Change-Id: Ifd55c3c04740aa88f349aae5abee7fa817b82e39
Reviewed-on: http://gerrit.dmdirc.com/1692
Automatic-Compile: DMDirc Local Commits <dmdirc@googlemail.com>
Reviewed-by: Chris Smith <chris@dmdirc.com>

Also don't duplicate code.

Change-Id: I5af52a8d50e0ddc93618586781c762e0f6eece5e
Reviewed-on: http://gerrit.dmdirc.com/1347
Automatic-Compile: DMDirc Local Commits <dmdirc@googlemail.com>
Reviewed-by: Chris Smith <chris@dmdirc.com>

Change-Id: I920ca1847ac1af6cf3a13e891e6340f68c6abb91
Reviewed-on: http://gerrit.dmdirc.com/1335
Automatic-Compile: DMDirc Local Commits <dmdirc@googlemail.com>
Reviewed-by: Gregory Holmes <greg@dmdirc.com>

Change-Id: I51d0e3c7975740b1ede5d6226639022bd15a482a
Reviewed-on: http://gerrit.dmdirc.com/1328
Automatic-Compile: DMDirc Local Commits <dmdirc@googlemail.com>
Reviewed-by: Gregory Holmes <greg@dmdirc.com>

Change-Id: I5fbdce2c0b0258d19e3dbb962826e3310616748d
Reviewed-on: http://gerrit.dmdirc.com/1326
Automatic-Compile: DMDirc Local Commits <dmdirc@googlemail.com>
Reviewed-by: Chris Smith <chris@dmdirc.com>

Switch everything that does something on a per-ircd basis to use ServerTypes not Strings.
Deprecate getIRCD() in favour of getServerSoftware() and getServerSoftwareType().

Change-Id: I324eaadeb39a9999f7dd9d7b8e6bd583dce7abca
Reviewed-on: http://gerrit.dmdirc.com/431
Reviewed-by: Chris Smith <chris@dmdirc.com>
Tested-by: Chris Smith <chris@dmdirc.com>

Change-Id: I581861c8c8743cb835c45d128c4b4768372de807
Reviewed-on: http://gerrit.dmdirc.com/201
Reviewed-by: Shane Mc Cormack <shane@dmdirc.com>
Tested-by: Shane Mc Cormack <shane@dmdirc.com>

If we have a 0-length string just ignore it.
Fixes issue 2942

Change-Id: I963aab86ff426fed0fa22fdbd1b3dce36818106a
Reviewed-on: http://gerrit.dmdirc.com/88
Reviewed-by: Chris Smith <chris@dmdirc.com>
Tested-by: Chris Smith <chris@dmdirc.com>