JSON8 / patch

Moved to https://github.com/sonnyp/JSON8/tree/master/packages/patch
ISC License
20 stars 2 forks source link

Slow performance and large size #4

Closed Akkuma closed 8 years ago

Akkuma commented 9 years ago

Compared to alternative implementations it comes out to be roughly 2-3x slower and is also larger in size http://jsperf.com/json-patch-benchmark/2

sonnyp commented 9 years ago

is also larger in size

What do you refer to?

I never took the time to optimize JSON8/patch but it should be easy to reach bruth implementation performances. https://github.com/Starcounter-Jack/JSON-Patch is faster but is not standard compliant

Akkuma commented 9 years ago

Larger in size refers to file size. Only Jiff appears to be larger, but its performance is comparable to the others.

sonnyp commented 8 years ago

About large file; JSON8/patch offers more functionally. But you might not need them them all. You can browserify/webpack/require only the one you need.

require('json8-patch/lib/apply')

sonnyp commented 8 years ago

About performance, thanks, I'll have a look into it. https://github.com/JSON8/patch/issues/4