Address performance issue with start matchStart

quentez / talonjs

JavaScript port of the Talon email quote parsing library.

MIT License

15 stars 9 forks source link

Address performance issue with start matchStart #27

Closed cagerton closed 5 years ago

cagerton commented 5 years ago

This addresses a performance problem that we've encountered with matchStart when encountering extremely long lines.

I found the extra dependency and tsconfig.json changes useful while debugging, but they're strictly not needed for the perf fix itself.

cagerton commented 5 years ago

FWIW, I have no ability to merge this PR. Once it's merged we'll still need @quentez to publish the NPM module. Only then can we update our service which depends on this change.

cwill commented 5 years ago

hmm, right -- maybe we should move/fork the repository to the frontapp org? (but in the meantime, let's see if we can track down Q)

cagerton commented 5 years ago

🎉