diff options
author | Benjamin Coe <ben@npmjs.com> | 2018-08-09 16:52:41 -0700 |
---|---|---|
committer | Benjamin Coe <ben@npmjs.com> | 2018-08-11 12:07:32 -0700 |
commit | bdef1b1eb45e2953e1ff68f0cc9a68ec83573e57 (patch) | |
tree | cf12dfaaa414432980c139066aa977ffc0724760 /src/node_file.h | |
parent | e0395247c899af101f8a1f76a8554be1ff14040a (diff) | |
download | android-node-v8-bdef1b1eb45e2953e1ff68f0cc9a68ec83573e57.tar.gz android-node-v8-bdef1b1eb45e2953e1ff68f0cc9a68ec83573e57.tar.bz2 android-node-v8-bdef1b1eb45e2953e1ff68f0cc9a68ec83573e57.zip |
fs: implement mkdir recursive (mkdirp)
Implements mkdirp functionality in node_file.cc. The Benefit
of implementing in C++ layer is that the logic is more easily
shared between the Promise and callback implementation and
there are notable performance improvements.
This commit is part of the Tooling Group Initiative.
Refs: https://github.com/nodejs/user-feedback/pull/70
PR-URL: https://github.com/nodejs/node/pull/21875
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
Reviewed-By: Ron Korving <ron@ronkorving.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: Sam Ruby <rubys@intertwingly.net>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Diffstat (limited to 'src/node_file.h')
-rw-r--r-- | src/node_file.h | 42 |
1 files changed, 42 insertions, 0 deletions
diff --git a/src/node_file.h b/src/node_file.h index fdd36efc77..af62be0fec 100644 --- a/src/node_file.h +++ b/src/node_file.h @@ -21,10 +21,50 @@ using v8::Value; namespace fs { +// structure used to store state during a complex operation, e.g., mkdirp. +class FSContinuationData : public MemoryRetainer { + public: + FSContinuationData(uv_fs_t* req, int mode, uv_fs_cb done_cb) + : req(req), mode(mode), done_cb(done_cb) { + } + + uv_fs_t* req; + int mode; + std::vector<std::string> paths; + + void PushPath(std::string&& path) { + paths.emplace_back(std::move(path)); + } + + void PushPath(const std::string& path) { + paths.push_back(path); + } + + std::string PopPath() { + CHECK_GT(paths.size(), 0); + std::string path = std::move(paths.back()); + paths.pop_back(); + return path; + } + + void Done(int result) { + req->result = result; + done_cb(req); + } + + void MemoryInfo(MemoryTracker* tracker) const override { + tracker->TrackThis(this); + tracker->TrackField("paths", paths); + } + + private: + uv_fs_cb done_cb; +}; class FSReqBase : public ReqWrap<uv_fs_t> { public: typedef MaybeStackBuffer<char, 64> FSReqBuffer; + std::unique_ptr<FSContinuationData> continuation_data = nullptr; FSReqBase(Environment* env, Local<Object> req, AsyncWrap::ProviderType type, bool use_bigint) @@ -97,6 +137,7 @@ class FSReqCallback : public FSReqBase { void MemoryInfo(MemoryTracker* tracker) const override { tracker->TrackThis(this); + tracker->TrackField("continuation_data", continuation_data); } ADD_MEMORY_INFO_NAME(FSReqCallback) @@ -162,6 +203,7 @@ class FSReqPromise : public FSReqBase { void MemoryInfo(MemoryTracker* tracker) const override { tracker->TrackThis(this); tracker->TrackField("stats_field_array", stats_field_array_); + tracker->TrackField("continuation_data", continuation_data); } ADD_MEMORY_INFO_NAME(FSReqPromise) |