Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: cdk/cdk
base: eda7ba76e42d
Choose a base ref
...
head repository: cdk/cdk
compare: c8a843ffab80
Choose a head ref
  • 9 commits
  • 9 files changed
  • 1 contributor

Commits on Dec 12, 2012

  1. Added unit test for methods related to ShortestPathFingerprint. This …

    …include unit test for RandomNumber, SimpleAtomComparator, SimpleAtomCanonicalisation.
    johnmay authored and egonw committed Dec 12, 2012
    Copy the full SHA
    bab373d View commit details
    Browse the repository at this point in the history
  2. Copy the full SHA
    21adb90 View commit details
    Browse the repository at this point in the history
  3. Removed redundant cast on comparator

    johnmay authored and egonw committed Dec 12, 2012
    Copy the full SHA
    fd5fb30 View commit details
    Browse the repository at this point in the history
  4. Removed new instance creation on RandomNumber generation. This also r…

    …equired the method was non-static.
    johnmay authored and egonw committed Dec 12, 2012
    Copy the full SHA
    80db3c3 View commit details
    Browse the repository at this point in the history
  5. Made ShortestPathWalker public and optimised/cleaned path iteration.

    The clean path creation does not need to check 'contains' as it is a set and thus all entires are unique.
    johnmay authored and egonw committed Dec 12, 2012
    Copy the full SHA
    c6fe476 View commit details
    Browse the repository at this point in the history
  6. Optimisations to SimpleAtomComparator. The LinkedList isn't needed an…

    …d has an added cost and was replaced with an ArrayList. The iterator was replaced with a foreach and the redundant counter was removed (the atoms were always added at their original index [they're also being sorted so order is not important]).
    johnmay authored and egonw committed Dec 12, 2012
    Copy the full SHA
    63e7358 View commit details
    Browse the repository at this point in the history
  7. Updated documentation and removed SVN tags. Renamed variables to be m…

    …ore concise and reflect their use.
    johnmay authored and egonw committed Dec 12, 2012
    Copy the full SHA
    fd7d0fe View commit details
    Browse the repository at this point in the history
  8. It is not clear why RandomNumber needs to be serialisable but as the …

    …RandomGenerator field is not it has been marked as transient so it is not serialised.
    johnmay authored and egonw committed Dec 12, 2012
    Copy the full SHA
    6b22cdd View commit details
    Browse the repository at this point in the history
  9. Unified spelling of Canonicalisation to the US spelling (z instead of…

    … s) and renamed the class name to be the noun instead of verb.
    johnmay authored and egonw committed Dec 12, 2012
    Copy the full SHA
    c8a843f View commit details
    Browse the repository at this point in the history