This patch fixes some issues raised with the shortest path fingerprint patch
Please note these need to be applied in order
- Unit tests for RandomNumber, Canonicallisation and Comparator [0001]
- Moved classes from standard to fingerprint [0002]
- Added some JavaDoc (more required)
- Optimizations on some parts I spotted [0003-0006,0008]
- Renaming variable/removed SVN tags (more needed) [0007]
- Changed class name to noun instead of verb (which should be reserved for methods) [0009]
Looks like I have to upload these 1 by 1
and apparently I have to provide content...
[patch 0002]
[patch 0003]
[patch 0004]
[patch 0005]
[patch 0006]
[patch 0007]
[patch 0008]
[patch 0009]
I'll keep this patch open for the small updates, if you like.
Okay, what needs revision?
Argh... where did that comment go?? Darn.
OK, from my memory: a few files had missing copyright headers (0002 I think)
There was a JavaDoc about @param 'n' with an incomplete JavaDoc ending with "... the".
That was it.
Okay cool, I thought it was a stray comment :-)
Okay have made the revisions.
applied and pushed by Egon