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

fix(localStorageDriver): fix odd length buffer serialization #329

Merged
merged 3 commits into from
Dec 27, 2014

Conversation

thgreasi
Copy link
Member

Rebases and resolves #285.
Closes #114.

@thgreasi
Copy link
Member Author

Got some code from websql driver.
#114 should now be one step closer.

@thgreasi
Copy link
Member Author

Just added a commit that abstracts the serialization code to a separate file and closes #114.
The file size gains of the minified files ranges from 4.72% to 6.04% (for localforage.min.js and locaforage.nopromises.min.js respectively).

@thgreasi
Copy link
Member Author

@tofumatt Just did an extra rebase, hope it helps.

@tofumatt
Copy link
Member

Whoa, sweet! 👍

Took a quick look and it seems awesome. I'll look over once more but I think this is good to merge. I'll get this in before I check on the batching PRs.

tofumatt added a commit that referenced this pull request Dec 27, 2014
fix(localStorageDriver): fix odd length buffer serialization
@tofumatt tofumatt merged commit eecd3fc into localForage:master Dec 27, 2014
@tofumatt
Copy link
Member

So awesome, thanks so much for this one! 👍

@thgreasi thgreasi deleted the odd-length-binary-fix branch June 25, 2015 17:42
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.

Refactor code to remove duplication
3 participants