mdumic/cufon — comment on commit 82c82828af2527b62f0fc803b96b8791dbbed5c9
File:  line 0
By mdumic:

We're getting closer. :) I'm OK with 1 and 3 (discretionary ligatures are next on my list and this was the way I hoped we could do it).
I'm not sure how to do 2, however. Your points are valid, but I don't see good solution:

* we could make BaseEngine (and put common stuff in and extend from that) but I don't see anything else common to both engines.
* we could make api.registerEngine inject util functions but that would be overkill as well.
* we could make transformer function an option but there is already `modifyText`, which, unfortunately is not appropriate for the use here.
* or we could just leave as is for now: `Cufon.CSS.textTransform(font.applyLigatures ? font.applyLigatures(text) : text, style).split('');`.

Is there other way? What do you suggest?

By the way I tried accessing chars with indexer (string[]) as well. This is way faster in IE but I wouldn't change to that though.

You didn't give me any comments on how to submit new patch?