Skip to content
This repository has been archived by the owner on Feb 22, 2018. It is now read-only.

Added MergeSort as new benchmark #29

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

DkWeb
Copy link

@DkWeb DkWeb commented Dec 14, 2014

I've added an implementation of MergeSort to the suite (dart-implementation, the derived dart.js-file and finally a javascript implementation). Surprisingly the javascript implementation seems to be twice as fast the dart counterpart.

@@ -26,6 +26,7 @@ final CATEGORIES = {
'FluidMotion',
'Tracer',
'Havlak',
'MergeSort',
Copy link
Contributor

Choose a reason for hiding this comment

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

Is spacing and indent messed up here? Can't tell if it's a code review UI thing.

@DkWeb
Copy link
Author

DkWeb commented Dec 15, 2014

Sorry for the inconvenience due the missing license and the wrong intent. This should be fixed now.

@sethladd
Copy link
Contributor

Have you signed the Google CLA? https://cla.developers.google.com/about/google-individual

@DkWeb
Copy link
Author

DkWeb commented Dec 15, 2014

Thanks for your fast response Seth. Actually I haven't signed the CLA up to now. I'll have a look at it tomorrow. At first sight google asks for a lot of personal information like the phone number and full address so I'll first have to check what they are allowed to do with this information ;-).

@azenla
Copy link
Contributor

azenla commented Jan 25, 2015

@sethladd Is it possible for me to use my CLA to help him get his code in?

@DkWeb If it's allowed, I could use my CLA so that your code is able to be merged.

@DkWeb
Copy link
Author

DkWeb commented Jan 25, 2015

@kaendfinger thanks for the offer. That would be great, because I really don't want to give Google all my personal information just to commit this simple benchmark.

@azenla
Copy link
Contributor

azenla commented Jan 25, 2015

@DkWeb No guarantees, but if they can do it I would be glad to help :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants