Skip to content

Added short and long tons. - #1

Merged
angularsen merged 1 commit into
angularsen:masterfrom
distantcam:master
Jul 22, 2013
Merged

Added short and long tons.#1
angularsen merged 1 commit into
angularsen:masterfrom
distantcam:master

Conversation

@distantcam

Copy link
Copy Markdown
Contributor

http://en.wikipedia.org/wiki/Short_ton
http://en.wikipedia.org/wiki/Long_ton

Also known as the US ton and the UK ton, these are currently the bane of my existence.

Thankyou for releasing this library. My team is constantly having to convert things and this will help.

@angularsen
angularsen merged commit 4347990 into angularsen:masterJul 22, 2013
@angularsen

Copy link
Copy Markdown
Owner

Thanks for contributing and for following the code style.
Found one mistake though, the short and long tons were implemented the other way around.

You wrote:
Assert.AreEqual(907.18474, oneKg.ShortTons, Delta);
Assert.AreEqual(1016.0469088, oneKg.LongTons, Delta);

One kg is not 907 short tons :-)

Corrected it and merged it.

@distantcam

Copy link
Copy Markdown
ContributorAuthor

Doh! Of course. This is why I need a library to handle this for me. :D

Sign up for freeto 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.

2 participants

@distantcam@angularsen