Refactorized ClosestPair.java in order to be compliant with java sun rules

[MarisaAfuera]

Refactorized ClosestPair.java: adding commets, eliminating unuseful parameters, reorganizing code lines to be less than 80 characteres, each variable declaration in one line.

I've deleted txt file to provide input data.

I've created a unit test class. To complete the tests I've created a auxiliary type Location.

[varunu28]

@MarisaAfuera You can add the algorithm to the master right now but we are adding tests to Development branch in order to avoid confusion. So you can do either of the two below mentioned things:

• Add only algorithm to master
• Add algorithm and test both to Development and it will be merged to master in future.

Avoid creating another class for location. You can put it up in same file

[varunu28]

• You have to remove the test if you want to push this to master. Else you can follow the guidelines mentioned in Development branch and make a PR for that branch
• Move the location class inside ClosestPair as we don't need a separate file for Location

[MarisaAfuera]

Yes varun. I was busy. Next weekend I will fix It. El jue., 6 sept. 2018 22:21, Varun Upadhyay <[email protected]> escribió:

…

***@***.**** requested changes on this pull request. - You have to remove the test if you want to push this to master. Else you can follow the guidelines mentioned in Development branch and make a PR for that branch - Move the location class inside ClosestPair as we don't need a separate file for Location — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#502 (review)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ASac7FyEzBIfijE4g0btptnzcNdKh5tWks5uYZHvgaJpZM4WT69b> .

[MarisaAfuera]

I think I've made the requesting changes. Now in files changed tab only is showed ClosestPair.java. No test neither Location class are available.

Also I squash the commits with git rebase. @varunu28 Is everything right?

[MarisaAfuera]

Hi, @varun I've made the requested changes. I everything right? El jue., 6 sept. 2018 a las 23:21, Varun Upadhyay (<[email protected]>) escribió:

…

***@***.**** requested changes on this pull request. - You have to remove the test if you want to push this to master. Else you can follow the guidelines mentioned in Development branch and make a PR for that branch - Move the location class inside ClosestPair as we don't need a separate file for Location — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#502 (review)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ASac7FyEzBIfijE4g0btptnzcNdKh5tWks5uYZHvgaJpZM4WT69b> .