Commits on Feb 13, 2017

1. Fix of the compare function …

   The previous version of the compare function was not consistent (it did not follow the requirements on the compare function), see https://tc39.github.io/ecma262/#sec-array.prototype.sort for details. You can see that the compare function is not correct by using it to sort larger arrays. For example, if you sort the letters of a well known sentence:
   `"the quick brown fox jumps over the lazy dog".split('').sort((a,b) => a>b).join('')`
   returns `"uge d ac  be e  f otjhmpsnooqrothrkliuvwxyz"` on Node.js 6.7.0. While `"the quick brown fox jumps over the lazy dog".split('').sort((a,b) => a > b ? 1 : (a < b ? -1 : 0)).join('')` returns the correct result `'        abcdeeefghhijklmnoooopqrrsttuuvwxyz'`.

   

   iamstolis authored Feb 13, 2017
   Configuration menu

   • View commit details
   • Copy full SHA for 0ad46f8
   • Browse repository at this point

   Copy the full SHA

   0ad46f8 View commit details

   Browse the repository at this point in the history