Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AMD port, plus bugfixes #1

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

AMD port, plus bugfixes #1

wants to merge 11 commits into from

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" [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:

#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