adamdebreceni commented on a change in pull request #976:
URL: https://github.com/apache/nifi-minifi-cpp/pull/976#discussion_r559357694
##########
File path: extensions/windows-event-log/ConsumeWindowsEventLog.h
##########
@@ -133,17 +141,33 @@ class ConsumeWindowsEventLog : public core::Processor {
std::wstring wstrQuery_;
std::string regex_;
bool resolve_as_attributes_;
- bool apply_identifier_function_;
+ bool apply_identifier_function_{false};
std::string provenanceUri_;
std::string computerName_;
uint64_t maxBufferSize_{};
DWORD lastActivityTimestamp_{};
std::mutex cache_mutex_;
std::map<std::string, wel::WindowsEventLogHandler > providers_;
- uint64_t batch_commit_size_;
+ uint64_t batch_commit_size_{};
+
+ struct OutputFormat {
+ void reset() {
+ *this = OutputFormat{};
+ }
+
+ bool xml{false};
+ bool plaintext{false};
+ struct {
+ explicit operator bool() const noexcept {
+ return raw || simple || flattened;
+ }
+
+ bool raw{false};
+ bool simple{false};
+ bool flattened{false};
+ } json;
+ } output_;
Review comment:
but the `reset()` is indeed not needed
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
