Skip to content

Conversation

@rbuels
Copy link

@rbuels rbuels commented Oct 26, 2012

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.

@dasmoth
Copy link
Owner

dasmoth commented Oct 26, 2012

Excellemt , thanks! I'm travelling at the moment but will merge when I get
back
On 26 Oct 2012 18:57, "Robert Buels" notifications@github.com 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:

#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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants