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

FileReader.prototype.open breaks for Node.js v10.0.0 #46

Open
bochap opened this issue Oct 7, 2018 · 4 comments
Open

FileReader.prototype.open breaks for Node.js v10.0.0 #46

bochap opened this issue Oct 7, 2018 · 4 comments

Comments

@bochap
Copy link

bochap commented Oct 7, 2018

Line 2388 will cause an exception in Node.js > v10.0.0. The callback parameter is no longer optional for fs.close. Not passing it will throw a TypeError at runtime. There was a depreciation warning since Node.js v7.0.0

@petersirka
Copy link
Owner

Hi @seetD , I need to update the entire source-code. I have new version, but I didn't have any time for update. I will think on it this/next month.

Thx!

@bochap
Copy link
Author

bochap commented Oct 7, 2018

Hey @petersirka,

Thanks for the quick reply. I was looking into a PR but it appears that you might have a whole lot of changes in what I believe is the Total.js repo https://github.com/totaljs/framework/blob/master/nosql.js. It appears to have some major rewrites and other dependencies within the framework so I will workaround it for now. I found the problem while working on code examples from a book and added a reference to an issue I filed there as well.

@GrosSacASac
Copy link

Hi @petersirka , why do you need to update entire source ode for this ? Do you need help ?

@petersirka
Copy link
Owner

Because I rewrote reading/writing data into the file. Currently is very effective and it can stream big data and updates don't modify entire DB file. I will work on it soon, but my time is limited for this module :-( ...

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

No branches or pull requests

3 participants