jszlib icon indicating copy to clipboard operation
jszlib copied to clipboard

AMD port, plus bugfixes

Open rbuels opened this issue 13 years ago • 1 comments

In my fork, I ported jszlib to AMD for use in JBrowse (you might not want to take those changes), but I've also fixed some more bugs on not-often-used code paths.

rbuels avatar Oct 26 '12 17:10 rbuels

Excellemt , thanks! I'm travelling at the moment but will merge when I get back On 26 Oct 2012 18:57, "Robert Buels" [email protected] wrote:

In my fork, I ported jszlib to AMD for use in JBrowse (you might not want to take those changes), but I've also fixed some more bugs on

not-often-used code paths.

You can merge this Pull Request by running:

git pull https://github.com/rbuels/jszlib master

Or view, comment on, or merge it at:

https://github.com/dasmoth/jszlib/pull/1 Commit Summary

  • fix whitespace and syntax abnormalities, and convert to AMD format
  • move inflate.js into root dir out of js/ dir, add a main.js to make i…
  • add newline to the end of README.md
  • add a package.json and a dojo build profile
  • don't fail to parse on platforms that don't support typed arrays
  • factor fast arrayCopy out into a separately-includable AMD module
  • fix some more missing "this"s in unusual code paths

File Changes

  • M README.md (2)
  • A arrayCopy.js (49)
  • R inflate.js (260)
  • A jszlib.profile.js (66)
  • A main.js (5)
  • A package.json (18)

Patch Links

  • https://github.com/dasmoth/jszlib/pull/1.patch

  • https://github.com/dasmoth/jszlib/pull/1.diff

    — Reply to this email directly or view it on GitHubhttps://github.com/dasmoth/jszlib/pull/1.

dasmoth avatar Oct 26 '12 18:10 dasmoth