Make seen & valid functions internal to indexnodes namespace.
Remove some more silly const-correctness.
Clarify coding style.

Especially const-correctness and object ownership.
Remove these getters. Tell, don't ask.
Use a read-write lock here instead.
Irrelevant / confusing.
TODO.
Missed this test programme when I removed the getters from indexnode.
Don't manipulate the list directly, instead raise the event.
Make this message more accurate.
Remove unused parameter.
Don't give fetcher the means to parse these protocols, do it yourself.
This aggregate type is pretty useless given that it aggreates only one record.
Remove useless wrapper.

The event is expecting an unparsed protocol, not a version anyway.
Better name for this function.
Better name for this function.
Get ID as well as protocol from indexnode, rather than making it up.
Ownership semantics.
Use new ownership semantics.
This should deal with static indexnodes too, which means reading from the config.