search

usnistgov / SCTK

Other
208 stars 52 forks source link

sort more things so that the hubscr tests pass #6

Closed jtrmal closed 6 years ago

jtrmal commented 6 years ago

I had to add more sorts on the top of d1e4744669e62be18cfaff97622bd0d2a4301ab5 Now the tests pass both on perl 5.18 and 5.26 (both on MacOSX).

Actually, not all of them were necessary to make the test pass (only a subset was), but I edited them anyway, I don't think there is a harm in iterating through hashes via sorted sequence. But I can provide the minimal patch file that has only the differences that were needed to make this test pass (in case there is a wish for "minimal" patch).

jfiscus commented 6 years ago

Thanks Jan. I've merged the patches. I agree they may not be necessary, but it's easier to be consistent.