We had discussed that. Would it make sense to specify that coll.compare would actually be a bound method? That's one of the things that had a little "oh, no good any way we do this", as Erik wouldn't know of any example we'd follow by defining a method on an object to be bound by default.

Axel

On 21.03.11 18:26, Mark S. Miller wrote:


On Mon, Mar 21, 2011 at 5:37 PM, Nebojša Ćirić <[email protected]
<mailto:[email protected]>> wrote:

    Sorry, I didn't get the difference first time around.

    Also, current fix is to do:

    array.sort(function(a,b) { return coll.compare(a,b); });


Why not

     array.sort(coll.compare.bind(coll))

?




    21. март 2011. 17.31, Erik Arvidsson <[email protected]
    <mailto:[email protected]>> је написао/ла:

        On Mon, Mar 21, 2011 at 17:28, Nebojša Ćirić <[email protected]
        <mailto:[email protected]>> wrote:
         > Started implementation of collator in Chrome and hit a problem:
         > var coll = locale.collator();
         > array.sort(coll.compare);
         > Compare method gets bound to the undefined or global object
        at the call
         > site. Erik mentioned that this problem will be solved in
        Harmony by passing
         > additional "this" parameter to for-each and likes. We would
        like to propose
         > extending this syntax to the sort method too.

        Not exactly. I said that the Array methods on ES5 already pass in an
        optional this parameter and maybe we *could* do the same for
        Array.prototype.sort in Harmony.

         > We discussed each part of the API in order to get detailed
        parameters of
         > each constructor and method.
         > General:
         >
         > Add options property to each class that would give you actual
        value for the
         > user parameters. For example, if user asked for islamic
        calendar, and we
         > only have islamic-civil, we set calendar property to
        islamic-civil. Allows
         > developer to iterate until satisfied with the result.
         > Use Unicode identifier vs. BCP47 in the API
         >
         > Collator:
         >
         > numeric - specifies numeric sort (9 comes before 12)
         > ignoreVariants - ignore all of case, width and kana
         > ignoreWidth, ignoreCase and ignoreKana - subvariants we may
        implement to
         > fine tune the behavior
         > ignoreAccents - ignore accents
         > ignoreSymbols - ignore punctuation and symbols
         > variant - phonebook, ... - string
         >
         > NumberFormat:
         >
         > Allow patterns to specify grouping, currency symbol position
        and sign
         > location
         > Start with ICU patterns and see if they work for everybody
         > Don't support overrides for grouping separator and decimal
        point for now
         > Specify both currencySymbol and currencyCode as override
         >
         > DateTimeFormatSymbols:
         >
         > Added Era and day period methods (AM/PM)
         > Moved all methods to DateTimeFormat class
         > Remove DTFSymbols class
         >
         > DateTimeFormat:
         >
         > Specify calendar names better (move work to Unicode/LDML and
        point to their
         > document).
         > Allow short/long dateType to get value from the system or cloud.
         > .options[skeleton] should contain best match for the given
        skeleton
         >
         > --
         > Nebojša Ćirić
         >
         > _______________________________________________
         > es-discuss mailing list
         > [email protected] <mailto:[email protected]>
         > https://mail.mozilla.org/listinfo/es-discuss
         >
         >



        --
        erik




    --
    Nebojša Ćirić

    _______________________________________________
    es-discuss mailing list
    [email protected] <mailto:[email protected]>
    https://mail.mozilla.org/listinfo/es-discuss




--
     Cheers,
     --MarkM



_______________________________________________
es-discuss mailing list
[email protected]
https://mail.mozilla.org/listinfo/es-discuss
_______________________________________________
es-discuss mailing list
[email protected]
https://mail.mozilla.org/listinfo/es-discuss

Reply via email to