mergesorts icon indicating copy to clipboard operation
mergesorts copied to clipboard

time measurement for all versions in run.sh

Open Imaniac230 opened this issue 4 years ago • 7 comments

How about some time measurements for the different versions? Currently it would measure compile time plus run time though.

Imaniac230 avatar Jul 21 '20 16:07 Imaniac230

So I wouldn't do this big code copy and paste. If you want this, the script needs a refactor.

geohot avatar Jul 22 '20 12:07 geohot

Ok, here's my proposal? I'm 51% certain I didn't do anything stupid, looking forward to the next roast on your stream. :D

Imaniac230 avatar Jul 22 '20 21:07 Imaniac230

Still seams a little complex to the original script, is there any over way ?

Mrgove10 avatar Jul 23 '20 10:07 Mrgove10

Might not be really any simpler but it now keeps the consistent ordering on output. Though I'm not sure the way I'm extracting the language names would be recommended like that.

Imaniac230 avatar Jul 23 '20 11:07 Imaniac230

It's still very boilerplatey. This doesn't have to stay written in bash.

geohot avatar Jul 23 '20 15:07 geohot

Well before changing to something completely different there's this simple way. Since the commands are typed and executed explicitly anyway there should be no problem with the eval.

Imaniac230 avatar Jul 24 '20 18:07 Imaniac230

If this thing ever gets resurrected ...final commit.

Imaniac230 avatar Aug 10 '20 15:08 Imaniac230