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
Adding ability to monitor whole folders #784
Conversation
@@ -125,6 +127,10 @@ Status listDirectoriesInDirectory(const boost::filesystem::path& path, | |||
Status resolveFilePattern(const boost::filesystem::path& fs_path, | |||
std::vector<std::string>& results); | |||
|
|||
Status resolveFilePattern(const boost::filesystem::path& fs_path, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you add a doxygen comment indicating what setting
is for?
looks good other than those two comments. |
@@ -245,10 +244,15 @@ Status doubleStarTraversal(const boost::filesystem::path& fs_path, | |||
*/ | |||
Status resolveLastPathComponent(const boost::filesystem::path& fs_path, | |||
std::vector<std::string>& results, | |||
unsigned int setting, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See [1] EnumName setting
1a6e26f
to
70c82b5
Compare
Fixed a bug where when using REC_LIST_FOLDERS, the root resolution directory would not be returned.
EXPECT_EQ(files["system_binaries"].size(), 5); | ||
EXPECT_EQ(files["downloads"].size(), 1); | ||
for (const auto& file : files["system_binaries"]) { | ||
std::cout << file << "\n"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove me plz
@@ -59,7 +59,9 @@ void FileChangesEventSubscriber::init() { | |||
|
|||
for (const auto& element_kv : file_map) { | |||
for (const auto& file : element_kv.second) { | |||
VLOG(1) << "Added Listener To: " << file; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You Don't Need To Cap Every Word :)
Adding ability to monitor whole folders
No description provided.