Skip to content

Conversation

FridaTveit
Copy link
Contributor

Removed duplicate of PeakComparator (extracted from PlateVerticalGraph in pull request #50 ) in BandGraph and CarSnapshotGraph. Also removed unneeded uses of 'this' in BandGraph and CarSnapshotGraph as per issue #41.

I have compared the PeakComparer classes that were present in BandGraph and CarSnapshotGraph and they are definitely the same as the PeakComparer which I extracted from PlateVerticalGraph.

PeakComparer which was extracted from PlaeVerticalGraph. I have compared
the old PeakComparer which was in PlateVerticalGraph with the one in
CarSnapshotGraph and they were the same. The old PeakComparer has been removed. Also removed unneeded uses of 'this' in CarSnapshopGraph as per issue oskopek#41.
…ate. Removed unneeded uses of 'this' in BandGraph as per issue oskopek#41.
@coveralls
Copy link

coveralls commented Jan 6, 2017

Coverage Status

Coverage decreased (-0.2%) to 14.291% when pulling 981b575 on FridaTveit:RemoveDuplicateOfPeakComparator into fde1d42 on oskopek:master.

Copy link
Owner

@oskopek oskopek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Thanks for the de-duplication.

@oskopek oskopek merged commit 4f6dfa7 into oskopek:master Jan 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants